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
DM-6316: Add option to allow miniconda3 to be installed #27
Conversation
c80057e
to
9f00935
Compare
@@ -282,11 +306,18 @@ set -e | |||
########## Download optional component (python, git, ...) | |||
|
|||
if true; then | |||
if [[ $WITH_MINICONDA2 = 1 ]]; then |
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.
The single =
here looked inconsistent so I changed that to ==
.
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.
I'm confused. sh uses = not ==.
I don't think [[ ... ]] is in posix:
The following words may be recognized as reserved words on some implementations (when none of the characters are quoted), causing unspecified results:
[[ ]] function select
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.
This script is actually documented to be a bash script and not a sh script (see the installation instructions: https://pipelines.lsst.io/install/newinstall.html ) so I don't think there is a problem.
I've cleaned up the tab issues. |
else | ||
if [[ $PYVEROK != 1 ]]; then | ||
cat <<-EOF | ||
|
||
LSST stack requires Python 2.7; you seem to have $(python -V 2>&1) on your | ||
LSST stack requires Python 2.7 or >=3.5; you seem to have $(python -V 2>&1) on your |
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.
If we're going to allow a (nonexistent) 2.8 in the code, shouldn't you put a ">=" on the 2.7, too?
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.
So long as that does not confuse people into thinking 3.1 is okay.
4a09f88
to
999db58
Compare
Also sources the loadLSST script to ensure that is a valid bash script.
999db58
to
574217c
Compare
No description provided.