Bug #49388 for Perl-Dist-Strawberry: Strawberry in directory with spaces in its name

Tue Sep 01 05:11:19 2009 cspatkos [...] gmanalytics.com - Ticket #49324: Ticket created

Subject:	Bug Report - CPAN/dmake error if path contains spaces
Date:	Tue, 01 Sep 2009 12:10:35 +0300
To:	bug-Perl-Dist-Strawberry [...] rt.cpan.org
From:	"Patkos Csaba" <cspatkos [...] gmanalytics.com>

Hi there, I found the following bug: - installing a package is impossible with CPAN on strawberry perl 5.10.x (latest release) Portable version, if the path in which the Strawberry Perl is placed contains spaces. For example I have it on my desktop, on a Windows XP (also tried on Win 2008 server) I can start cpan but when I try to install a package, dmake fails when looking for startup.mk file. The error originates in the wrong interpretation of the path. - what exactly happens is that a path like "C:\Documents and Settings\Administrator\Desktop\Perl Portable\c\bin" transforms in "C:\ Settings\Administrator\Desktop\ Portable\c\bin" ... and obviosly it doesn't work. (note the [space] character before "Settings" and "Portable") Best regards, -- ing. Patkós Csaba SysAdmin @ GM Analytic Software Tel: +40 - 745056549 E-Mail: cspatkos@gmanalytics.com --------------------------------------- "You need chaos in your soul to give birth to a dancing star" -- Friedrich Nietzsche

Wed Sep 02 23:42:55 2009 kmx [...] cpan.org - Ticket created

Subject:

Strawberry in directory with spaces in its name

moved discussion here from: http://rt.cpan.org/Ticket/Display.html?id=49010#txn-658160

Thu Sep 03 00:12:26 2009 kmx [...] cpan.org - Correspondence added

According to my knowledge to have strawberry in a directory with name containing spaces we have to: 1) fix dmake http://svn.ali.as/cpan/users/kmx/dmake_patch/dmake_svn-rev275670_patch.diff http://svn.ali.as/cpan/users/kmx/dmake_patch/dmake_svn-rev275670+kmx-patch.zip (=patched compiled version) 2) fix ExtUtils-MakeMaker http://svn.ali.as/cpan/users/kmx/perl-modules-patched/ExtUtils-MakeMaker-6.55_02.diff http://svn.ali.as/cpan/users/kmx/perl-modules-patched/ExtUtils-MakeMaker-6.55_02.tar.gz (=patched version) 3) fix Test-Harness http://svn.ali.as/cpan/users/kmx/perl-modules-patched/Test-Harness-3.17.diff http://svn.ali.as/cpan/users/kmx/perl-modules-patched/Test-Harness-3.17.tar.gz (=patched version) With fixes 1) + 2) + 3) I was able to do: - perl Makefile.pl - dmake - dmake test - dmake install 4) next we need to fix CPAN module It is able to install an module but it is not able to properly check what version (if any) was already installed. I have no fix for that. -- kmx

Thu Sep 03 00:12:27 2009 kmx [...] cpan.org - Status changed from 'new' to 'open'

Fri Sep 04 00:21:52 2009 kmx [...] cpan.org - Correspondence added

1) fix dmake reported here: http://tools.openoffice.org/issues/show_bug.cgi?id=104727 2) fix ExtUtils-MakeMaker RT for this: http://rt.cpan.org/Ticket/Display.html?id=49418 3) fix Test-Harness RT for this: http://rt.cpan.org/Ticket/Display.html?id=49417

Fri Sep 04 16:17:31 2009 kmx [...] cpan.org - Correspondence added

I was wrong about 4) - CPAN is OK with "spaces". This 3 patches seems to be enough: 1) fix dmake http://svn.ali.as/cpan/users/kmx/dmake_patch/dmake_svn-rev275670_patch.diff http://svn.ali.as/cpan/users/kmx/dmake_patch/dmake_svn-rev275670+kmx-patch.zip (=patched compiled version) 2) fix ExtUtils-MakeMaker (UPDATED!!!) http://svn.ali.as/cpan/users/kmx/perl-modules-patched/ExtUtils-MakeMaker-6.55_02_v2.diff http://svn.ali.as/cpan/users/kmx/perl-modules-patched/ExtUtils-MakeMaker-6.55_02.tar.gz (=patched version) 3) fix Test-Harness http://svn.ali.as/cpan/users/kmx/perl-modules-patched/Test-Harness-3.17.diff http://svn.ali.as/cpan/users/kmx/perl-modules-patched/Test-Harness-3.17.tar.gz (=patched version) I have tested with portable strawberry perl unziped in a dir with spaces and it seems to work. IMPORTANT NOTICE: unfortunately many other modules are not ready for spaces in strings like $^X - so making strawberry "space-in-dirname-aware" is just half of the story (and the second half in not under our control). To sum up: - my proof-of-concept revealed that installdir with spaces is possible - there is still open issue with non-us-ascii chars in dirname - I am leaving this RT till somebody decide that we really want to invest more energy into it. -- kmx

Fri Sep 04 22:32:20 2009 csjewell [...] cpan.org - Correspondence added

Show quoted text

> - my proof-of-concept revealed that installdir with spaces is possible > - there is still open issue with non-us-ascii chars in dirname > - I am leaving this RT till somebody decide that we really want to > invest more energy into it.

Right now, I've got a Perl Foundation grant coming my way, and part of it includes installer relocatability, as far as is possible. I didn't commit to making it relocatable into a directory with spaces, but once the October version is released, I want to (and I should be able to) help you more with this, because I'd like to have it space-relocatable if at all possible. --Curtis

Fri Sep 04 22:32:22 2009 csjewell [...] cpan.org - Status changed from 'open' to 'stalled'

Wed Sep 09 02:30:59 2009 kmx [...] cpan.org - Ticket #49324: Correspondence added

You are right, there are some problems with dmake but also with other modules. I have done some research - see more details here http://rt.cpan.org/Ticket/Display.html?id=49388 I guess that this RT should be merged with RT#49388 -- kmx

Wed Sep 09 02:31:00 2009 The RT System itself - Ticket #49324: Status changed from 'new' to 'open'

Wed Sep 09 13:48:12 2009 csjewell [...] cpan.org - Ticket #49324: Merged into ticket #49388

Wed Sep 09 13:48:12 2009 csjewell [...] cpan.org - Merged into ticket #49388

Mon Oct 12 03:24:51 2009 csjewell [...] cpan.org - Dependency by ticket #50409 added

Wed Feb 17 16:32:02 2010 csjewell [...] cpan.org - Dependency by ticket #50409 deleted

Wed Feb 17 16:32:02 2010 csjewell [...] cpan.org - Dependency by ticket #54728 added

Thu Apr 22 09:34:22 2010 csjewell [...] cpan.org - Dependency by ticket #54728 deleted

Tue May 04 04:21:52 2010 kmx [...] cpan.org - Correspondence added

I have revealed another space-in-dirname related issue - it regards unfortunately perl core.

The problem is with $Config{libpth} variable which uses space as item separator.

See: output of perl -MConfig -e "print $Config{libpth}"
D:\strawberry32\c\lib D:\strawberry32\c\i686-w64-mingw32\lib

Obviously when the strawberry perl will be installed in directory with spaces we are gonna have a big trouble.

(P.S. I have not reported this to per5 bugrtacker yet)

--
kmx

Tue May 04 04:21:53 2010 The RT System itself - Status changed from 'stalled' to 'open'

Wed May 19 19:33:45 2010 csjewell [...] cpan.org - Status changed from 'open' to 'stalled'

Wed Jan 18 06:51:56 2012 p.h.vd.muijzenberg [...] rsg-sneek.nl - Ticket #74119: Ticket created

Subject:	Still can't install in C:/Program Files
Date:	Wed, 18 Jan 2012 12:51:41 +0100
To:	<bug-Perl-Dist-Strawberry [...] rt.cpan.org>
From:	"Peter Hans Muijzenberg, van den" <p.h.vd.muijzenberg [...] rsg-sneek.nl>

Hi, Regardless of all the resolved bugs about relocatable installs, the version available at <http://strawberryperl.com/> as "Download Strawberry Perl 5.12.3.0 XP/2003/Vista/2008/Win7, English, 40MB, May 2011" still won't install to C:/Program Files/Strawberry, as Microsoft specifies it should, as the installer won't accept that path since "Strawberry Perl cannot be installed in a directory with spaces or non-ASCII characters". (It probably wouldn't hurt to make the error messages distinguish between the two, BTW.) -- BFN, Peter Hans van den Muijzenberg

Thu Jan 19 08:12:26 2012 kmx [...] cpan.org - Ticket #74119: Correspondence added

Show quoted text

> Regardless of all the resolved bugs about relocatable installs, the
> version available at <http://strawberryperl.com/> as "Download
> Strawberry Perl 5.12.3.0 XP/2003/Vista/2008/Win7, English, 40MB,
> May 2011" still won't install to C:/Program Files/Strawberry, as
> Microsoft specifies it should, as the installer won't accept that
> path since "Strawberry Perl cannot be installed in a directory
> with spaces or non-ASCII characters".
> (It probably wouldn't hurt to make the error messages distinguish
> between the two, BTW.)

The message "Strawberry Perl cannot be installed in a directory with spaces or non-ASCII characters" was put there intentionally as we simply block an attempt to install strawberry perl into a dir with spaces.

Believe me it really does not work correctly.

--
kmx

Thu Jan 19 08:12:26 2012 The RT System itself - Ticket #74119: Status changed from 'new' to 'open'

Thu Jan 19 09:23:09 2012 p.h.vd.muijzenberg [...] rsg-sneek.nl - Ticket #74119: Correspondence added

Subject:	Betr.: [rt.cpan.org #74119] Still can't install in C:/Program Files
Date:	Thu, 19 Jan 2012 15:22:55 +0100
To:	<bug-Perl-Dist-Strawberry [...] rt.cpan.org>
From:	"Peter Hans Muijzenberg, van den" <p.h.vd.muijzenberg [...] rsg-sneek.nl>

Hi, Op Donderdag 19 Januari 2012 om 14:12 is door "kmx via RT" <bug-Perl-Dist-Strawberry@rt.cpan.org> geschreven: <> URL: https://rt.cpan.org/Ticket/Display.html?id=74119 > Show quoted text

>> Regardless of all the resolved bugs about relocatable installs, the >> version available at <http://strawberryperl.com/> as "Download >> Strawberry Perl 5.12.3.0 XP/2003/Vista/2008/Win7, English, 40MB, >> May 2011" still won't install to C:/Program Files/Strawberry, as >> Microsoft specifies it should ...

> The message "Strawberry Perl cannot be installed in a directory with spaces > or non-ASCII characters" was put there intentionally ...

I have no doubt it was. But that's preventing the bug's occurrence, rather than solving the bug. Strawberry Perl will first be completely relocatable when it can be located on any location in the filesystem. And, barring changes in Microsoft Policies that I may have missed, it will be Windows compliant only when it's possible to install it in "C:\Program Files\Strawberry" (regardless of whether it accesses that directory as "C:\PROGRA~1\STRAWB~1", or whatever the installer got from dir /x). Hence, my message about this not yet being possible. -- BFN, Peter Hans van den Muijzenberg

Fri May 25 08:41:17 2012 kmx [...] cpan.org - Ticket #74119: Merged into ticket #49388

Fri May 25 08:41:17 2012 kmx [...] cpan.org - Merged into ticket #49388