Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: libdirac - Dirac is an open source video codec https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=221405 rpm@xxxxxxxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED ------- Additional Comments From rpm@xxxxxxxxxxxxxx 2007-01-20 17:23 EST ------- Per http://fedoraproject.org/wiki/Packaging/NamingGuidelines#head-cfd71146dbb6f00cec9fe3623ea619f843394837 the release field should be: Release: 5.%{cvs}cvs%{?dist} Why is the testsuite disabled? I don't like /usr/bin/ binaries in a package called libwhatever. Moreover, if the source tarball is called dirac, why is the main package called libdirac? Please put the binaries in a package named dirac There are two redundant BuildRequires: libtool requires automake and automake requires autoconf. Builds fine in mock/x86_64/devel. Shouldn't package docs be in %{_datadir}/doc/dirac-docs-%{version} ? -- 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