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 <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]

--- Comment #3 from Volker Fröhlich <[email protected]> 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.
_______________________________________________
package-review mailing list
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to