search

Raku / old-issue-tracker

Tickets from RT
https://github.com/Raku/old-issue-tracker/issues
2 stars 1 forks source link

space in prefix path #2002

Closed p6rt closed 5 years ago

p6rt commented 14 years ago

Migrated from rt.perl.org#76930 (status was 'resolved')

Searchable as RT76930$

p6rt commented 14 years ago

From sven@genomics.princeton.edu

  I tried to configure rakudo-star-2010.07 in a directory with a space in it (@​#$% Ubuntu One) and it didn't work. It builds fine when I don't include the space.

$ perl Configure.pl --gen-parrot --prefix=/home/sven/Ubuntu\ One/opt/perl6 . . . cc -o installable_parrot \   src/main.o src/install_config.o \   -Wl,-rpath=/home/sven/Ubuntu One/opt/perl6/lib -L/usr/local/src/rakudo-star-2010.07/parrot-2.6.0/blib/lib -lparrot -lm -L/usr/lib -licuuc -licudata -lm -ldl -lm -lpthread -lcrypt -lrt -lreadline -fstack-protector -L/usr/local/lib -Wl,-E cc​: One/opt/perl6/lib​: No such file or directory make​: *** [installable_parrot] Error 1 Command failed (status 512)​: make install-dev ===SORRY!=== Unable to locate parrot_config

To automatically build the version of Parrot that came with this distribution (2.6.0), try re-running Configure.pl with the '--gen-parrot' option. Or, use the '--parrot-config' option to explicitly specify the location of parrot_config to be used to build Rakudo Star.

p6rt commented 14 years ago

From @moritz

FWIW, on my machine parrot doesn't even install with a space in the prefix path​:

http://trac.parrot.org/parrot/ticket/1716

I guess there's not much point in trying to tackle this issue until parrot gets some sanity.

p6rt commented 14 years ago

The RT System itself - Status changed from 'new' to 'open'

p6rt commented 12 years ago

From pete@clueball.com

12​:23 \< peteretep> The Configure.pl buildstep for rakudo-star-2012.01 appears to   dislike my Cwd having a space in it 12​:23 \< peteretep> perl Configure.pl --gen-parrot 12​:24 \< peteretep> Tries to run​: 12​:24 \< peteretep> /usr/bin/perl Configure.pl --with-parrot=/mnt/hgfs/Code   Folder/rakudo-star-2012.01/install/bin/parrot --make-install

Eventually I got it working by installing it in /tmp/

p6rt commented 12 years ago

From @coke

On Mon Aug 02 12​:55​:42 2010, moritz wrote​:

FWIW, on my machine parrot doesn't even install with a space in the prefix path​:

http://trac.parrot.org/parrot/ticket/1716

I guess there's not much point in trying to tackle this issue until parrot gets some sanity.

This seems to now fail when running nqp​:

Command failed (status 512)​: /Users/coke/perl5/perlbrew/perls/perl-5.16.0_THREAD/bin/perl Configure.pl --with-parrot=/tmp/a space/bin/parrot --make-install

The argument to --with-parrot needs to be quoted.

-- Will "Coke" Coleda

p6rt commented 11 years ago

From @coke

On Thu Nov 15 19​:25​:59 2012, coke wrote​:

On Mon Aug 02 12​:55​:42 2010, moritz wrote​:

FWIW, on my machine parrot doesn't even install with a space in the prefix path​:

http://trac.parrot.org/parrot/ticket/1716

I guess there's not much point in trying to tackle this issue until parrot gets some sanity.

This seems to now fail when running nqp​:

Command failed (status 512)​: /Users/coke/perl5/perlbrew/perls/perl- 5.16.0_THREAD/bin/perl Configure.pl --with-parrot=/tmp/a space/bin/parrot --make-install

The argument to --with-parrot needs to be quoted.

I've updated parrot and nqp to better handle processing and passing quoting path arguments.

An update to rakudo-star that pulls the latest nqp & parrot's should handle this now.

Leaving open until R* is updated.

-- Will "Coke" Coleda

p6rt commented 10 years ago

From @usev6

This is still an issue. Fixing this will require a deep review of the build system (cmp. the following short discussion on #perl​: http://irclog.perlgeek.de/perl6/2014-10-02#i_9447222).

p6rt commented 10 years ago

@usev6 - Status changed from 'new' to 'open'

p6rt commented 9 years ago

From @coke

On Sat Oct 04 09​:03​:20 2014, bartolin@​gmx.de wrote​:

This is still an issue. Fixing this will require a deep review of the build system (cmp. the following short discussion on #perl​: http://irclog.perlgeek.de/perl6/2014-10-02#i_9447222).

Current status - This is busted in a MoarVM build, let alone star.

-- Will "Coke" Coleda

p6rt commented 8 years ago

From @briandfoy

In the rakudo-star-2016.07 release, the value given to Configure's --prefix might end up with whitespace (even if the initial value does not have it). This path is not quoted in nqp/Makefile, so that fails and the build cannot continue.

I ran into this because I used a path that was a symlink​:

% perl Configure.pl --gen-moar --prefix /Users/brian/Dropbox/perl6s/rakudo-star-2016.07

Various things had the same problem​:

% perl Configure.PL --gen-moar --prefix "/Volumes/Big Scratch/Dropbox/perl6s/rakudo-star-2016.07" % perl Configure.PL --gen-moar --prefix /Volumes/"Big Scratch"/Dropbox/perl6s/rakudo-star-2016.07 % perl Configure.PL --gen-moar --prefix /Volumes/Big\\ Scratch/Dropbox/perl6s/rakudo-star-2016.07

Paths interpolated into shell programs or makefiles should be protected in some fashion.

-- brian d foy \brian\.d\.foy@&#8203;gmail\.com http://www.pair.com/~comdog/

p6rt commented 5 years ago

From @usev6

Using spaces in prefix path should work now. There was some serious revamp of the build system during the last couple of months. I didn't have all the details, but some relevant pull requests for Rakudo have probably been​: * https://github.com/rakudo/rakudo/pull/2693 * https://github.com/rakudo/rakudo/pull/3031

Also​:

  $ cd ~/perl6/tmp/dir\ with\ space/   $ mkdir perl6   $ git clone https://github.com/rakudo/rakudo.git   $ cd rakudo   $ perl Configure.pl --backends=moar --gen-nqp --gen-moar --prefix ~/perl6/tmp/dir\ with\ space/perl6 && make install   $ ~/perl6/tmp/dir\ with\ space/perl6/bin/perl6 -e 'say 42'   42

I'm closing this ticket as 'resolved'.

p6rt commented 5 years ago

From @usev6

Using spaces in prefix path should work now. There was some serious revamp of the build system during the last couple of months. I didn't have all the details, but some relevant pull requests for Rakudo have probably been​: * https://github.com/rakudo/rakudo/pull/2693 * https://github.com/rakudo/rakudo/pull/3031

Also​:

  $ cd ~/perl6/tmp/dir\ with\ space/   $ mkdir perl6   $ git clone https://github.com/rakudo/rakudo.git   $ cd rakudo   $ perl Configure.pl --backends=moar --gen-nqp --gen-moar --prefix ~/perl6/tmp/dir\ with\ space/perl6 && make install   $ ~/perl6/tmp/dir\ with\ space/perl6/bin/perl6 -e 'say 42'   42

I'm closing this ticket as 'resolved'.

p6rt commented 5 years ago

@usev6 - Status changed from 'open' to 'resolved'