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

Change envAppend to envPrepend

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      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

          relates to

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-868 Audit ups table files to replace envAppend with envPrepend

          • Done

            Activity

            Ascending order - Click to sort in descending order
            6 older comments
            Hide
            Permalink
            kannawad Arun Kannawadi added a comment -

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

            Show
            kannawad Arun Kannawadi added a comment - Tim Jenness , I don't see the use of envAppend in fgcmcal
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            https://github.com/lsst/fgcm/blob/master/ups/fgcm.table

            Show
            tjenness Tim Jenness added a comment - https://github.com/lsst/fgcm/blob/master/ups/fgcm.table
            Hide
            Permalink
            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?

            Show
            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?
            Hide
            Permalink
            ktl Kian-Tat Lim added a comment -

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

            Show
            ktl Kian-Tat Lim added a comment - A problem with obs_wiyn, and I think lsstserver should remain as is, but otherwise OK.
            Hide
            Permalink
            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.

            Show
            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

              Assignee:
              kannawad Arun Kannawadi
              Reporter:
              kannawad Arun Kannawadi
              Reviewers:
              Kian-Tat Lim
              Watchers:
              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.