search

conda-forge / glpk-feedstock

A conda-smithy repository for glpk.
BSD 3-Clause "New" or "Revised" License
2 stars 12 forks source link

OS X recipe failing on `make check` step of build. #3

Closed jjhelmus closed 8 years ago

jjhelmus commented 8 years ago

The OS X CI build is failing in the make check step due to the incorrect libgmp being loaded. The error is:

+ make check
Making check in src
make[1]: Nothing to be done for `check'.
Making check in examples
./glpsol --version
dyld: Library not loaded: @rpath/libgmp.10.dylib
  Referenced from: /Users/travis/miniconda3/conda-bld/work/glpk-4.59/examples/.libs/glpsol
  Reason: Incompatible library version: glpsol requires version 14.0.0 or later, but libgmp.10.dylib provides version 13.0.0
make[1]: *** [check] Trace/BPT trap: 5
make: *** [check-recursive] Error 1
Command failed: /bin/bash -x -e /Users/travis/build/conda-forge/glpk-feedstock/recipe/build.sh

I think the brew libgmp may be picked up by the loader which is causing this issue. I'm do not know why an earlier run of this build in PR #2 was successful.

jakirkham commented 8 years ago

Yes, the problem is it is using gmp from brew. Though we now provide our own gmp. Try using this.

jakirkham commented 8 years ago

Probably we should eliminate everything from brew as it is becoming a nuisance.

jakirkham commented 8 years ago

Added this issue ( https://github.com/conda-forge/conda-smithy/issues/103 ) so that we can take a look at just purging brew in all cases. We don't want to accidentally become somewhat dependent on brew IMHO. So, we should just start purging it completely and see what we may still need as dependencies in our packaging ecosystem. Perhaps some select things will be installed afterwards.

jjhelmus commented 8 years ago

I don't think the LD_LIBRARY_PATH trick will work because the variables is not exported in the makefile. I'll submit a PR to check but at least on my Mac (with no system libgmp) I still get an image missing error.

jakirkham commented 8 years ago

So, maybe something is broken with our gmp.

jakirkham commented 8 years ago

Looks like it does work, but we should definitely move away from having to do this. The greatest benefit of CI is we can use a minimal image to build and test what we need. At present, it doesn't feel minimal enough.