https://bugzilla.redhat.com/show_bug.cgi?id=1049557



--- Comment #6 from Denis Fateyev <de...@fateyev.com> ---
(In reply to Matthew Miller from comment #5)
> Funny, I was just thinking it'd be handy to have this packaged up. I'll let
> Igor finish the official review but throw in a few comments.
> 
> It's not a big deal but I'd like to see more descriptive Summary --
> something that mentions the error-checking capability, since we have a
> number of other utilities which do the listing and none that do the
> checking. I'd also suggest sneaking '.jpg' into the summary so that comes up
> in people's searches. 

Matthew, thanks for these pretty informative descriptions! I have just added
them into the spec.

I've also ajusted License tag, removed getopt files, and added Provides as
requested.
As for BuildRequires, "libjpeg-devel" is already presented there, seems there
are no other specific dependencies. Please check the spec file if I'm wrong.

Updated spec: http://www.fateyev.com/RPMS/Fedora19/testing/jpeginfo.spec
Source RPM:
http://www.fateyev.com/RPMS/Fedora19/testing/SRPMS/jpeginfo-1.6.1-1.fc19.denf.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to