https://bugzilla.redhat.com/show_bug.cgi?id=915791 --- Comment #19 from Petr Pisar <ppisar@xxxxxxxxxx> --- (In reply to Petr Šabata from comment #18) > * Several of the listed build dependencies don't appear to be used at build > time, including: > - perl(DBD::SQLite) This is used when running t/store-sqlite.t. > - perl(Fcntl) Removed. > - perl(Perlbal) Removed. > - perl(Perlbal::Socket) Removed. > - perl(Perlbal::TCPListener) Removed. > - net-tools Removed. > > * You could use the NO_PACKLIST feature. > Done. > * mogstored is missing some runtime dependencies, namely: > - perl(Mogstored::ChildProcess::DiskUsage) Added. > - perl(Mogstored::ChildProcess::IOStat) Added. > - perl(Pod::Usage) Added. > * Does it make sense to have the None backend in a separate subpackage? > For the symmetry. > * To me, "Same terms as Perl itself. Artistic/GPLv2, at your choosing" > doesn't read as "(GPL+ or Artistic) and (GPLv2 or Artistic)" but more like > "GPL+ or GPLv2 or Artistic". > I can see your point. My reading of the second sentence is this is an explanation (a wrong one) of the first sentence. Your reading can be simplified to "GPL+ or Artistic". My reading can be simplified to "GPLv2 or Artistic". I will use stricter "GPLv2 or Artistic" that conform to both readings and I will try to ask the author. Spec URL: https://ppisar.fedorapeople.org/perl-MogileFS-Server/perl-MogileFS-Server.spec SRPM URL: https://ppisar.fedorapeople.org/perl-MogileFS-Server/perl-MogileFS-Server-2.72-1.fc27.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 To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx