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: libdv https://bugzilla.redhat.com/show_bug.cgi?id=226000 ------- Additional Comments From matthias@xxxxxxxxxxxx 2007-09-12 10:05 EST ------- Yeah, kill the tabs! :-) One more minor change I forgot : The man pages in %files aren't listed consistently, or even right. They should all be foo.1* with no foo.1.gz since the compression used is an rpm build time option. I also just double checked a rebuild, and it seems that this package got affected by the recent rpm/popt split : checking for poptGetContext in -lpopt... no configure: WARNING: playdv needs the popt package for command line parsing. The source archive for popt is available at: ftp://ftp.redhat.com/pub/redhat/code/popt So you need to add "BuildRequires: popt-devel". Apart from this, the package looks good. -- Configure bugmail: https://bugzilla.redhat.com/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