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

Adjust sconsUtils to query python on path for executable location

    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: