Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: concurrent https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225653 ------- Additional Comments From pcheung@xxxxxxxxxx 2007-04-04 15:18 EST ------- (In reply to comment #4) ... > > > > > This is OK as Public Domain, please see > > https://www.redhat.com/archives/fedora-packaging/2007-March/msg00142.html > Right, I know public domain is acceptable, but is this project really public > domain if it has that clause in there? I've listed the Sun License and a link to the pdf in the original mail, and it was described as Public Domain, feel free to further discuss that in that discussion thread if you see fit. ... > X Do we want to be advertising for that book? Good catch, got rid of it. ... > * consider using cp -p to preserve timestamps > X missing a -p on the first cp in %prep Fixed. > rpm -qp --requires concurrent-javadoc-1.3.4-5jpp.1.fc7.x86_64.rpm > /bin/ln > /bin/rm > /bin/rm > > X it should not need a requires on these Fixed. New spec file and srpm uploaded at the same location. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review