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=498246 Jussi Lehtola <jussi.lehtola@xxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs? --- Comment #7 from Jussi Lehtola <jussi.lehtola@xxxxxx> 2009-05-25 03:20:58 EDT --- (In reply to comment #6) > Sorry it took so long. > > (In reply to comment #2) > > No, it is quite standard to ship executable scripts in %doc, as the program > > works without them; they're just examples of use. > > You can ship scripts in %doc if they are not executable, so no problem here. > But IMO you should consider packaging the Examples separately, it's 9 MB one > the disk and as you said people don't necessarily need the files. Done. > Add a conditional openmpi on F-12. The rest is minor, the package is APPROVED > by cwickert Ugh. I'll wait for the openmpi maintainers to add Provides: openmpi-devel (or provide a good excuse not to). Thanks for the review! New Package CVS Request ======================= Package Name: towhee Short Description: A Monte Carlo molecular simulation code Owners: jussilehtola Branches: F-10 F-11 EL-5 InitialCC: -- 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@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review