[Bug 464429] Review Request: judy - C Library for Dynamic Sparse Arrays

bugzilla at redhat.com bugzilla at redhat.com
Tue Oct 7 01:07:03 UTC 2008


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=464429





--- Comment #3 from Charles R. Anderson <cra at wpi.edu>  2008-10-06 21:07:02 EDT ---
Ok, so I missed this one and made my own:

Spec URL: http://cra.fedorapeople.org/Judy/Judy.spec
SRPM URL: http://cra.fedorapeople.org/Judy/Judy-1.0.4-1.fc9.src.rpm

Some differences I note:

- I named mine "Judy" not "judy".  This matches the upstream tarball name and
hence doesn't require %setup -n Judy-%{version}.  I believe this is desirable
according to the Naming Guidelines:

"General Naming

When naming a package, the name should match the upstream tarball or project
name from which this software came. In some cases, this naming choice may be
more complicated. If this package has been packaged by other
distributions/packagers in the past, then you should try to match their name
for consistency. In any case, try to use your best judgement, and other
developers will help in the final decision."

- "%configure --disable-static" to prevent building static libraries
altogether--better than just deleting them afterwards.

- I packaged the HTML versions of the docs.

Would you consider my changes and like to co-maintain this package once it gets
in?

-- 
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.




More information about the Fedora-package-review mailing list