[Bug 1367569] Review Request: perl-PFT - Hacker friendly static blog generator, core library

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

 



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



--- Comment #9 from Giovanni <dacav@xxxxxxxxxxxxxxx> ---
Hello Parag

In reply to Comment 7:

1) Done. I found a bit weird the fact that rpmdev-bumpspec is not using the
angular bracket. However I've fixed it manually.

2) I've fixed the README files upstream, however the SPEC file is pointing with
Source0 to the archive of version 1.0.3. In order to enable the new README I
should bump the version of the whole package and probably re-publish on CPAN
too. The same applies to APP::PFT 1.0.5.

3) Thanks for the suggestion. I've used tangerine as suggested, and you will
find the updated SPEC (links follow). Just out of curiosity I've checked with
mock if the RPM was constructed with the original set of `BuildRequires:`, and
it turned out to be feasible nevertheless (the missing `BuildRequires:` were
not installed as transitive dependencies!). I think however it's good to be
conservative and to rely on the tangerine tool.

Finally, I'm currently involved in the review of someone else's package:
https://bugzilla.redhat.com/show_bug.cgi?id=1307271

SPEC:
http://copr-dist-git.fedorainfracloud.org/cgit/dacav/pft/perl-PFT.git/plain/perl-PFT.spec
SRPM:
https://copr-be.cloud.fedoraproject.org/results/dacav/pft/fedora-rawhide-x86_64/00446658-perl-PFT/perl-PFT-1.0.3-4.fc26.src.rpm

-- 
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
https://lists.fedoraproject.org/admin/lists/package-review@xxxxxxxxxxxxxxxxxxxxxxx




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