Uploaded image for project: 'Request For Comments'
  1. Request For Comments
  2. RFC-375

Change default python version in newinstall and lsstsw to Python3 for v14

    Details

    • Type: RFC
    • Status: Implemented
    • Resolution: Done
    • Component/s: DM
    • Labels:
      None

      Description

      I propose that we change the default version of python 3 that is installed via newinstall or bin/deploy to be Python3. I would like this to be done for the v14 release so that we can make a statement that this is where we are heading. The release notes should absolutely state how to use python2 for a build. We regularly use python3 now and Jenkins has been testing it for ages and sims builds fine.

      This is not talking about removing python2. I am just saying that in the absence of any options given to these commands we should use python3.

        Attachments

          Issue Links

            Activity

            Hide
            Parejkoj John Parejko added a comment -

            +1: a great idea.

            Show
            Parejkoj John Parejko added a comment - +1: a great idea.
            Hide
            swinbank John Swinbank added a comment - - edited

            For what it's worth, I don't see why switching shared stacks to Python 3 should be coupled to using devtoolset-6 (except that it could save a little effort if we were to change everything at once).

            Show
            swinbank John Swinbank added a comment - - edited For what it's worth, I don't see why switching shared stacks to Python 3 should be coupled to using devtoolset-6 (except that it could save a little effort if we were to change everything at once).
            Hide
            tjenness Tim Jenness added a comment -

            John Swinbank I agree. It's a bit less work but they are independent decisions.

            Show
            tjenness Tim Jenness added a comment - John Swinbank I agree. It's a bit less work but they are independent decisions.
            Hide
            rowen Russell Owen added a comment - - edited

            +1 seems like a great time to make this change

            Show
            rowen Russell Owen added a comment - - edited +1 seems like a great time to make this change
            Hide
            tjenness Tim Jenness added a comment -

            No complaints. Adopting. I've opened pull requests for the minor changes.

            Show
            tjenness Tim Jenness added a comment - No complaints. Adopting. I've opened pull requests for the minor changes.

              People

              • Assignee:
                tjenness Tim Jenness
                Reporter:
                tjenness Tim Jenness
                Watchers:
                Adam Thornton, Frossie Economou, Hsin-Fang Chiang, John Parejko, John Swinbank, Jonathan Sick, Joshua Hoblitt, Pim Schellart [X] (Inactive), Russell Owen, Steve Pietrowicz, Tim Jenness
              • Votes:
                1 Vote for this issue
                Watchers:
                11 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Planned End:

                  Summary Panel