Bug #124072 for Nodejs-Util: will not install on MacOS, because of /private/var/folders/...

Sat Jan 13 10:23:57 2018 Th.J.v.Hoesel+CPAN [...] gmail.com - Ticket created

Subject:

will not install on MacOS, because of /private/var/folders/...

it seems to have a problem with the tests t/00-compile, lines 74/78/84/85: got '/private/var/folders/... expected '/var/folders/... hope this will help you fixing this problem ... and thus help install all and everything that depends on it

Sun Jan 14 10:17:45 2018 PERLANCAR [...] cpan.org - Correspondence added

On Sat, 13 Jan 2018 15:23:57 GMT, VANHOESEL wrote: Show quoted text

> it seems to have a problem with the tests > > t/00-compile, lines 74/78/84/85: > > got '/private/var/folders/... > expected '/var/folders/... > > hope this will help you fixing this problem ... and thus help install > all and everything that depends on it

Which version of Nodejs::Util? what is perl version and OS version? I don't have access to a MacOS system, but looking at this: http://www.cpantesters.org/distro/N/Nodejs-Util.html doesn't (yet) show any failures for MacOS.

Sun Jan 14 10:17:45 2018 The RT System itself - Status changed from 'new' to 'open'

Mon Jan 15 03:58:49 2018 th.j.v.hoesel [...] gmail.com - Correspondence added

CC:	Th.J.v.Hoesel+CPAN [...] gmail.com
Subject:	Re: [rt.cpan.org #124072] will not install on MacOS, because of /private/var/folders/...
Date:	Mon, 15 Jan 2018 09:58:28 +0100
To:	bug-Nodejs-Util [...] rt.cpan.org
From:	Theo van Hoesel <th.j.v.hoesel [...] gmail.com>

it all boils down to the use of $CWD, File::chdir and Cwd related stuff on MacOS, /var (and /tmp /etc alike) are links to directories inside /private (some leftovers from NeXTstep) as you can see: ls -l / lrwxr-xr-x@ 1 root wheel 11 Oct 1 16:19 /var -> private/var unfortunately, I have no correct solution on how to fix this, too much ugly things can happen with path, readapt, resolving paths with symbolic links etc, pwd, cwd, chdir … for what it’s worth: MacOS High Siera ( version 10.13.2 ) Perl (under perlbrew) 5.26.1 Nodejs::Util, from GitHub Release v0.009 Hope this is something you can work with Greetings, Theo Show quoted text

> On 14 Jan 2018, at 16:17, Perl Ancar via RT <bug-Nodejs-Util@rt.cpan.org> wrote: > > <URL: https://rt.cpan.org/Ticket/Display.html?id=124072 > > > On Sat, 13 Jan 2018 15:23:57 GMT, VANHOESEL wrote:

>> it seems to have a problem with the tests >> >> t/00-compile, lines 74/78/84/85: >> >> got '/private/var/folders/... >> expected '/var/folders/... >> >> hope this will help you fixing this problem ... and thus help install >> all and everything that depends on it

> > > Which version of Nodejs::Util? what is perl version and OS version? > > I don't have access to a MacOS system, but looking at this: > > http://www.cpantesters.org/distro/N/Nodejs-Util.html > > doesn't (yet) show any failures for MacOS.

Mon Jan 15 04:01:19 2018 th.j.v.hoesel [...] gmail.com - Correspondence added

Subject:	Re: [rt.cpan.org #124072] AutoReply: will not install on MacOS, because of /private/var/folders/...
Date:	Mon, 15 Jan 2018 10:00:58 +0100
To:	bug-Nodejs-Util [...] rt.cpan.org
From:	Theo van Hoesel <th.j.v.hoesel [...] gmail.com>

dirty patch: @@ -14,6 +14,10 @@ use Nodejs::Util qw( ); my $tempdir = tempdir(CLEANUP => !$ENV{DEBUG}); +{ + local $CWD = $tempdir; + $tempdir = $CWD; +} if ($ENV{DEBUG}) { note "tempdir=$tempdir"; } Show quoted text

> prove -I lib

t/01-basic.t .. ok All tests successful. Files=1, Tests=2, 0 wallclock secs ( 0.02 usr 0.01 sys + 0.07 cusr 0.01 csys = 0.11 CPU) Result: PASS Show quoted text

> On 13 Jan 2018, at 16:23, Bugs in Nodejs-Util via RT <bug-Nodejs-Util@rt.cpan.org> wrote: > > > Greetings, > > This message has been automatically generated in response to the > creation of a trouble ticket regarding: > "will not install on MacOS, because of /private/var/folders/...", > a summary of which appears below. > > There is no need to reply to this message right now. Your ticket has been > assigned an ID of [rt.cpan.org #124072]. Your ticket is accessible > on the web at: > > https://rt.cpan.org/Ticket/Display.html?id=124072 > > Please include the string: > > [rt.cpan.org #124072] > > in the subject line of all future correspondence about this issue. To do so, > you may reply to this message. > > Thank you, > bug-Nodejs-Util@rt.cpan.org > > ------------------------------------------------------------------------- > it seems to have a problem with the tests > > t/00-compile, lines 74/78/84/85: > > got '/private/var/folders/... > expected '/var/folders/... > > hope this will help you fixing this problem ... and thus help install all and everything that depends on it

Tue Feb 06 03:19:18 2018 SREZIC [...] cpan.org - Correspondence added

RT-Send-CC:

th.j.v.hoesel [...] gmail.com

I also see this problem (version 0.009). My solution: set TMPDIR=/private/tmp when installing the module. On 2018-01-15 04:01:19, th.j.v.hoesel@gmail.com wrote: Show quoted text

> dirty patch: > > @@ -14,6 +14,10 @@ use Nodejs::Util qw( > ); > > my $tempdir = tempdir(CLEANUP => !$ENV{DEBUG}); > +{ > + local $CWD = $tempdir; > + $tempdir = $CWD; > +} > if ($ENV{DEBUG}) { > note "tempdir=$tempdir"; > } > >

> > prove -I lib

> t/01-basic.t .. ok > All tests successful. > Files=1, Tests=2, 0 wallclock secs ( 0.02 usr 0.01 sys + 0.07 cusr > 0.01 csys = 0.11 CPU) > Result: PASS > >

> > On 13 Jan 2018, at 16:23, Bugs in Nodejs-Util via RT <bug-Nodejs- > > Util@rt.cpan.org> wrote: > > > > > > Greetings, > > > > This message has been automatically generated in response to the > > creation of a trouble ticket regarding: > > "will not install on MacOS, because of > > /private/var/folders/...", > > a summary of which appears below. > > > > There is no need to reply to this message right now. Your ticket has > > been > > assigned an ID of [rt.cpan.org #124072]. Your ticket is accessible > > on the web at: > > > > https://rt.cpan.org/Ticket/Display.html?id=124072 > > > > Please include the string: > > > > [rt.cpan.org #124072] > > > > in the subject line of all future correspondence about this issue. To > > do so, > > you may reply to this message. > > > > Thank you, > > bug-Nodejs-Util@rt.cpan.org > > > > ------------------------------------------------------------------------- > > it seems to have a problem with the tests > > > > t/00-compile, lines 74/78/84/85: > > > > got '/private/var/folders/... > > expected '/var/folders/... > > > > hope this will help you fixing this problem ... and thus help install > > all and everything that depends on it

Sat Mar 03 05:04:02 2018 SREZIC [...] cpan.org - Correspondence added

RT-Send-CC:

th.j.v.hoesel [...] gmail.com

And I think the "correct" solution would be to use Cwd::realpath() before comparing the paths. realpath() may apply some kind of canonicalization to paths (i.e. resolving symlinks). Regards, Slaven On 2018-02-06 03:19:18, SREZIC wrote: Show quoted text

> I also see this problem (version 0.009). My solution: set > TMPDIR=/private/tmp when installing the module. > > On 2018-01-15 04:01:19, th.j.v.hoesel@gmail.com wrote:

> > dirty patch: > > > > @@ -14,6 +14,10 @@ use Nodejs::Util qw( > > ); > > > > my $tempdir = tempdir(CLEANUP => !$ENV{DEBUG}); > > +{ > > + local $CWD = $tempdir; > > + $tempdir = $CWD; > > +} > > if ($ENV{DEBUG}) { > > note "tempdir=$tempdir"; > > } > > > >

> > > prove -I lib

> > t/01-basic.t .. ok > > All tests successful. > > Files=1, Tests=2, 0 wallclock secs ( 0.02 usr 0.01 sys + 0.07 cusr > > 0.01 csys = 0.11 CPU) > > Result: PASS > > > >

> > > On 13 Jan 2018, at 16:23, Bugs in Nodejs-Util via RT <bug-Nodejs- > > > Util@rt.cpan.org> wrote: > > > > > > > > > Greetings, > > > > > > This message has been automatically generated in response to the > > > creation of a trouble ticket regarding: > > > "will not install on MacOS, because of > > > /private/var/folders/...", > > > a summary of which appears below. > > > > > > There is no need to reply to this message right now. Your ticket > > > has > > > been > > > assigned an ID of [rt.cpan.org #124072]. Your ticket is accessible > > > on the web at: > > > > > > https://rt.cpan.org/Ticket/Display.html?id=124072 > > > > > > Please include the string: > > > > > > [rt.cpan.org #124072] > > > > > > in the subject line of all future correspondence about this issue. > > > To > > > do so, > > > you may reply to this message. > > > > > > Thank you, > > > bug-Nodejs-Util@rt.cpan.org > > > > > > ------------------------------------------------------------------------- > > > it seems to have a problem with the tests > > > > > > t/00-compile, lines 74/78/84/85: > > > > > > got '/private/var/folders/... > > > expected '/var/folders/... > > > > > > hope this will help you fixing this problem ... and thus help > > > install > > > all and everything that depends on it

Sat Mar 03 19:46:18 2018 PERLANCAR [...] cpan.org - Correspondence added

RT-Send-CC:

th.j.v.hoesel [...] gmail.com

Ok guys, I've canonicalized the paths (with realpath a.k.a. abs_path) before comparing. Please the new release. Thanks. On Sat, 3 Mar 2018 10:04:02 GMT, SREZIC wrote: Show quoted text

> And I think the "correct" solution would be to use Cwd::realpath() > before comparing the paths. realpath() may apply some kind of > canonicalization to paths (i.e. resolving symlinks). > > Regards, > Slaven > > On 2018-02-06 03:19:18, SREZIC wrote:

> > I also see this problem (version 0.009). My solution: set > > TMPDIR=/private/tmp when installing the module. > > > > On 2018-01-15 04:01:19, th.j.v.hoesel@gmail.com wrote:

> > > dirty patch: > > > > > > @@ -14,6 +14,10 @@ use Nodejs::Util qw( > > > ); > > > > > > my $tempdir = tempdir(CLEANUP => !$ENV{DEBUG}); > > > +{ > > > + local $CWD = $tempdir; > > > + $tempdir = $CWD; > > > +} > > > if ($ENV{DEBUG}) { > > > note "tempdir=$tempdir"; > > > } > > > > > >

> > > > prove -I lib

> > > t/01-basic.t .. ok > > > All tests successful. > > > Files=1, Tests=2, 0 wallclock secs ( 0.02 usr 0.01 sys + 0.07 > > > cusr > > > 0.01 csys = 0.11 CPU) > > > Result: PASS > > > > > >

> > > > On 13 Jan 2018, at 16:23, Bugs in Nodejs-Util via RT <bug-Nodejs- > > > > Util@rt.cpan.org> wrote: > > > > > > > > > > > > Greetings, > > > > > > > > This message has been automatically generated in response to the > > > > creation of a trouble ticket regarding: > > > > "will not install on MacOS, because of > > > > /private/var/folders/...", > > > > a summary of which appears below. > > > > > > > > There is no need to reply to this message right now. Your ticket > > > > has > > > > been > > > > assigned an ID of [rt.cpan.org #124072]. Your ticket is > > > > accessible > > > > on the web at: > > > > > > > > https://rt.cpan.org/Ticket/Display.html?id=124072 > > > > > > > > Please include the string: > > > > > > > > [rt.cpan.org #124072] > > > > > > > > in the subject line of all future correspondence about this > > > > issue. > > > > To > > > > do so, > > > > you may reply to this message. > > > > > > > > Thank you, > > > > bug-Nodejs-Util@rt.cpan.org > > > > > > > > ------------------------------------------------------------------------- > > > > it seems to have a problem with the tests > > > > > > > > t/00-compile, lines 74/78/84/85: > > > > > > > > got '/private/var/folders/... > > > > expected '/var/folders/... > > > > > > > > hope this will help you fixing this problem ... and thus help > > > > install > > > > all and everything that depends on it

Fri Mar 09 00:54:06 2018 PERLANCAR [...] cpan.org - Correspondence added

CPAN Testers seems to have given the green lights. Closing this ticket. On Sun, 4 Mar 2018 00:46:18 GMT, PERLANCAR wrote: Show quoted text

> Ok guys, I've canonicalized the paths (with realpath a.k.a. abs_path) > before comparing. Please the new release. Thanks. > > On Sat, 3 Mar 2018 10:04:02 GMT, SREZIC wrote:

> > And I think the "correct" solution would be to use Cwd::realpath() > > before comparing the paths. realpath() may apply some kind of > > canonicalization to paths (i.e. resolving symlinks). > > > > Regards, > > Slaven > > > > On 2018-02-06 03:19:18, SREZIC wrote:

> > > I also see this problem (version 0.009). My solution: set > > > TMPDIR=/private/tmp when installing the module. > > > > > > On 2018-01-15 04:01:19, th.j.v.hoesel@gmail.com wrote:

> > > > dirty patch: > > > > > > > > @@ -14,6 +14,10 @@ use Nodejs::Util qw( > > > > ); > > > > > > > > my $tempdir = tempdir(CLEANUP => !$ENV{DEBUG}); > > > > +{ > > > > + local $CWD = $tempdir; > > > > + $tempdir = $CWD; > > > > +} > > > > if ($ENV{DEBUG}) { > > > > note "tempdir=$tempdir"; > > > > } > > > > > > > >

> > > > > prove -I lib

> > > > t/01-basic.t .. ok > > > > All tests successful. > > > > Files=1, Tests=2, 0 wallclock secs ( 0.02 usr 0.01 sys + 0.07 > > > > cusr > > > > 0.01 csys = 0.11 CPU) > > > > Result: PASS > > > > > > > >

> > > > > On 13 Jan 2018, at 16:23, Bugs in Nodejs-Util via RT <bug- > > > > > Nodejs- > > > > > Util@rt.cpan.org> wrote: > > > > > > > > > > > > > > > Greetings, > > > > > > > > > > This message has been automatically generated in response to > > > > > the > > > > > creation of a trouble ticket regarding: > > > > > "will not install on MacOS, because of > > > > > /private/var/folders/...", > > > > > a summary of which appears below. > > > > > > > > > > There is no need to reply to this message right now. Your > > > > > ticket > > > > > has > > > > > been > > > > > assigned an ID of [rt.cpan.org #124072]. Your ticket is > > > > > accessible > > > > > on the web at: > > > > > > > > > > https://rt.cpan.org/Ticket/Display.html?id=124072 > > > > > > > > > > Please include the string: > > > > > > > > > > [rt.cpan.org #124072] > > > > > > > > > > in the subject line of all future correspondence about this > > > > > issue. > > > > > To > > > > > do so, > > > > > you may reply to this message. > > > > > > > > > > Thank you, > > > > > bug-Nodejs-Util@rt.cpan.org > > > > > > > > > > ------------------------------------------------------------------------- > > > > > it seems to have a problem with the tests > > > > > > > > > > t/00-compile, lines 74/78/84/85: > > > > > > > > > > got '/private/var/folders/... > > > > > expected '/var/folders/... > > > > > > > > > > hope this will help you fixing this problem ... and thus help > > > > > install > > > > > all and everything that depends on it

Fri Mar 09 00:54:07 2018 PERLANCAR [...] cpan.org - Status changed from 'open' to 'resolved'