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

Adjust sconsUtils to query python on path for executable location

    XMLWordPrintable

Details

    • Story
    • Status: Done
    • Resolution: Done
    • None
    • sconsUtils
    • 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

            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).

            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).
            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.

            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.
            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.

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

            Merged. Cleaned up the comments a bit.

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

            People

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

              Dates

                Created:
                Updated:
                Resolved:

                Jenkins

                  No builds found.