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

            No builds found.
            tjenness Tim Jenness created issue -
            tjenness Tim Jenness made changes -
            Field Original Value New Value
            Epic Link DM-6179 [ 24693 ]
            tjenness Tim Jenness made changes -
            Link This issue is blocked by DM-5756 [ DM-5756 ]
            tjenness Tim Jenness made changes -
            Link This issue blocks DM-6316 [ DM-6316 ]
            tjenness Tim Jenness made changes -
            Link This issue blocks DM-6319 [ DM-6319 ]
            tjenness Tim Jenness made changes -
            Link This issue relates to DM-3680 [ DM-3680 ]
            Hide
            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.

            Show
            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 made changes -
            Status To Do [ 10001 ] In Progress [ 3 ]
            Hide
            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.

            Show
            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.
            Hide
            tjenness Tim Jenness added a comment -

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

            Show
            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 made changes -
            Reviewers J Matt Peterson, Paul Price [ jmatt, price ]
            Status In Progress [ 3 ] In Review [ 10004 ]
            Hide
            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.

            Show
            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 made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            Hide
            tjenness Tim Jenness added a comment -

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

            Show
            tjenness Tim Jenness added a comment - Merged with update comment on installation of UPS dir and with expanded commit message.
            tjenness Tim Jenness made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]
            tjenness Tim Jenness made changes -
            Summary Update Scons to v3.0 Update Scons EUPS package to work with Python 3 builds
            tjenness Tim Jenness made changes -
            Description Scons v3 is the scons version that supports both Python 3 and Python 2.7. This ticket is for updating Scons and ensuring that the Python 2.7 stack still builds.

            This work depends on the Scons developers delivering a new Scons by mid July. Whilst work is ongoing it may be necessary to help out with the port if we wish to meet our Python 3 target.
            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.
            tjenness Tim Jenness made changes -
            Link This issue relates to DM-8560 [ DM-8560 ]
            tjenness Tim Jenness made changes -
            Link This issue relates to DM-3680 [ DM-3680 ]

              People

              Assignee:
              tjenness Tim Jenness
              Reporter:
              tjenness Tim Jenness
              Reviewers:
              J Matt Peterson [X] (Inactive), Paul Price
              Watchers:
              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.