https://bugzilla.redhat.com/show_bug.cgi?id=1680376
--- Comment #4 from Björn 'besser82' Esser <besse...@fedoraproject.org> ---
Thank you for the review!
(In reply to Robert-André Mauchin from comment #3)
> - %{__perl} → perl
I think that is a matter of personal preference.
Your other remarks have been addressed.
***
=== Updated package ===
Changelog:
* Sun Feb 24 2019 Björn Esser <besse...@fedoraproject.org> - 0.003003-0.3
- Changes as suggested in review (#1680376)
- Add a BR for Perl required version
- Add a set of explicit BuildRequires
- Use %%make_install
- Drop cleanups using find
- Drop dist.ini fom %%doc
Urls:
Spec URL:
https://pagure.io/besser82/package-review/raw/master/f/perl-Log-Any-Adapter-TAP.spec
SRPM URL:
https://pagure.io/besser82/package-review/raw/master/f/perl-Log-Any-Adapter-TAP-0.003003-0.3.fc31.src.rpm
Scratch build (rawhide):
https://koji.fedoraproject.org/koji/taskinfo?taskID=33011455
--
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
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives:
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
