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

Change envAppend to envPrepend

    XMLWordPrintable

Details

    • Story
    • Status: Done
    • Resolution: Done
    • None
    • None
    • None

    Description

      There are several Python packages in the stack, where the package's python directory is appended to PYTHONPATH environment variable instead of prepending. This should be fixed across all packages in the stack.

       

      This ticket is filed motivated by this conversation in a code review: https://github.com/lsst-dm/meas_extensions_gaap/pull/3#discussion_r578639435

      Attachments

        Issue Links

          Activity

            tjenness, I don't see the use of envAppend in fgcmcal

            kannawad Arun Kannawadi added a comment - tjenness , I don't see the use of envAppend in fgcmcal
            tjenness Tim Jenness added a comment - https://github.com/lsst/fgcm/blob/master/ups/fgcm.table

            Thanks, fixed it. I'll let Eli merge the PR to fgcm when convenient. For other packages, would either Tim or K-T be the reviewers?

            kannawad Arun Kannawadi added a comment - Thanks, fixed it. I'll let Eli merge the PR to fgcm when convenient. For other packages, would either Tim or K-T be the reviewers?
            ktl Kian-Tat Lim added a comment -

            A problem with obs_wiyn, and I think lsstserver should remain as is, but otherwise OK.

            ktl Kian-Tat Lim added a comment - A problem with obs_wiyn, and I think lsstserver should remain as is, but otherwise OK.

            Fixed the problem with obs_wiyn and merged all PRs except the lsstserver package one. Marking it as Done now.

            kannawad Arun Kannawadi added a comment - Fixed the problem with obs_wiyn and merged all PRs except the lsstserver package one. Marking it as Done now.

            People

              kannawad Arun Kannawadi
              kannawad Arun Kannawadi
              Kian-Tat Lim
              Arun Kannawadi, Kian-Tat Lim, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Jenkins

                  No builds found.