Details
-
Type:
Improvement
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: None
-
Labels:
-
Story Points:2
-
Epic Link:
-
Sprint:Alert Production F16 - 9
-
Team:Alert Production
Description
The LSST software build tool setup script (in bin/setup.sh) currently assumes that the user is using a bash shell. Running this command from zsh doesn't generate any visible warnings, but since zsh doesn't know what to do with $BASH_SOURCE, it just leaves it blank, resulting in $LSSTSW being (silently!) set to the parent directory of the right directory instead of the right directory.
I think it should be pretty easy to just make a separate setup script for zsh (and (t)csh for that matter too), similar to what's already in $LSSTSW/eups/current/bin/. As far as I can tell, only the setup script needs to be duplicated, since the rest (e.g., rebuild) start with a bash she-bang to change the shell anyway.
Part of this issue should also be to modify the instructions at https://developer.lsst.io/build-ci/lsstsw.html to reflect the possibility of different shells.
Attachments
Issue Links
- duplicates
-
DM-4618 Make LSSTSW install work with tcsh
- Invalid
Sorry. Had forgotten to mark the review complete. In my rush I made a comment without closing it. One of you has to decide to own this ticket and merge the code. Simon Krughoff turned it into an Alert Production ticket so he should sort out who's closing it at his end.