Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Git-Wrapper CPAN distribution.

Report information
The Basics
Id: 70949
Status: resolved
Priority: 0/
Queue: Git-Wrapper
People
Owner: Nobody in particular
Requestors: AYOUNG [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Critical
Broken in: 0.016
Fixed in: (no value)

History Show all quoted text
  Tue Sep 13 20:58:19 2011 AYOUNG [...] cpan.org - Ticket created
Subject: failure to pass test in 5.8.8
lib/Git/Wrapper.pm has 'use 5.006' but line 20 has my $GIT = $ENV{GIT_WRAPPER_GIT} // 'git'; '//' was introduced in 5.12. Either 'use 5.012' should be used or '||' on line 20. I would prefer '||' -- when I changed line 20 to use '||' and ran 'make test'. all tests passed.
  Sun Oct 09 14:10:33 2011 GENEHACK [...] cpan.org - Correspondence added
On Tue Sep 13 20:58:19 2011, AYOUNG wrote: Show quoted text
> lib/Git/Wrapper.pm has 'use 5.006' but line 20 has > > my $GIT = $ENV{GIT_WRAPPER_GIT} // 'git'; > > '//' was introduced in 5.12. Either 'use 5.012' should be used or '||' > on line 20. > > I would prefer '||' -- when I changed line 20 to use '||' and ran 'make > test'. all tests passed.
Thanks for the report -- this was fixed in 0.017.
  Sun Oct 09 14:10:34 2011 The RT System itself - Status changed from 'new' to 'open'
  Sun Oct 09 14:10:34 2011 GENEHACK [...] cpan.org - Status changed from 'open' to 'resolved'