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

Conda eups packages don't work if eups is already configured

    Details

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

      Description

      When Tim Jenness attempted to use the Conda repository he ran into a problem installing and using the packages because he already had an active EUPS_DIR and EUPS_PATH.

      When the eups package is installed and linked, it should warn users when these environment variables are set.

      I'm open to another solution but would prefer it doesn't change the eups package behavior. Changing behavior goes against the best practices for Conda and more generally packaging.

        Attachments

          Activity

          Hide
          tjenness Tim Jenness added a comment -

          I've added a note to the pipelines docs (on a ticket branch). You may or may not agree that it should be merged in this form. It might be more sensible in a troubleshooting section.

          Show
          tjenness Tim Jenness added a comment - I've added a note to the pipelines docs (on a ticket branch). You may or may not agree that it should be merged in this form. It might be more sensible in a troubleshooting section.
          Hide
          jmatt J Matt Peterson [X] (Inactive) added a comment -

          Reviewed on PR branch. Tim made the patch. I changed it per discussion. Jonathan Sick needs it merged and complete for now.

          Show
          jmatt J Matt Peterson [X] (Inactive) added a comment - Reviewed on PR branch. Tim made the patch. I changed it per discussion. Jonathan Sick needs it merged and complete for now.

            People

            • Assignee:
              jmatt J Matt Peterson [X] (Inactive)
              Reporter:
              jmatt J Matt Peterson [X] (Inactive)
              Reviewers:
              Tim Jenness
              Watchers:
              Frossie Economou, J Matt Peterson [X] (Inactive), Joshua Hoblitt, Mario Juric, Tim Jenness
            • Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Summary Panel