Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the CGI-Github-Webhook CPAN distribution.

Report information
The Basics
Id: 111076
Status: resolved
Priority: 0/
Queue: CGI-Github-Webhook

People
Owner: abe+cpan [...] deuxchevaux.org
Requestors: SREZIC [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: 0.01
Fixed in: 0.02



Subject: System perl used in tests
Tests fail on Debian systems, probably because the system perl is used (@INC of the following test output excerpt contains /etc/perl, which is an indication for Debian's /usr/bin/perl): # Failed test 'CGI output as expected' # at t/basic-functionality.t line 47. # got: 'Can't locate Moo.pm in @INC (@INC contains: t/../lib /tmpfs/.cpan-build/2016011106/CGI-Github-Webhook-0.01-pq1fDo/blib/lib /tmpfs/.cpan-build/2016011106/CGI-Github-Webhook-0.01-pq1fDo/blib/arch /etc/perl /usr/local/lib/perl/5.14.2 /usr/local/share/perl/5.14.2 /usr/lib/perl5 /usr/share/perl5 /usr/lib/perl/5.14 /usr/share/perl/5.14 /usr/local/lib/site_perl .) at t/../lib/CGI/Github/Webhook.pm line 11. # BEGIN failed--compilation aborted at t/../lib/CGI/Github/Webhook.pm line 11. # Compilation failed in require at t/cgi/basic.pl line 10. # BEGIN failed--compilation aborted at t/cgi/basic.pl line 10. # ' Probably you have to use somewhere in the tests $^X instead of "perl".
Subject: Re: [rt.cpan.org #111076] System perl used in tests
Date: Mon, 11 Jan 2016 09:57:30 +0100
To: Slaven_Rezic via RT <bug-CGI-Github-Webhook [...] rt.cpan.org>
From: Axel Beckert <abe [...] deuxchevaux.org>
Hi Slaven, On Mon, Jan 11, 2016 at 01:18:46AM -0500, Slaven_Rezic via RT wrote: Show quoted text
> Tests fail on Debian systems, probably because the system perl is > used (@INC of the following test output excerpt contains /etc/perl, > which is an indication for Debian's /usr/bin/perl):
Thanks for this analysis. I already looked at http://www.cpantesters.org/distro/C/CGI-Github-Webhook.html#CGI-Github-Webhook-0.01 and wondered where those failures come from. :-) Show quoted text
> Probably you have to use somewhere in the tests $^X instead of > "perl".
Ah, thanks for that hint. It's probably the shebang line in the test suite. Would "#!/usr/bin/env perl" alternatively also help there? Kind regards, Axel -- /~\ Plain Text Ribbon Campaign | Axel Beckert \ / Say No to HTML in E-Mail and News | abe@deuxchevaux.org (Mail) X See http://www.nonhtmlmail.org/campaign.html | abe@noone.org (Mail+Jabber) / \ I love long mails: http://email.is-not-s.ms/ | http://abe.noone.org/ (Web)
Subject: Re: [rt.cpan.org #111076] System perl used in tests
Date: Mon, 11 Jan 2016 09:59:59 +0100
To: Axel Beckert via RT <bug-CGI-Github-Webhook [...] rt.cpan.org>
From: Axel Beckert <abe [...] deuxchevaux.org>
Hi again, On Mon, Jan 11, 2016 at 03:57:49AM -0500, Axel Beckert via RT wrote: Show quoted text
> > Probably you have to use somewhere in the tests $^X instead of > > "perl".
> > Ah, thanks for that hint. It's probably the shebang line in the test > suite. Would "#!/usr/bin/env perl" alternatively also help there?
Let me answer that myself: No, it wouldn't. I already use just "perl" to call the scripts. Also works fine with Travis-CI (which fiddles with $PATH for that). Kind regards, Axel -- /~\ Plain Text Ribbon Campaign | Axel Beckert \ / Say No to HTML in E-Mail and News | abe@deuxchevaux.org (Mail) X See http://www.nonhtmlmail.org/campaign.html | abe@noone.org (Mail+Jabber) / \ I love long mails: http://email.is-not-s.ms/ | http://abe.noone.org/ (Web)
Hi Slaven, should be fixed in 0.02 I uploaded earlier this morning.
On 2016-01-11 05:52:56, ABE wrote: Show quoted text
> Hi Slaven, > > should be fixed in 0.02 I uploaded earlier this morning.
Hi Axel, looks good now: http://matrix.cpantesters.org/?dist=CGI-Github-Webhook%200.02 Regards, Slaven