https://bugzilla.redhat.com/show_bug.cgi?id=1452985
--- Comment #5 from Gerd Pokorra <g...@zimt.uni-siegen.de> --- (In reply to Paul Howarth from comment #4) > A few quick comments: > > I think running rpm database queries in the package build is a bad idea. > This may fail when building in a chroot environment populated using a > version of rpm using a different version of libdb to the target for > instance. Can't you get the rakudo version from rakudo itself? The command perl6 -e 'say $*PERL.version' v6.c do not what we need. As far as I know there is no special variable for it. The only think I know is to filter the output of 'perl6 -v' For example: perl6 -v | head -n1 | sed -e 's/This is Rakudo version //' | sed -e 's/ built on.*// > Do you know of any other perl6 packages in Fedora already? At this stage > there don't appear to be any Perl6 packaging guidelines so it would be good > to at least be consistent with what's there already, if anything. There are currently only the packages MoarVM > NQP > rakudo in Fedora. My be could have a group package (rakudo-star) on top, for installing all the stuff together like it is for livetex. Perhaps the experience of packaging Perl 5 will help. > > I don't think you need to gzip the manpage; rpm should do that itself, and > may choose to use a different compression format in the future. > > You should BR: perl-podlators for pod2man. Consider also optionally adding > BR: for coreutils and sed, which the spec calls for in %install. The new srpm-URL is: ftp://ftp.uni-siegen.de/pub/review/zef/srpm/zef-0.20170520git.1490608-2.fc25.src.rpm ftp://ftp.uni-siegen.de/pub/review/zef/spec/zef.spec links to version 2 of the spec file. - the manpage is not explicitly gzipped any more - add BR: coreutils sed and perl-podlators -- 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
