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

Cliquer install fails to create intermediate directories #18808

Closed
vbraun opened this issue Jun 29, 2015 · 8 comments
Closed

Cliquer install fails to create intermediate directories #18808

vbraun opened this issue Jun 29, 2015 · 8 comments

Comments

@vbraun
Copy link
Member

vbraun commented Jun 29, 2015

Probably triggered by #18710, but really this was a landmine waiting to go off:

  • No error handling, failure to install headers is ignored
  • Once again we see that cp is not an appropriate way to install stuff, this is why everybody else uses a special install script/program to handle permissions and intermediate directories.
/mnt/disk/home/buildslave-sage/slave/sage_git/build/src/bin/sage-env: line 434: /mnt/disk/home/buildslave-sage/slave/sage_git/build/local/lib/R/share/Makevars.site: No such file or directory
Found local metadata for cliquer-1.21.p2
Using cached file /mnt/disk/home/buildslave-sage/slave/sage_git/build/upstream/cliquer-1.21.tar.bz2
cliquer-1.21.p2
====================================================
Setting up build directory for cliquer-1.21.p2
Finished set up
****************************************************
Host system:
Linux volker-desktop 4.0.5-300.fc22.x86_64 #1 SMP Mon Jun 8 16:15:26 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux
****************************************************
C compiler: gcc
C compiler version:
Using built-in specs.
COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=/usr/libexec/gcc/x86_64-redhat-linux/5.1.1/lto-wrapper
Target: x86_64-redhat-linux
Configured with: ../configure --enable-bootstrap --enable-languages=c,c++,objc,obj-c++,fortran,ada,go,lto --prefix=/usr --mandir=/usr/share/man --infodir=/usr/share/info --with-bugurl=http://bugzilla.redhat.com/bugzilla --enable-shared --enable-threads=posix --enable-checking=release --enable-multilib --with-system-zlib --enable-__cxa_atexit --disable-libunwind-exceptions --enable-gnu-unique-object --enable-linker-build-id --with-linker-hash-style=gnu --enable-plugin --enable-initfini-array --disable-libgcj --with-default-libstdcxx-abi=c++98 --with-isl --enable-libmpx --enable-gnu-indirect-function --with-tune=generic --with-arch_32=i686 --build=x86_64-redhat-linux
Thread model: posix
gcc version 5.1.1 20150618 (Red Hat 5.1.1-4) (GCC) 
****************************************************
Applying ../patches/Makefile.patch
patching file Makefile
make[2]: Entering directory '/mnt/disk/home/buildslave-sage/slave/sage_git/build/local/var/tmp/sage/build/cliquer-1.21.p2/src'
gcc  -g -O3 -funroll-loops -fomit-frame-pointer -fPIC -Wall -DENABLE_LONG_OPTIONS -o cl.o -c cl.c
gcc  -g -O3 -funroll-loops -fomit-frame-pointer -fPIC -Wall  -I/mnt/disk/home/buildslave-sage/slave/sage_git/build/local/include  -c -o cliquer.o cliquer.c
gcc  -g -O3 -funroll-loops -fomit-frame-pointer -fPIC -Wall  -I/mnt/disk/home/buildslave-sage/slave/sage_git/build/local/include  -c -o graph.o graph.c
gcc  -g -O3 -funroll-loops -fomit-frame-pointer -fPIC -Wall  -I/mnt/disk/home/buildslave-sage/slave/sage_git/build/local/include  -c -o reorder.o reorder.c
gcc  -L/mnt/disk/home/buildslave-sage/slave/sage_git/build/local/lib -shared -Wl,-soname,libcliquer.so -o libcliquer.so cl.o cliquer.o graph.o reorder.o
make[2]: Leaving directory '/mnt/disk/home/buildslave-sage/slave/sage_git/build/local/var/tmp/sage/build/cliquer-1.21.p2/src'
mkdir: cannot create directory '/mnt/disk/home/buildslave-sage/slave/sage_git/build/local/include/cliquer/': No such file or directory
cp: target '/mnt/disk/home/buildslave-sage/slave/sage_git/build/local/include/cliquer/' is not a directory

real 0m2.027s
user 0m1.896s
sys 0m0.159s
Successfully installed cliquer-1.21.p2

IMHO we should just disallow new standard C-level packages that don't use autotools, or don't pass "make distcheck". Custom makefiles are always a recipe for failure.

CC: @jdemeyer @nathanncohen

Component: packages: standard

Author: Volker Braun

Branch: f5baeac

Reviewer: François Bissey

Issue created by migration from https://trac.sagemath.org/ticket/18808

@vbraun vbraun added this to the sage-6.8 milestone Jun 29, 2015
@vbraun
Copy link
Member Author

vbraun commented Jul 5, 2015

@vbraun
Copy link
Member Author

vbraun commented Jul 5, 2015

Author: Volker Braun

@vbraun
Copy link
Member Author

vbraun commented Jul 5, 2015

New commits:

f5baeacCreate intermediate directories when installing cliquer

@vbraun
Copy link
Member Author

vbraun commented Jul 5, 2015

Commit: f5baeac

@kiwifb
Copy link
Member

kiwifb commented Jul 5, 2015

Reviewer: François Bissey

@vbraun
Copy link
Member Author

vbraun commented Jul 7, 2015

@fchapoton
Copy link
Contributor

Changed commit from f5baeac to none

@fchapoton

This comment has been minimized.

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

3 participants