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

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

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 <[email protected]> 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
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to