Package could not be built if parent directory of package contains white space #29

Closed
shyuan opened this Issue Aug 9, 2012 · 1 comment

Projects

None yet

2 participants

@shyuan
shyuan commented Aug 9, 2012

It seems that use PWD in Makefile is not a good idea.

I can reproduce this problem as follow:

$ cd /tmp
$ mkdir a\ b
$ git clone https://github.com/pgriess/node-msgpack.git
Cloning into 'node-msgpack'...
remote: Counting objects: 319, done.
remote: Compressing objects: 100% (179/179), done.
remote: Total 319 (delta 136), reused 272 (delta 128)
Receiving objects: 100% (319/319), 462.26 KiB | 127 KiB/s, done.
Resolving deltas: 100% (136/136), done.
$ cd node-msgpack 
$ make
cd deps/msgpack && \
        mkdir -p dist && \
        ./configure --enable-static --disable-shared \
            --prefix=/tmp/a b/node-msgpack/deps/msgpack/dist && \
        make && \
        make install
configure: WARNING: you should use --build, --host, --target
configure: WARNING: invalid host type: b/node-msgpack/deps/msgpack/dist
checking for a BSD-compatible install... /usr/bin/install -c
checking whether build environment is sane... yes
checking for a thread-safe mkdir -p... /bin/mkdir -p
checking for gawk... gawk
checking whether make sets $(MAKE)... yes
checking for b/node-msgpack/deps/msgpack/dist-gcc... no
checking for gcc... gcc
checking whether the C compiler works... yes
checking for C compiler default output file name... a.out
checking for suffix of executables... 
checking whether we are cross compiling... no
checking for suffix of object files... o
checking whether we are using the GNU C compiler... yes
checking whether gcc accepts -g... yes
checking for gcc option to accept ISO C89... none needed
checking for style of include used by make... GNU
checking dependency style of gcc... gcc3
checking for b/node-msgpack/deps/msgpack/dist-g++... no
checking for b/node-msgpack/deps/msgpack/dist-c++... no
checking for b/node-msgpack/deps/msgpack/dist-gpp... no
checking for b/node-msgpack/deps/msgpack/dist-aCC... no
checking for b/node-msgpack/deps/msgpack/dist-CC... no
checking for b/node-msgpack/deps/msgpack/dist-cxx... no
checking for b/node-msgpack/deps/msgpack/dist-cc++... no
checking for b/node-msgpack/deps/msgpack/dist-cl.exe... no
checking for b/node-msgpack/deps/msgpack/dist-FCC... no
checking for b/node-msgpack/deps/msgpack/dist-KCC... no
checking for b/node-msgpack/deps/msgpack/dist-RCC... no
checking for b/node-msgpack/deps/msgpack/dist-xlC_r... no
checking for b/node-msgpack/deps/msgpack/dist-xlC... no
checking for g++... g++
checking whether we are using the GNU C++ compiler... yes
checking whether g++ accepts -g... yes
checking dependency style of g++... gcc3
configure: WARNING: Libtool does not cope well with whitespace in `pwd`
checking build system type... Invalid configuration `b/node-msgpack/deps/msgpack/dist': machine `b/node' not recognized
configure: error: /bin/bash ac/config.sub b/node-msgpack/deps/msgpack/dist failed
make: *** [deps/msgpack/dist/lib/libmsgpack.a] Error 1
@godsflaw
Collaborator

The build system is now node-gyp, I'm going to close this for now.

@godsflaw godsflaw closed this Jun 14, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment