Bug #124777 for Time-HiRes: Comment in qw(...) (1.9755)

Wed Mar 14 17:28:30 2018 SREZIC [...] cpan.org - Ticket created

Subject:

Comment in qw(...) (1.9755)

On one smoker I see these warnings while running make test: ... Possible attempt to put comments in qw() list at /home/eserte/.cpan/build/2018031418/Time-HiRes-1.9755-KbUE4j/blib/lib/Time/HiRes.pm line 54. Possible attempt to separate words with commas at /home/eserte/.cpan/build/2018031418/Time-HiRes-1.9755-KbUE4j/blib/lib/Time/HiRes.pm line 54. t/alarm.t ......... ok ... Probably caused by the comments in the @EXPORT_OK = qw(...) block --- these comments are not really comments, but real list elements. So we can import now funny things: $ perl5.18.4 -Mblib -MTime::HiRes=TODO: -e1 $ perl5.18.4 -Mblib -MTime::HiRes=somehow. -e1 $ perl5.18.4 -Mblib -MTime::HiRes=suprerset -e1 (All sample oneliners without errors)

Wed Mar 14 20:14:06 2018 JHI [...] cpan.org - Correspondence added

On Wed Mar 14 17:28:30 2018, SREZIC wrote: Show quoted text

> On one smoker I see these warnings while running make test: > > ... > Possible attempt to put comments in qw() list at > /home/eserte/.cpan/build/2018031418/Time-HiRes-1.9755- > KbUE4j/blib/lib/Time/HiRes.pm line 54. > Possible attempt to separate words with commas at > /home/eserte/.cpan/build/2018031418/Time-HiRes-1.9755- > KbUE4j/blib/lib/Time/HiRes.pm line 54. > t/alarm.t ......... ok > ... > > Probably caused by the comments in the @EXPORT_OK = qw(...) block --- > these comments are not really comments, but real list elements. So we > can import now funny things: > > $ perl5.18.4 -Mblib -MTime::HiRes=TODO: -e1 > $ perl5.18.4 -Mblib -MTime::HiRes=somehow. -e1 > $ perl5.18.4 -Mblib -MTime::HiRes=suprerset -e1 > > (All sample oneliners without errors)

Duh, fixed (and 1.9755 set to delete in three days). I am just wondering how my 5.26.1 testing did not spew these warnings.

Wed Mar 14 20:14:07 2018 The RT System itself - Status changed from 'new' to 'open'

Thu Mar 15 03:32:05 2018 SREZIC [...] cpan.org - Correspondence added

On 2018-03-14 20:14:06, JHI wrote: Show quoted text

> On Wed Mar 14 17:28:30 2018, SREZIC wrote:

> > On one smoker I see these warnings while running make test: > > > > ... > > Possible attempt to put comments in qw() list at > > /home/eserte/.cpan/build/2018031418/Time-HiRes-1.9755- > > KbUE4j/blib/lib/Time/HiRes.pm line 54. > > Possible attempt to separate words with commas at > > /home/eserte/.cpan/build/2018031418/Time-HiRes-1.9755- > > KbUE4j/blib/lib/Time/HiRes.pm line 54. > > t/alarm.t ......... ok > > ... > > > > Probably caused by the comments in the @EXPORT_OK = qw(...) block --- > > these comments are not really comments, but real list elements. So we > > can import now funny things: > > > > $ perl5.18.4 -Mblib -MTime::HiRes=TODO: -e1 > > $ perl5.18.4 -Mblib -MTime::HiRes=somehow. -e1 > > $ perl5.18.4 -Mblib -MTime::HiRes=suprerset -e1 > > > > (All sample oneliners without errors)

> > Duh, fixed (and 1.9755 set to delete in three days). > > I am just wondering how my 5.26.1 testing did not spew these warnings.

On my smokers the warnings happened only with one perl out of 131. strace reveals the difference: 9310 08:28:48.390740 execve("/opt/perl-5.26.1/bin/perl", ["/opt/perl-5.26.1/bin/perl", "t/alarm.t"], [/* 70 vars */] <unfinished ...> 5431 08:27:23.308484 execve("/opt/perl-5.18.4/bin/perl", ["/opt/perl-5.18.4/bin/perl", "-w", "t/alarm.t"], [/* 69 vars */] <unfinished ...> So it seems that there's a something (a particular version of Test::Harness?) which sets -w.

Thu Mar 15 09:07:10 2018 jhi [...] iki.fi - Correspondence added

Subject:	Re: [rt.cpan.org #124777] Comment in qw(...) (1.9755)
Date:	Thu, 15 Mar 2018 06:06:47 -0700
To:	bug-Time-HiRes [...] rt.cpan.org
From:	Jarkko Hietaniemi <jhi [...] iki.fi>

Show quoted text

>> I am just wondering how my 5.26.1 testing did not spew these warnings.

> > On my smokers the warnings happened only with one perl out of 131. strace reveals the difference: > > 9310 08:28:48.390740 execve("/opt/perl-5.26.1/bin/perl", ["/opt/perl-5.26.1/bin/perl", "t/alarm.t"], [/* 70 vars */] <unfinished ...> > > 5431 08:27:23.308484 execve("/opt/perl-5.18.4/bin/perl", ["/opt/perl-5.18.4/bin/perl", "-w", "t/alarm.t"], [/* 69 vars */] <unfinished ...> > > So it seems that there's a something (a particular version of Test::Harness?) which sets -w.

Very curious, thanks. Due to the rarity of this I now undeleted the 1.9755, but since the 1.9756 is out, shouldn't matter much.