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

Adjust sconsUtils to query python on path for executable location

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: sconsUtils
    • Labels:
      None

      Description

      sconsUtils has to be modified to ensure it works with Python 3. Additionally SWIG calls might need to be changed to trigger Python 3 mode.

        Attachments

          Issue Links

            Activity

            Hide
            tjenness Tim Jenness added a comment -

            If DM-6312 does not result in SCons running on Python 3 then the work on this ticket is much simpler. The call to sys.executable has to be replaced with a search of the user PATH. It does not seem to be required to pass any special flags to SWIG to get it to use Python3 compatible output (-python -py3 is not required).

            Show
            tjenness Tim Jenness added a comment - If DM-6312 does not result in SCons running on Python 3 then the work on this ticket is much simpler. The call to sys.executable has to be replaced with a search of the user PATH. It does not seem to be required to pass any special flags to SWIG to get it to use Python3 compatible output ( -python -py3 is not required).
            Hide
            tjenness Tim Jenness added a comment -

            Confirming that no port was done because we are still running SCons with Python 2. The only change on this ticket is the switch to ask the Python on PATH where it is located.

            Show
            tjenness Tim Jenness added a comment - Confirming that no port was done because we are still running SCons with Python 2. The only change on this ticket is the switch to ask the Python on PATH where it is located.
            Hide
            price Paul Price added a comment -

            The documentation standard isn't up to par, but you pointed out on GitHub that you're being consistent, which is the main thing.

            No objection to merging.

            Show
            price Paul Price added a comment - The documentation standard isn't up to par, but you pointed out on GitHub that you're being consistent, which is the main thing. No objection to merging.
            Hide
            tjenness Tim Jenness added a comment -

            Merged. Cleaned up the comments a bit.

            Show
            tjenness Tim Jenness added a comment - Merged. Cleaned up the comments a bit.

              People

              Assignee:
              tjenness Tim Jenness
              Reporter:
              tjenness Tim Jenness
              Reviewers:
              Paul Price
              Watchers:
              Paul Price, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.