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

Sam P. <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|needinfo?([email protected])    |



--- Comment #6 from Sam P. <[email protected]> ---
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 -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to