Skip Menu |

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

Report information
The Basics
Id: 124962
Status: open
Priority: 0/
Queue: App-csv2sqlite
People
Owner: Nobody in particular
Requestors: NHORNE [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in: 0.004
Fixed in: (no value)

History Show all quoted text
  Wed Mar 28 09:19:55 2018 NHORNE [...] cpan.org - Ticket created
Subject: Hard coded path
Running csv2sqlite on Appveyor fails because of this hard-coded path in bin/csv2sqlite: #!/home/rando/perl5/perlbrew/perls/local/bin/perl That path doesn't exist. The line should read #!/usr/bin/env perl
  Thu Mar 29 19:42:27 2018 RWSTAUNER [...] cpan.org - Correspondence added
On Wed Mar 28 06:19:55 2018, NHORNE wrote: Show quoted text
> Running csv2sqlite on Appveyor fails because of this hard-coded path > in bin/csv2sqlite: > > #!/home/rando/perl5/perlbrew/perls/local/bin/perl > > That path doesn't exist. > > The line should read > > #!/usr/bin/env perl
The installer is supposed to update the path. For example, my currently installed version of this shows: #!/Users/rstauner/perl5/plenv/versions/5.24.0/bin/perl5.24.0 I believe if the shebang reads "#!/usr/bin/env perl" the installer will *not* change it, which can actually make it break because the dependencies are separated from the runtime.
  Thu Mar 29 19:42:28 2018 The RT System itself - Status changed from 'new' to 'open'
  Thu Mar 29 19:43:58 2018 RWSTAUNER [...] cpan.org - Correspondence added
Admittedly, the path that exists is a little silly, so I'd be happy to change it to... /usr/local/bin/perl ... if that would help.
  Fri Mar 30 08:27:11 2018 njh [...] bandsman.co.uk - Correspondence added
Subject: Re: [rt.cpan.org #124962] Hard coded path
Date: Fri, 30 Mar 2018 08:25:57 -0400
To: bug-App-csv2sqlite [...] rt.cpan.org
From: Nigel Horne <njh [...] bandsman.co.uk>
"The installer is supposed to update the path.". I'll give it a go on my Windows machine the next time I have the chance and let you know what I see.  I am wondering if there's a Windows based issue.
  Sun Apr 01 16:24:44 2018 RWSTAUNER [...] cpan.org - Correspondence added
On Fri Mar 30 05:27:11 2018, njh@bandsman.co.uk wrote: Show quoted text
> "The installer is supposed to update the path.". > > I'll give it a go on my Windows machine the next time I have the chance > and let you know what I see.  I am wondering if there's a Windows based > issue.
https://metacpan.org/pod/ExtUtils::MakeMaker#EXE_FILES Specifically the code in ExtUtils::MM_Unix, which ExtUtils::MM_Win32 inherits from: https://metacpan.org/source/BINGOS/ExtUtils-MakeMaker-7.34/lib/ExtUtils/MM_Unix.pm#L1242 I _could_ release an update with a simpler shebang, but the effect would be the same.
  Sun Apr 01 18:48:11 2018 NHORNE [...] cpan.org - Correspondence added
I tried to reproduce the problem with my Windows machine, but failed. Something is different with the Appveyor environment. Have you ever used it?
  Mon Apr 02 13:07:37 2018 RWSTAUNER [...] cpan.org - Correspondence added
On Sun Apr 01 15:48:11 2018, NHORNE wrote: Show quoted text
> I tried to reproduce the problem with my Windows machine, but failed. > Something is different with the Appveyor environment. Have you ever > used it?
Sorry, no, I have no idea what that is.