Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the App-Licensecheck CPAN distribution.

Report information
The Basics
Id: 131925
Status: rejected
Priority: 0/
Queue: App-Licensecheck

People
Owner: Nobody in particular
Requestors: SREZIC [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: v3.0.45
Fixed in: (no value)



Subject: re::engine::RE2 needed?
re::engine::RE2 is a problematic dependency --- there are no pass reports since perl 5.23.4, see http://matrix.cpantesters.org/?dist=re-engine-RE2+0.13 However, it seems that re::engine::RE2 is not used at all in the distribution. Maybe it simply can be removed as a dependency?
Subject: Re: [rt.cpan.org #131925] re::engine::RE2 needed?
Date: Sat, 22 Feb 2020 11:12:03 +0100
To: bug-App-Licensecheck [...] rt.cpan.org
From: Jonas Smedegaard <jonas [...] jones.dk>
Quoting Jonas Smedegaard (2020-02-22 11:09:46) Show quoted text
> Without re::engine::RE2, App::Licensecheck performs 10-30 times > slower.
Sorry, only up to 3 times slower. - Jonas -- * Jonas Smedegaard - idealist & Internet-arkitekt * Tlf.: +45 40843136 Website: http://dr.jones.dk/ [x] quote me freely [ ] ask before reusing [ ] keep private
Download signature.asc
application/pgp-signature 833b

Message body not shown because it is not plain text.

Subject: Re: [rt.cpan.org #131925] re::engine::RE2 needed?
Date: Sat, 22 Feb 2020 11:09:46 +0100
To: bug-App-Licensecheck [...] rt.cpan.org
From: Jonas Smedegaard <jonas [...] jones.dk>
Quoting Slaven_Rezic via RT (2020-02-22 10:06:53) Show quoted text
> re::engine::RE2 is a problematic dependency --- there are no pass > reports since perl 5.23.4, see > http://matrix.cpantesters.org/?dist=re-engine-RE2+0.13
Works fine when linked against system shared libre2 library in Debian: https://buildd.debian.org/status/logs.php?pkg=libre-engine-re2-perl Debian (and as I recall other distributions as well) carry patches for several bugreports, including https://rt.cpan.org/Public/Bug/Display.html?id=108357 and https://rt.cpan.org/Public/Bug/Display.html?id=129585 Show quoted text
> However, it seems that re::engine::RE2 is not used at all in the > distribution. Maybe it simply can be removed as a dependency?
Without re::engine::RE2, App::Licensecheck performs 10-30 times slower. - Jonas -- * Jonas Smedegaard - idealist & Internet-arkitekt * Tlf.: +45 40843136 Website: http://dr.jones.dk/ [x] quote me freely [ ] ask before reusing [ ] keep private
Download signature.asc
application/pgp-signature 833b

Message body not shown because it is not plain text.

On 2020-02-22 05:17:13, jonas@jones.dk wrote: Show quoted text
> Quoting Slaven_Rezic via RT (2020-02-22 10:06:53)
> > re::engine::RE2 is a problematic dependency --- there are no pass > > reports since perl 5.23.4, see > > http://matrix.cpantesters.org/?dist=re-engine-RE2+0.13
> > Works fine when linked against system shared libre2 library in Debian: > https://buildd.debian.org/status/logs.php?pkg=libre-engine-re2-perl > > Debian (and as I recall other distributions as well) carry patches for > several bugreports, including > https://rt.cpan.org/Public/Bug/Display.html?id=108357 and > https://rt.cpan.org/Public/Bug/Display.html?id=129585 > >
> > However, it seems that re::engine::RE2 is not used at all in the > > distribution. Maybe it simply can be removed as a dependency?
> > Without re::engine::RE2, App::Licensecheck performs 10-30 times slower.
OK. I was grepping for occurrence of the string "re::engine::RE2" in the distribution, and did not found anything, so it was my impression that the module was not used. But now I found the occurrence in Regexp::Pattern::License. I'll reject the ticket then.
On 2020-02-22 08:21:46, SREZIC wrote: Show quoted text
> On 2020-02-22 05:17:13, jonas@jones.dk wrote:
> > Quoting Slaven_Rezic via RT (2020-02-22 10:06:53)
> > > re::engine::RE2 is a problematic dependency --- there are no pass > > > reports since perl 5.23.4, see > > > http://matrix.cpantesters.org/?dist=re-engine-RE2+0.13
> > > > Works fine when linked against system shared libre2 library in > > Debian: > > https://buildd.debian.org/status/logs.php?pkg=libre-engine-re2-perl > > > > Debian (and as I recall other distributions as well) carry patches > > for > > several bugreports, including > > https://rt.cpan.org/Public/Bug/Display.html?id=108357 and > > https://rt.cpan.org/Public/Bug/Display.html?id=129585 > > > >
> > > However, it seems that re::engine::RE2 is not used at all in the > > > distribution. Maybe it simply can be removed as a dependency?
> > > > Without re::engine::RE2, App::Licensecheck performs 10-30 times > > slower.
> > OK. I was grepping for occurrence of the string "re::engine::RE2" in > the distribution, and did not found anything, so it was my impression > that the module was not used. But now I found the occurrence in > Regexp::Pattern::License. > > I'll reject the ticket then.
(Though from a compatibility aspect it would be better to make re::engine::RE2 an optional dependency, so users of other distributions/operating systems may also use this module/script).
Subject: Re: [rt.cpan.org #131925] re::engine::RE2 needed?
Date: Sat, 22 Feb 2020 14:44:00 +0100
To: , Slaven_Rezic via RT <bug-App-Licensecheck [...] rt.cpan.org>
From: Jonas Smedegaard <jonas [...] jones.dk>
Quoting Slaven_Rezic via RT (2020-02-22 14:23:14) Show quoted text
> Queue: App-Licensecheck > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=131925 > > > On 2020-02-22 08:21:46, SREZIC wrote:
> > On 2020-02-22 05:17:13, jonas@jones.dk wrote:
> > > Quoting Slaven_Rezic via RT (2020-02-22 10:06:53)
> > > > re::engine::RE2 is a problematic dependency --- there are no pass > > > > reports since perl 5.23.4, see > > > > http://matrix.cpantesters.org/?dist=re-engine-RE2+0.13
> > > > > > Works fine when linked against system shared libre2 library in > > > Debian: > > > https://buildd.debian.org/status/logs.php?pkg=libre-engine-re2-perl > > > > > > Debian (and as I recall other distributions as well) carry patches > > > for > > > several bugreports, including > > > https://rt.cpan.org/Public/Bug/Display.html?id=108357 and > > > https://rt.cpan.org/Public/Bug/Display.html?id=129585 > > > > > >
> > > > However, it seems that re::engine::RE2 is not used at all in the > > > > distribution. Maybe it simply can be removed as a dependency?
> > > > > > Without re::engine::RE2, App::Licensecheck performs 10-30 times > > > slower.
> > > > OK. I was grepping for occurrence of the string "re::engine::RE2" in > > the distribution, and did not found anything, so it was my impression > > that the module was not used. But now I found the occurrence in > > Regexp::Pattern::License. > > > > I'll reject the ticket then.
> > (Though from a compatibility aspect it would be better to make re::engine::RE2 an optional dependency, so users of other distributions/operating systems may also use this module/script).
I certainly agree. Just haven't figured out a way to do that yet. - Jonas -- * Jonas Smedegaard - idealist & Internet-arkitekt * Tlf.: +45 40843136 Website: http://dr.jones.dk/ [x] quote me freely [ ] ask before reusing [ ] keep private
Download signature.asc
application/pgp-signature 833b

Message body not shown because it is not plain text.