-
Notifications
You must be signed in to change notification settings - Fork 17
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
Fixes for VirtualBox #755
Fixes for VirtualBox #755
Conversation
We no longer install cython via apt, so try to find both python and cython in the path (prefering python3/cython3).
@@ -203,7 +215,8 @@ SuperBuild(){ | |||
-DBUILD_CIL=ON\ | |||
-DCYTHON_EXECUTABLE="$CYTHON_EXECUTABLE"\ | |||
-DPYTHON_EXECUTABLE="$PYTHON_EXECUTABLE"\ | |||
-DBUILD_pet_rd_tools=ON | |||
-DBUILD_pet_rd_tools=ON\ | |||
-DCYTHON_ROOT="${HOME}/.local/bin" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not necessary I believe
@@ -79,4 +79,5 @@ bash $userHOME/devel/SIRF-SuperBuild/VirtualBox/scripts/INSTALL_python_packages. | |||
# Therefore no browser is installed on the VM | |||
|
|||
chown -R $SIRFUSERNAME:users $userHOME | |||
export PATH=${userHOME}/.local/bin:${PATH} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
export PATH=${userHOME}/.local/bin:${PATH} |
This PATH will be ignored, so why set it
SET(Cython_FOUND TRUE) | ||
SET(Cython_Compilation_Failed FALSE) | ||
set(CYTHON_EXECUTABLE ${cy_exe}) | ||
message(status "YAS! ${cy_exe} ${CYTHON_EXECUTABLE} ${cython_version}") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
probably change this message :-)
export DEBIAN_FRONTEND=noninteractive | ||
SUDO=sudo | ||
APT_GET_INSTALL="$SUDO apt-get install -y --no-install-recommends" | ||
${APT_GET_INSTALL} ${PYTHON}-dev |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
move to INSTALL_...with_apt...
# $PYTHON -m pip install $PIPOPTIONS --upgrade pip wheel setuptools | ||
# $PYTHON -m pip install $PIPOPTIONS --only-binary=numpy,scipy,matplotlib numpy scipy matplotlib nose coverage docopt deprecation nibabel pytest tqdm | ||
# $PYTHON -m pip install $PIPOPTIONS jupyter spyder | ||
# $PYTHON -m pip uninstall $PIPOPTIONS -y spyder-kernels |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
# $PYTHON -m pip install $PIPOPTIONS --upgrade pip wheel setuptools | |
# $PYTHON -m pip install $PIPOPTIONS --only-binary=numpy,scipy,matplotlib numpy scipy matplotlib nose coverage docopt deprecation nibabel pytest tqdm | |
# $PYTHON -m pip install $PIPOPTIONS jupyter spyder | |
# $PYTHON -m pip uninstall $PIPOPTIONS -y spyder-kernels |
PYTHON_EXECUTABLE=$(which python3) | ||
else | ||
PYTHON_EXECUTABLE=$(which python) | ||
fi | ||
PYTHON_EXECUTABLE=/usr/bin/python3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PYTHON_EXECUTABLE=/usr/bin/python3 |
so, did it work? |
doing most of these myself at the moment |
I'm going to squash-merge this now, and then fix some things later. |
Yes, it did! I opened the PR only after it all worked! |
great. I'm changing it anyway 😄 PR coming later today. |
Creates the VM from the docker files.
Installs custom pip.
closes #742