#922: GSL ----------------------+----------------------------------------------------- Reporter: knarf | Owner: Type: defect | Status: new Priority: critical | Milestone: ET_2012_05 Component: Other | Version: Keywords: | ----------------------+----------------------------------------------------- ExternalLibraries doesn't put -L directories in GSL_LIB_DIRS, but in GSL_LIBS, thus Cactus doesn't add an -rpath thus execution fails on some machines (trestles being one).
#922: GSL -----------------------+---------------------------------------------------- Reporter: knarf | Owner: eschnett Type: defect | Status: assigned Priority: critical | Milestone: ET_2012_05 Component: Other | Version: Resolution: | Keywords: -----------------------+---------------------------------------------------- Changes (by eschnett):
* owner: => eschnett * status: new => assigned
Comment:
Well spotted. GSL.sh needs to be corrected.
#922: GSL -----------------------+---------------------------------------------------- Reporter: knarf | Owner: eschnett Type: defect | Status: review Priority: critical | Milestone: ET_2012_05 Component: Other | Version: Resolution: | Keywords: -----------------------+---------------------------------------------------- Changes (by eschnett):
* status: assigned => review
Comment:
{{{ $ svn diff Index: GSL.sh =================================================================== --- GSL.sh (revision 26) +++ GSL.sh (working copy) @@ -147,12 +147,15 @@
# Set options if [ -x ${GSL_DIR}/bin/gsl-config ]; then - # Obtain configuration options from GSL's configuration - GSL_INC_DIRS="$(echo '' $(${GSL_DIR}/bin/gsl-config --cflags) '' | sed -e 's/ -I/ /g' | sed -e 's+ -L/include + +g;s+ -L/usr/include + +g;s+ -L/usr/local/include + +g')" - # Don't try to split library directories and libraries -- just put - # everything into "libraries" - GSL_LIB_DIRS='' - GSL_LIBS="$(echo '' $(${GSL_DIR}/bin/gsl-config --libs) '' | sed -e 's/ -l/ /g' | sed -e 's+ -L/lib + +g;s+ -L/lib64 + +g;s+ -L/usr/lib + +g;s+ -L/usr/lib64 + +g;s+ -L/usr/local/lib + +g;s+ -L/usr/local/lib64 + +g')" + # Obtain configuration options from GSL's configuration: + # - for INC_DIRS, remove "standard" directories, and remove -I + # prefix from flags + # - for LIB_DIRS, remove all -l flags, and remove "standard" + # directories, and remove -L prefix from flags + # - for LIBS, keep only -l flags, and remove -l prefix from flags + GSL_INC_DIRS="$(echo '' $(${GSL_DIR}/bin/gsl-config --cflags) '' | sed -e 's+ -I/include + +g;s+ -I/usr/include + +g;s+ -I/usr/local/include + +g' | sed -e 's/ -I/ /g')" + GSL_LIB_DIRS="$(echo '' $(${GSL_DIR}/bin/gsl-config --libs) '' | sed -e 's/ -l[^ ]*/ /g' | sed -e 's+ -L/lib + +g;s+ -L/lib64 + +g;s+ -L/usr/lib + +g;s+ -L/usr/lib64 + +g;s+ -L/usr/local/lib + +g;s+ -L/usr/local/lib64 + +g' | sed -e 's/ -L/ /g')" + GSL_LIBS="$(echo '' $(${GSL_DIR}/bin/gsl-config --libs) '' | sed -e 's/ -[^l][^ ]*/ /g' | sed -e 's/ -l/ /g')" fi
# Pass options to Cactus }}}
#922: GSL -----------------------+---------------------------------------------------- Reporter: knarf | Owner: eschnett Type: defect | Status: review Priority: critical | Milestone: ET_2012_05 Component: Other | Version: Resolution: | Keywords: -----------------------+----------------------------------------------------
Comment (by knarf):
The patch looks good and works for lonestar and trestles. Please apply.
#922: GSL -----------------------+---------------------------------------------------- Reporter: knarf | Owner: eschnett Type: defect | Status: reviewed_ok Priority: critical | Milestone: ET_2012_05 Component: Other | Version: Resolution: | Keywords: -----------------------+---------------------------------------------------- Changes (by knarf):
* status: review => reviewed_ok
#922: GSL -----------------------+---------------------------------------------------- Reporter: knarf | Owner: eschnett Type: defect | Status: closed Priority: critical | Milestone: ET_2012_05 Component: Other | Version: Resolution: fixed | Keywords: -----------------------+---------------------------------------------------- Changes (by eschnett):
* status: reviewed_ok => closed * resolution: => fixed
Comment:
Applied.
trac@lists.einsteintoolkit.org