[Bug 613857] Review Request: uvcdynctrl - Command line interface to libwebcam

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


https://bugzilla.redhat.com/show_bug.cgi?id=613857

Volker Fröhlich <volker27 at gmx.at> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |volker27 at gmx.at

--- Comment #3 from Volker Fröhlich <volker27 at gmx.at> 2010-07-13 05:16:35 EDT ---
Just what I found looking over it:

The release number should be something like 1.20100322svn%{?dist}, if I'm not
wrong.
  You could also add the tar command. Is there a specific reason you use .xz?

http://fedoraproject.org/wiki/Packaging/NamingGuidelines#Package_Release

I don't know if it's useful to have a base package that requires it's own
sub-package.

There is no package libwebcam-devel, as you state in the BR. Libwebcam seems
bundled in the tarball. Libraries should not be bundled. So will most likely
have to create two separate packages. Besides that, there are two licenses in
the libwebcam sub-directory. Please make sure, which one applies.

http://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries

You will also need ldconfig calls when shipping a library.

http://fedoraproject.org/wiki/Packaging:Guidelines#Shared_Libraries

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