Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-1781

Tcsh special alias cwdcmd causes the v10 loadLSST.csh to fail.

    Details

    • Team:
      SQuaRE

      Description

      In loadLSST.csh the LSST_HOME environment variable is set by:

      if ( ! ${?LSST_HOME} ) then
        set LSST_HOME = `dirname ${this_script}`
        set LSST_HOME = `cd ${LSST_HOME} && pwd`
      endif
      

      If cwdcmd is used to set the window title with something like this:

      alias cwdcmd 'echo -n "2;${HOST}:$cwd ^G"'
      

      the cd command in the script echos those commands and does not set LSST_HOME properly.

        Attachments

          Issue Links

            Activity

            Hide
            ktl Kian-Tat Lim added a comment -

            The loadLSST scripts for other, more reasonable, shells look fine.

            While the stack is apparently not completely relocatable, we should be striving to make it so.

            Show
            ktl Kian-Tat Lim added a comment - The loadLSST scripts for other, more reasonable, shells look fine. While the stack is apparently not completely relocatable, we should be striving to make it so.
            Hide
            tjenness Tim Jenness added a comment -

            This code has not been fixed in nearly five years so it's highly likely we aren't going to fix it any time soon. Should we keep this ticket open and hope that someone else fixes it?

            Show
            tjenness Tim Jenness added a comment - This code has not been fixed in nearly five years so it's highly likely we aren't going to fix it any time soon. Should we keep this ticket open and hope that someone else fixes it?
            Hide
            swinbank John Swinbank added a comment -

            Give that we clearly don't have effort to prioritise this on-project, I think our options are to simply leave the ticket open (as Tim suggests), or to acknowledge that we can't support this code and remove loadLSST.csh entirely. I have no real preference.

            Show
            swinbank John Swinbank added a comment - Give that we clearly don't have effort to prioritise this on-project, I think our options are to simply leave the ticket open (as Tim suggests), or to acknowledge that we can't support this code and remove loadLSST.csh entirely. I have no real preference.
            Hide
            tjenness Tim Jenness added a comment -

            I vote for the option that involves less work on this ticket. (ie leave it open and move on). If someone else complains about tcsh support we can reconsider.

            Show
            tjenness Tim Jenness added a comment - I vote for the option that involves less work on this ticket. (ie leave it open and move on). If someone else complains about tcsh support we can reconsider.
            Hide
            cwalter Chris Walter added a comment -

            Sorry... am traveling now. I agree that if we close it we should remove the .csh script and say it is not supported.

            I personally moved to Bash because of this.  But as long as csh is actually supported there should be a way for someone to find this information. It is possible to use but you need to be careful.

            So I agree with leaving it open.

            Show
            cwalter Chris Walter added a comment - Sorry... am traveling now. I agree that if we close it we should remove the .csh script and say it is not supported. I personally moved to Bash because of this.  But as long as csh is actually supported there should be a way for someone to find this information. It is possible to use but you need to be careful. So I agree with leaving it open.

              People

              • Assignee:
                frossie Frossie Economou
                Reporter:
                cwalter Chris Walter
                Watchers:
                Chris Walter, John Swinbank, Kian-Tat Lim, Tim Jenness
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:

                  Summary Panel