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



--- Comment #2 from Denis Fateyev <[email protected]> ---
Thanks for the review.

> Please, fix these directories
> %{buildroot}/%{_bindir}/jpegoptim
> %{buildroot}/%{_mandir}/man1/jpegoptim.1
> there is an extra slash.

Right, but it also helps to split the path visually ;) I'm aware of extra
slashes but use them since there is no explicit prohibition in guidelines.

> I don't know if you already know that your package is not built 
> in epel5 because of old MD5 used.

Thanks for the reminder, it's related to an old MD5 checking bug existing in
RHEL5. I think it won't affect "fedpkg build" since it imports only source
files into Git unlike "koji" which builds packages from SRPMs.

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

Reply via email to