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

Update newinstall.sh to support Python 3

    XMLWordPrintable

    Details

      Description

      newinstall.sh currently insists on installing and checking for python 2.7. This needs to be changed to allow Python 3.

      Requires sconsUtils works with Python 3 as the lsst EUPS package is installed as part of newinstall.sh.

        Attachments

          Issue Links

          is blocked by

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-6312 Update Scons EUPS package to work with Python 3 builds

          • Done

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-6313 Create miniconda3 EUPS package

          • Done

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-6319 Adjust sconsUtils to query python on path for executable location

          • Done
          is duplicated by

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-5047 newinstall.sh should allow Python 3 to be used

          • Invalid
          is triggering

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-7705 Fix two minor issues with newinstall script

          • Done

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            The real question here is whether we are adding a -3 command line switch to newinstall.sh or creating a new newinstall3.sh version. The former would be simpler but and minimizes code duplication. The latter is more obvious.

            Show
            tjenness Tim Jenness added a comment - The real question here is whether we are adding a -3 command line switch to newinstall.sh or creating a new newinstall3.sh version. The former would be simpler but and minimizes code duplication. The latter is more obvious.
            Hide
            Permalink
            jhoblitt Joshua Hoblitt added a comment -

            My preference would be for the former on the basis that it would avoid needing to worry about divergence between two versions of the essentially the same script.

            Show
            jhoblitt Joshua Hoblitt added a comment - My preference would be for the former on the basis that it would avoid needing to worry about divergence between two versions of the essentially the same script.
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            Can you please take a look at the new version of newinstall.sh. I can't test it in "real life" because no-one has published miniconda3 yet on our EUPS server and I'm not sure how to do that in the new system. Running with -n seems to do the right thing.

            Show
            tjenness Tim Jenness added a comment - Can you please take a look at the new version of newinstall.sh. I can't test it in "real life" because no-one has published miniconda3 yet on our EUPS server and I'm not sure how to do that in the new system. Running with -n seems to do the right thing.
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            Joshua Hoblitt if you can't review this can you please let me know so that I can find someone else. I need to close this epic.

            Show
            tjenness Tim Jenness added a comment - Joshua Hoblitt if you can't review this can you please let me know so that I can find someone else. I need to close this epic.
            Hide
            Permalink
            ktl Kian-Tat Lim added a comment -

            Mostly OK (see PR comments, including a fatal one), but won't work until miniconda3 is published.

            Show
            ktl Kian-Tat Lim added a comment - Mostly OK (see PR comments, including a fatal one), but won't work until miniconda3 is published.
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            Merged. I added travis tests for python 3 and extended the tests to also ensure that loadLSST.bash could be sourced.

            Show
            tjenness Tim Jenness added a comment - Merged. I added travis tests for python 3 and extended the tests to also ensure that loadLSST.bash could be sourced.

              People

              Assignee:
              tjenness Tim Jenness
              Reporter:
              tjenness Tim Jenness
              Reviewers:
              Kian-Tat Lim
              Watchers:
              Frossie Economou, J Matt Peterson [X] (Inactive), Joshua Hoblitt, Kian-Tat Lim, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.