Skip Menu |

This queue is for tickets about the Tie-File CPAN distribution.

Report information
The Basics
Id: 15595
Status: resolved
Priority: 0/
Queue: Tie-File

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

Bug Information
Severity: Important
Broken in: 0.96
Fixed in: (no value)



Subject: INSTALLDIRS not set in Makefile.PL
As Tie::File is a perl core module (according to Module::CoreList since perl version 5.007003), the Makefile.PL should have a line INSTALLDIRS => $] >= 5.007003 ? 'perl' : 'site', so Tie::File gets installed in the right place. For a reference, see the Makefile.PL of Storable. And while writing this bugreport, just another very minor nit: in line 169 there's some dead code my $fh = $self->{FH}; $fh is never used, and FH (upper case) is wrong anyway. Regards, Slaven
This has been resolved for a while best I can tell
On 2013-03-28 01:48:00, TODDR wrote: Show quoted text
> This has been resolved for a while best I can tell
Well, this line looks dubious: https://github.com/toddr/Tie-File/blob/master/Makefile.PL#L14 The condition can never be true. Regards, Slaven
On Tue Apr 02 16:35:01 2013, SREZIC wrote: Show quoted text
> On 2013-03-28 01:48:00, TODDR wrote:
> > This has been resolved for a while best I can tell
> > Well, this line looks dubious: > https://github.com/toddr/Tie-File/blob/master/Makefile.PL#L14 > The condition can never be true.
We just rediscovered this issue on freenode/#perl. It's still broken.
Ticket migrated to github as https://github.com/toddr/Tie-File/issues/10