[Bug 761063] Review Request: cover_grabber - Download album cover art

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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=761063

--- Comment #7 from Terje Røsten <terjeros@xxxxxxxxxxxx> 2011-12-07 15:59:15 EST ---

I like line up the tags like this:

Summary:  Download album cover art
Name:     cover_grabber
Version:  0.0.2
Release:  1%{?dist}

you might want that.

Your BuildRoot: tag seems too simple. (Or you can remove it.)


%setup -q cover_grabber-0.0.2

You can drop the cover_grabber-0.0.2 here


Please use the %changelog and increase release when you do changes.
Also innclude links to updated spec and srpm in each update comment.
It's them much simpler for the reviewer to follow the process.

koji build looks good, thanks.

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



[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]