search

arvindm95 / unladen-swallow

Automatically exported from code.google.com/p/unladen-swallow
Other
0 stars 0 forks source link

make DESTDIR="dirToInstall" install does not work #122

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
Hi,

I'm trying to make a pkg for my distro (ArchLinux) but to do this I have to 
specify a folder to install unladen instead of installing it on my root 
file sytem. The install process works fine until the "# Install the include 
files" sections of the Makefile is reach (starting at line 1057).
At this point the install process try to install files on my root file 
system instead of trying to do it on the specified folder :

Creating directory /usr/lib/python2.6/lib-dynload
/bin/install: cannot create regular file `/usr/lib/python2.6/lib-
dynload/libpython2.6.bc': Permission denied
make: *** [libainstall] Error 1

I didn't find why this is happening in the Makefile.

Original issue reported on code.google.com by dory.thi...@gmail.com on 16 Jan 2010 at 3:52

GoogleCodeExporter commented 8 years ago 
My make-fu is weak.  However, I thought the right way to do this was to say
./configure --prefix=/my/build/root/.  Does that work?  I don't think I've ever 
used
make install for this project.  :)

Original comment by reid.kle...@gmail.com on 27 Jan 2010 at 1:31

GoogleCodeExporter commented 8 years ago 
DESTDIR is commonly used by packagers, for example, to allow installation 
without 
superuser privileges. The value of DESTDIR should be prepended to all installed 
paths, 
but the binaries should not contain any references to it.

Original comment by danchr on 27 Jan 2010 at 9:20

GoogleCodeExporter commented 8 years ago 
At least for quick testing, this patch solves the main issues I hit.

That said I've not yet ran it against gentoo's sandbox (which is a fair bit 
better at 
snagging accesses) so there may be a few more lurking.

At the very least, this is enough to get installs working for me w/ DESTDIR 
behaving 
properly.

Original comment by ferringb on 30 Jan 2010 at 8:56

Attachments:

GoogleCodeExporter commented 8 years ago 
Thx ferringb, your patch is working great for me!

Original comment by dory.thi...@gmail.com on 30 Jan 2010 at 10:07

GoogleCodeExporter commented 8 years ago 
Patch applied as r1132. Thanks for the patch!

Original comment by collinw on 11 Mar 2010 at 10:26