https://bugzilla.redhat.com/show_bug.cgi?id=2073377
Paul Howarth <paul(a)city-fan.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Flags|needinfo?(paul(a)city-fan.org |
|) |
--- Comment #2 from Paul Howarth <paul(a)city-fan.org> ---
(In reply to Michal Josef Spacek from comment #1)
@paul(a)city-fan.org
"BuildRequires: coreutils"
It is not needed, sure?
%{_fixperms} in the %install section is implemented using chmod from coreutils.
"BuildRequires: perl(strict)"
You could move to "Module Runtime" section
Yes, you're right, it should be there.
"%doc Changes README t/"
No contain t/ in doc
I don't usually package the tests. It's a habit from a long time ago when you
could get additional dependencies from documentation files.
I'm open to persuasion on this.
--
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2073377