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

Update Scons EUPS package to work with Python 3 builds

    XMLWordPrintable

Details

    Description

      Modify the Scons eups package such that it can be used to build a Python 3 stack.

      Originally the hope was to use a Scons that will work natively on Python 3 and 2.7 but this turned out not to be possible on the timescale required.

      Attachments

        Issue Links

          Activity

            tjenness Tim Jenness added a comment -

            Progress on porting Scons to python 3 by the Scons development team has been somewhat slower than expected. I am monitoring the situation.

            tjenness Tim Jenness added a comment - Progress on porting Scons to python 3 by the Scons development team has been somewhat slower than expected. I am monitoring the situation.
            tjenness Tim Jenness added a comment -

            A test branch has been created on my personal Github area as an experiment and significant progress has been made. Unfortunately only about 50% of the tests pass and there are many fiddly areas where bytes vs strings are not being handled properly. To make progress on the rest of the Python 3 port I have switched tack to forcing python2 for SCons but python3 for execution.

            tjenness Tim Jenness added a comment - A test branch has been created on my personal Github area as an experiment and significant progress has been made. Unfortunately only about 50% of the tests pass and there are many fiddly areas where bytes vs strings are not being handled properly. To make progress on the rest of the Python 3 port I have switched tack to forcing python2 for SCons but python3 for execution.
            tjenness Tim Jenness added a comment -

            Please review the branch tickets/DM-6312-usepy2 where SCons is patched to force python 2.7.

            tjenness Tim Jenness added a comment - Please review the branch tickets/ DM-6312 -usepy2 where SCons is patched to force python 2.7.
            price Paul Price added a comment -

            Please provide a bit more information in the commit message (e.g., why is the change necessary?).

            One comment says "EUPS has a bug". Please reference the bug report.

            price Paul Price added a comment - Please provide a bit more information in the commit message (e.g., why is the change necessary?). One comment says "EUPS has a bug". Please reference the bug report.
            tjenness Tim Jenness added a comment -

            Merged with update comment on installation of UPS dir and with expanded commit message.

            tjenness Tim Jenness added a comment - Merged with update comment on installation of UPS dir and with expanded commit message.

            People

              tjenness Tim Jenness
              tjenness Tim Jenness
              J Matt Peterson [X] (Inactive), Paul Price
              J Matt Peterson [X] (Inactive), Paul Price, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Jenkins

                  No builds found.