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@xxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |volker27@xxxxxx --- Comment #3 from Volker Fröhlich <volker27@xxxxxx> 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 package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review