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=708711 Volker Fröhlich <volker27@xxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |volker27@xxxxxx Bug 708711 depends on bug 708554, which changed state. Bug 708554 Summary: Review Request: umph - Command line tool for parsing video links from Youtube feeds https://bugzilla.redhat.com/show_bug.cgi?id=708554 What |Old Value |New Value ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Status|ASSIGNED |MODIFIED Status|MODIFIED |ON_QA Resolution| |ERRATA Status|ON_QA |CLOSED --- Comment #8 from Volker Fröhlich <volker27@xxxxxx> 2011-08-27 03:06:23 EDT --- License is GPLv3+, according to the headers. Buildroot, clean section and the rm in the install section are only necessary, when you plan to go for EPEL5 or older. Defattr is no longer necessary. Please handle the locales as described in http://fedoraproject.org/wiki/Packaging:Guidelines#Handling_Locale_Files. You won't need gettext though, but you'll have to append --with-qt to the %find_lang call. I recommend putting each BuildRequires and Requires on a separate line. -- 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