Skip to content

Commit

Permalink
Merge pull request #76 from logicabrity/clean-up-bin-dir
Browse files Browse the repository at this point in the history
Clean up bin dir
  • Loading branch information
logicabrity authored Aug 9, 2016
2 parents 58cbfea + 5ef6d96 commit d4be4ac
Show file tree
Hide file tree
Showing 6 changed files with 16 additions and 55 deletions.
7 changes: 6 additions & 1 deletion fix_load_path_mac.py → bin/fix_load_path_mac.py
Original file line number Diff line number Diff line change
@@ -1,3 +1,8 @@
"""
Creates working dylibs on Mac OS X by adding the full path to referenced libraries.
This fixes ImportErrors that may appear starting with SUNDIALS 2.6.
"""
import os
import glob
import re
Expand Down Expand Up @@ -27,4 +32,4 @@ def extract_library(so_file):

for so in sos:
extract_library(so)
print("Done for %s!"%so)
print("Done for %s!"%so)
1 change: 1 addition & 0 deletions bin/install-python-packages.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
pip install --upgrade cython psutil ipywidgets
46 changes: 0 additions & 46 deletions bin/install-sundials-2.5.sh

This file was deleted.

9 changes: 6 additions & 3 deletions bin/install-ubuntu-packages.sh
Original file line number Diff line number Diff line change
@@ -1,4 +1,7 @@
# required to compile fidimag
apt-get install python-pip python-numpy python-dev python-scipy cmake
# required for tests and running it
apt-get install python-pytest python-pyvtk ipython python-matplotlib
deps_compilation="python-pip python-numpy python-dev python-scipy cmake"

# required for tests and running fidimag
deps_live="python-pytest python-pyvtk ipython python-matplotlib"

apt-get install $deps_compilation $deps_live
6 changes: 2 additions & 4 deletions bin/ubuntu_install_script.sh
Original file line number Diff line number Diff line change
@@ -1,15 +1,13 @@
echo "Super user authentication required to add packages (you will be \
prompted for confirmation)."
sudo bash install-ubuntu-packages.sh
sudo bash install-python-packages.sh
bash install-fftw.sh
bash install-sundials.sh
sudo pip install cython --upgrade
sudo pip install psutil ipywidgets
pushd .. > /dev/null
make

# Adds Fidimag environment variables to profile.d, if they're not already
# there.
# Adds fidimag environment variables to profile.d, if they're not already there.
FIDIMAG_PROFILE_PATH=/etc/profile.d/fidimag.sh
if [ ! -e "$FIDIMAG_PROFILE_PATH" ]; then
echo "Warning: Adding Fidimag to path at $FIDIMAG_PROFILE_PATH."
Expand Down
2 changes: 1 addition & 1 deletion doc/install.rst
Original file line number Diff line number Diff line change
Expand Up @@ -259,7 +259,7 @@ which can be seen by using ``otool -L fidimag/extensions/clib.so``, ::
/usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1213.0.0)
/opt/local/lib/libgcc/libgcc_s.1.dylib (compatibility version 1.0.0, current version 1.0.0)

A solution is to change these path manually with ``install_name_tool``, so we have provided a python script (``fix_load_path_mac.py``) to do this.
A solution is to change these path manually with ``install_name_tool``, so we have provided a python script (``bin/fix_load_path_mac.py``) to do this.
After run the script, the library path is fixed, for example, the output of the cmd ``otool -L fidimag/extensions/clib.so`` gives ::

fidimag/extensions/neb_clib.so:
Expand Down

0 comments on commit d4be4ac

Please sign in to comment.