Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: miglayout - Versatile and flexible Swing and SWT layout manager
https://bugzilla.redhat.com/show_bug.cgi?id=644135
Summary: Review Request: miglayout - Versatile and flexible Swing and SWT layout manager Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: eric@brouhaha.com QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora
Spec URL: http://fedorapeople.org/~brouhaha/miglayout/miglayout.spec SRPM URL: http://fedorapeople.org/~brouhaha/miglayout/miglayout-3.7.3.1-1.fc12.src.rpm Koji scratch build for F13: http://koji.fedoraproject.org/koji/taskinfo?taskID=2541597 Koji scratch build for F14: http://koji.fedoraproject.org/koji/taskinfo?taskID=2541599
Description:
MiGLayout is a versatile SWT/Swing layout manager. It uses String or API type-checked constraints to format the layout. MiGLayout can produce flowing, grid based, absolute (with links), grouped and docking layouts. MiGLayout is created to be to manually coded layouts what Matisse/GroupLayout is to IDE supported visual layouts.
Additional note to potential reviewers:
I am packaging miglayout because I want to package replicatorg, which is software to control 3D printers such as RepRap and MakerBot, and replicatorg depends on miglayout. Also, miglayout has been proposed for inclusion in a future release of the JDK, so this will allow those that develop Java software on Fedora to get a headstart with it.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=644135
Hans de Goede hdegoede@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |hdegoede@redhat.com AssignedTo|nobody@fedoraproject.org |hdegoede@redhat.com
--- Comment #1 from Hans de Goede hdegoede@redhat.com 2010-11-03 10:38:22 EDT --- Oh, goodie someone has alreayd packaged up miglayout, as miglayout is a new dep for upgrading freecol to the latest upstream release. I'm very interested in this!
I'll start a review right away.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=644135
Hans de Goede hdegoede@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-review+
--- Comment #2 from Hans de Goede hdegoede@redhat.com 2010-11-03 10:53:21 EDT --- Full review done:
Good: - rpmlint checks return: miglayout.noarch: W: no-documentation miglayout-javadoc.noarch: W: spelling-error Summary(en_US) Javadocs -> Java docs, Java-docs, Javanese 4 packages and 0 specfiles checked; 0 errors, 2 warnings. These can all be ignored - package meets naming guidelines - package meets packaging guidelines - license (BSD) OK, text not in %doc, but that is normal for BSD, all source files do have BSD copyright headers - spec file legible, in am. english - source matches upstream - package compiles on f14 (x86_64) - no missing BR - no unnecessary BR - no locales - not relocatable - owns all directories that it creates - no duplicate files - permissions ok - %clean ok - macro use consistent - code, not content - javadocs in -javadoc subpackage - nothing in %doc affects runtime - no need for .desktop file
Now that is how easy all reviews should be, this is one well done package :)
APPROVED!
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=644135
--- Comment #3 from Hans de Goede hdegoede@redhat.com 2010-11-04 10:53:58 EDT --- Note I would be grateful if you could review one of the other 2 packages needed to be able to update freecol in return :) Bug 649777 - Review Request: jorbis - Pure Java Ogg Vorbis Decoder Bug 649781 - Review Request: cortado - Java media framework
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=644135
Eric Smith eric@brouhaha.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
--- Comment #4 from Eric Smith eric@brouhaha.com 2010-11-04 13:07:57 EDT --- New Package SCM Request ======================= Package Name: miglayout Short Description: Versatile and flexible Swing and SWT layout manager Owners: brouhaha Branches: F13 F14
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=644135
--- Comment #5 from Jason Tibbitts tibbs@math.uh.edu 2010-11-05 13:25:31 EDT --- Git done (by process-git-requests).
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=644135
--- Comment #6 from Hans de Goede hdegoede@redhat.com 2010-11-11 07:24:14 EST --- Ping? Can we please get this build for at least rawhide?
Thanks,
Hans
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=644135
--- Comment #7 from Eric Smith eric@brouhaha.com 2010-11-11 12:11:17 EST --- I'm having trouble importing it into git; there's some problem with the lookaside upload of the sources failing. I've verified that my certificate is good, and even tried generating and installing a new certificate, but it's still not working.
My laptop is running Fedora 12; perhaps there was some regression in one of the updates that broke the fedpkg stuff, which was working for me previously. I need to back it up and upgrade to Fedora 14. I'm hoping to have time to do that this weekend.
Sorry about the delay.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=644135
--- Comment #8 from Jason Tibbitts tibbs@math.uh.edu 2010-11-11 14:23:44 EST --- F12 should work, but you do need the nss update that was released some time ago. I have nss-3.12.8-2.fc12.x86_64 and don't have any issues.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=644135
--- Comment #9 from Eric Smith eric@brouhaha.com 2010-11-11 15:19:27 EST --- Thanks Jason, that was the problem. I thought I'd gotten all the updates, but somehow missed some.
I've submitted a build for rawhide, and builds for F13 and F14 are in progress.
Hans, if I have time this weekend I'll review your packages, if someone else hasn't already.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=644135
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |MODIFIED
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=644135
--- Comment #10 from Fedora Update System updates@fedoraproject.org 2010-11-11 15:30:47 EST --- miglayout-3.7.3.1-1.fc13 has been submitted as an update for Fedora 13. https://admin.fedoraproject.org/updates/miglayout-3.7.3.1-1.fc13
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=644135
--- Comment #11 from Hans de Goede hdegoede@redhat.com 2010-11-12 03:13:16 EST --- Hi,
(In reply to comment #9)
Thanks Jason, that was the problem. I thought I'd gotten all the updates, but somehow missed some.
I've submitted a build for rawhide, and builds for F13 and F14 are in progress.
Thanks, great!
Hans, if I have time this weekend I'll review your packages, if someone else hasn't already.
jorbis has already been reviewed. But if you could do cortado that would be great: Bug 649781 - Review Request: cortado - Java media framework
Note please don't feel obligated to do this, I reviewed miglayout because freecol needs it, iow for my own reasons. But if you could review cortado in return I would be very grateful.
Regards,
Hans
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=644135
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|MODIFIED |ON_QA
--- Comment #12 from Fedora Update System updates@fedoraproject.org 2010-11-13 17:06:19 EST --- miglayout-3.7.3.1-1.fc13 has been pushed to the Fedora 13 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update miglayout'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/miglayout-3.7.3.1-1.fc13
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=644135
--- Comment #13 from Fedora Update System updates@fedoraproject.org 2010-11-29 16:36:59 EST --- miglayout-3.7.3.1-1.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=644135
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Fixed In Version| |miglayout-3.7.3.1-1.fc13 Resolution| |ERRATA Last Closed| |2010-11-29 16:37:03
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=644135
Eric Smith eric@brouhaha.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs+ |fedora-cvs?
--- Comment #14 from Eric Smith eric@brouhaha.com 2012-03-11 03:26:51 EDT --- Package Change Request ====================== Package Name: miglayout New Branches: el6 Owners: brouhaha
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=644135
--- Comment #15 from Jon Ciesla limburgher@gmail.com 2012-03-12 08:04:08 EDT --- Git done (by process-git-requests).
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=644135
--- Comment #16 from Fedora Update System updates@fedoraproject.org 2012-03-12 15:25:22 EDT --- miglayout-4.0-2.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/miglayout-4.0-2.el6
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=644135
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version|miglayout-3.7.3.1-1.fc13 |miglayout-4.0-2.el6
--- Comment #17 from Fedora Update System updates@fedoraproject.org 2012-03-28 16:37:51 EDT --- miglayout-4.0-2.el6 has been pushed to the Fedora EPEL 6 stable repository.
package-review@lists.fedoraproject.org