[Bug 500246] Pre-Review Request: plexus-classworlds - Plexus Classworlds Classloader 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=500246 Andrew Overholt overh...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Comment #16 from Andrew Overholt overh...@redhat.com 2009-08-20 11:23:59 EDT --- Built in rawhide: http://koji.fedoraproject.org/koji/taskinfo?taskID=1617781 (Full review was bug #518310) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 500246] Pre-Review Request: plexus-classworlds - Plexus Classworlds Classloader 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=500246 Fernando Nasser fnas...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #14 from Fernando Nasser fnas...@redhat.com 2009-05-17 21:36:18 EDT --- New Package CVS Request === Package Name: plexus-classworlds Short Description: Plexus Classworlds Classloader Framework Owners: fnasser, dbhole Branches: devel InitialCC: jkeating -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 500246] Pre-Review Request: plexus-classworlds - Plexus Classworlds Classloader 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=500246 Kevin Fenzi ke...@tummy.com changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #15 from Kevin Fenzi ke...@tummy.com 2009-05-18 00:23:35 EDT --- cvs done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 500246] Pre-Review Request: plexus-classworlds - Plexus Classworlds Classloader 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=500246 Andrew Overholt overh...@redhat.com changed: What|Removed |Added Flag||fedora-review+ --- Comment #11 from Andrew Overholt overh...@redhat.com 2009-05-15 09:56:51 EDT --- Thanks, Fernando. I'd still like to see some comments about the XML source files stating that they're only for building but that's a Review thing :) I'm okay with the latest SRPM for pre-review. I'm going to set the review flag but someone please change it if that is incorrect. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 500246] Pre-Review Request: plexus-classworlds - Plexus Classworlds Classloader 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=500246 Fernando Nasser fnas...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #12 from Fernando Nasser fnas...@redhat.com 2009-05-15 11:51:47 EDT --- Requested CVS. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 500246] Pre-Review Request: plexus-classworlds - Plexus Classworlds Classloader 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=500246 Kevin Fenzi ke...@tummy.com changed: What|Removed |Added Flag|fedora-cvs? | --- Comment #13 from Kevin Fenzi ke...@tummy.com 2009-05-15 19:58:01 EDT --- Can you add a cvs template here so I know what you need? Reset the cvs flag when ready. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 500246] Pre-Review Request: plexus-classworlds - Plexus Classworlds Classloader 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=500246 Andrew Overholt overh...@redhat.com changed: What|Removed |Added CC||tcall...@redhat.com Flag|needinfo? |needinfo?(tcall...@redhat.c ||om) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 500246] Pre-Review Request: plexus-classworlds - Plexus Classworlds Classloader 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=500246 Andrew Overholt overh...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||overh...@redhat.com AssignedTo|nob...@fedoraproject.org|overh...@redhat.com Flag||needinfo? --- Comment #1 from Andrew Overholt overh...@redhat.com 2009-05-14 09:43:09 EDT --- - I think the license field should be ASL 2.0 and not Apache Software License 2.0 but see my last comment - there are 3 source files (plexus-classworlds-1.2-build.xml, plexus-classworlds-settings.xml, plexus-classworlds-1.2-jpp-depmap.xml) that do not have URLs - the md5sums of the tarball that's in the SRPM and the one that I generate are not equal but a recursive diff of the exploded contents shows not differences so I assume timestamps are the issue - I don't know what license this is. spot, can you please take a look? http://overholt.fedorapeople.org/plexus-classworlds-license.txt -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 500246] Pre-Review Request: plexus-classworlds - Plexus Classworlds Classloader 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=500246 --- Comment #3 from Tom spot Callaway tcall...@redhat.com 2009-05-14 09:52:04 EDT --- Nevermind, looks like it is removed from the spec, but somehow made it into the SRPM... weird. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 500246] Pre-Review Request: plexus-classworlds - Plexus Classworlds Classloader 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=500246 Tom spot Callaway tcall...@redhat.com changed: What|Removed |Added Flag|needinfo?(tcall...@redhat.c | |om) | --- Comment #2 from Tom spot Callaway tcall...@redhat.com 2009-05-14 09:51:04 EDT --- This package's naming is in violation of the Fedora Packaging Guidelines. Please see: http://fedoraproject.org/wiki/Packaging/JPackagePolicy Basically, the jpp tag needs to be removed. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 500246] Pre-Review Request: plexus-classworlds - Plexus Classworlds Classloader 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=500246 --- Comment #4 from Andrew Overholt overh...@redhat.com 2009-05-14 09:57:52 EDT --- I don't see naming guidelines on: https://fedoraproject.org/wiki/User:Toshio/Pre-review_Guidelines -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 500246] Pre-Review Request: plexus-classworlds - Plexus Classworlds Classloader 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=500246 --- Comment #5 from Tom spot Callaway tcall...@redhat.com 2009-05-14 10:01:14 EDT --- Yeah, they're not there for this pre-review, but it is worth pointing out that it would cause an issue at final review. If the issue was there, which it doesn't seem to be. I'm not sure how the SRPM ended up with the jpp tag when the spec inside it doesn't have it. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 500246] Pre-Review Request: plexus-classworlds - Plexus Classworlds Classloader 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=500246 --- Comment #6 from Andrew Overholt overh...@redhat.com 2009-05-14 10:21:33 EDT --- Any comment about the license, spot? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 500246] Pre-Review Request: plexus-classworlds - Plexus Classworlds Classloader 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=500246 --- Comment #7 from Tom spot Callaway tcall...@redhat.com 2009-05-14 10:32:40 EDT --- So, on an initial look, it looks like an Apache 1.1 derived license. I'm waiting on Red Hat Legal for some final details before I can properly answer. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 500246] Pre-Review Request: plexus-classworlds - Plexus Classworlds Classloader 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=500246 --- Comment #8 from Tom spot Callaway tcall...@redhat.com 2009-05-14 15:29:59 EDT --- License is free, but GPL incompatible. For this package, use: License: ASL 2.0 and Plexus -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 500246] Pre-Review Request: plexus-classworlds - Plexus Classworlds Classloader 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=500246 --- Comment #9 from Fernando Nasser fnas...@redhat.com 2009-05-14 22:27:48 EDT --- The 3 files plexus-classworlds-1.2-build.xml, plexus-classworlds-settings.xml, plexus-classworlds-1.2-jpp-depmap.xml are written for this RPM so they have no upstream URL (only needed for the rpmbuild). As for the license, I guess it has to be changed for all plexus-* RPMs, so we will have to recreate/reupload all plexus-* srpms. P.S.: The dist tag expands to different things depending on where you run rpmbuild, so it will take its desired value when rebuilt in Koji. Regards! -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 500246] Pre-Review Request: plexus-classworlds - Plexus Classworlds Classloader 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=500246 --- Comment #10 from Fernando Nasser fnas...@redhat.com 2009-05-14 22:32:50 EDT --- Spec URL: http://people.fedoraproject.org/~fnasser/plexus-classworlds.spec http://people.fedoraproject.org/~fnasser/plexus-classworlds-1.2-0.a9.6.jpp6.src.rpm Have the new license specification. Thanks! -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 500246] Pre-Review Request: plexus-classworlds - Plexus Classworlds Classloader 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=500246 Alexander Kurtakov akurt...@redhat.com changed: What|Removed |Added Blocks||429551 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
