[Bug 1362130] Review Request: olpcau-abc123-fonts - A nice font for kids/ readability

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

 



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

Sam P. <sam@sam.today> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|needinfo?(sam@sam.today)    |



--- Comment #6 from Sam P. <sam@sam.today> ---
Thanks for the review Zbigniew!

> I don't think you have to copy sources in %prep. This breaks 'fedpkg local' builds too.

I tried removing the copy, and it continued to work via mock and rpmbuild
locally on my computer.  However, it caused it to fail when building on COPR
[1].  So I kept it in the end.

I've updated the package:

Spec URL: 
http://copr-dist-git.fedorainfracloud.org/cgit/samtoday/olpcau-abc123-fonts/olpcau-abc123-fonts.git/plain/olpcau-abc123-fonts.spec?id=68ad2759b09e46cd16f1ac7e3935c6bcce1766a7 
SRPM URL: 
https://copr-be.cloud.fedoraproject.org/results/samtoday/olpcau-abc123-fonts/fedora-25-x86_64/00476460-olpcau-abc123-fonts/olpcau-abc123-fonts-20130716-2.fc25.src.rpm

[1]
https://copr.fedorainfracloud.org/coprs/samtoday/olpcau-abc123-fonts/build/476456/
and
https://copr-be.cloud.fedoraproject.org/results/samtoday/olpcau-abc123-fonts/fedora-25-x86_64/00476456-olpcau-abc123-fonts/build.log.gz

-- 
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@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx




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