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.leht...@iki.fi> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|                            |fedora-cvs?




--- Comment #7 from Jussi Lehtola <jussi.leht...@iki.fi>  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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

Reply via email to