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



--- Comment #3 from Petr Pisar <[email protected]> ---
(In reply to Xavier Bachelot from comment #2)
> Shouldn't the common::sense BR, be in the "regular" BRs rather than in
> runtime BRs ? Tangerine marks it as COMPILE.
> 
The sections are not about in which phase the Perl interpreter loads them (that
is what tangerine reports).
The sections are about which ExtUtils::MakeMaker/rpmbuild phase use them.
Technically the run-time section could be merged with the tests section, but I
keep them intentionally distinct to be able to cross check with dependencies of
the produced binary RPM package.

The module is used from lib/Net/IDN/PP.pm. That's why I put it into the
run-time section.

That's how I use the sections.

> rpmlint reports E: incorrect-fsf-address
> /usr/share/licenses/perl-Net-IDN-PP/LICENSE. That should be reported
> upstream.
> 
That's futile. The LICENSE file is supplied by Dist::Zilla. Net-IDN-PP upstream
simply forwards what Dist::Zilla produces. Dist::Zilla takes the license texts
from 
Software::License. And Software::License will only accept text used by Perl
foundation <https://dev.perl.org/licenses/gpl1.html>. And changing the text
there is practically impossible. I haven't tried, maybe I could.

> Maybe the patch against tests you submitted upstream could be applied ?
> 
I wanted to give the upstream some time. The patch is not important for Fedora.


-- 
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
https://bugzilla.redhat.com/show_bug.cgi?id=2404119

Report this comment as SPAM: 
https://bugzilla.redhat.com/enter_bug.cgi?product=Bugzilla&format=report-spam&short_desc=Report%20of%20Bug%202404119%23c3

-- 
_______________________________________________
package-review mailing list -- [email protected]
To unsubscribe send an email to [email protected]
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/[email protected]
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue

Reply via email to