Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

space in prefix path #2002

Closed
p6rt opened this issue Aug 2, 2010 · 13 comments
Closed

space in prefix path #2002

p6rt opened this issue Aug 2, 2010 · 13 comments

Comments

@p6rt
Copy link

p6rt commented Aug 2, 2010

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

Searchable as RT76930$

@p6rt
Copy link
Author

p6rt commented Aug 2, 2010

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
Copy link
Author

p6rt commented Aug 2, 2010

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
Copy link
Author

p6rt commented Aug 2, 2010

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

@p6rt
Copy link
Author

p6rt commented Feb 18, 2012

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
Copy link
Author

p6rt commented Nov 16, 2012

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
Copy link
Author

p6rt commented Jun 9, 2013

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
Copy link
Author

p6rt commented Oct 4, 2014

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
Copy link
Author

p6rt commented Oct 4, 2014

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

@p6rt
Copy link
Author

p6rt commented Aug 26, 2015

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
Copy link
Author

p6rt commented Oct 19, 2016

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@​gmail.com>
http://www.pair.com/~comdog/

@p6rt
Copy link
Author

p6rt commented Aug 29, 2019

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​:
* rakudo/rakudo#2693
* rakudo/rakudo#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'.

1 similar comment
@p6rt
Copy link
Author

p6rt commented Aug 29, 2019

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​:
* rakudo/rakudo#2693
* rakudo/rakudo#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
Copy link
Author

p6rt commented Aug 29, 2019

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant