[Bug 226000] Merge Review: libdv

[bugzilla at redhat.com]

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 at rpmforge.net  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.