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

New conda-build breaks conda-lsst

    Details

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

      Description

      The current version of conda-build creates a file in the working directory and breaks eups. This creates an error about TaP packages.

      The long term fix is to make our eups TaP packages use the new convention which is an environment variable (TAP_PACKAGE=1). This needs to be added to the eupspkg.cfg.sh in all LSST TaP packages.

      The workaround until this has happened is to move to an older version of conda-build.

        Attachments

          Issue Links

            Activity

            Hide
            tjenness Tim Jenness added a comment -

            Not all EUPS tap packages. Some of them do declare they are TaP.

            Show
            tjenness Tim Jenness added a comment - Not all EUPS tap packages. Some of them do declare they are TaP.
            Hide
            jmatt J Matt Peterson [X] (Inactive) added a comment -

            Use an older conda-build version until we implement a fix for this.

            Show
            jmatt J Matt Peterson [X] (Inactive) added a comment - Use an older conda-build version until we implement a fix for this.
            Hide
            jmatt J Matt Peterson [X] (Inactive) added a comment -

            The current version of conda-build is creating a conda_build.sh file in the work directory (./miniconda/conda-bld/work). This is identified by eups as a problem and eups creates an error about TaP.

            Show
            jmatt J Matt Peterson [X] (Inactive) added a comment - The current version of conda-build is creating a conda_build.sh file in the work directory (./miniconda/conda-bld/work). This is identified by eups as a problem and eups creates an error about TaP.
            Hide
            tjenness Tim Jenness added a comment -

            What am I meant to be reviewing?

            Show
            tjenness Tim Jenness added a comment - What am I meant to be reviewing?
            Hide
            jmatt J Matt Peterson [X] (Inactive) added a comment -

            Ahh I need to PR my ticket branch back from jmatt to lsst-sqre. But it's nearly trivial. I'm just capturing the work.

            Show
            jmatt J Matt Peterson [X] (Inactive) added a comment - Ahh I need to PR my ticket branch back from jmatt to lsst-sqre. But it's nearly trivial. I'm just capturing the work.
            Hide
            jmatt J Matt Peterson [X] (Inactive) added a comment -

            Use the older version of conda-build to avoid the TaP package problem created by the new version of conda-build.

            Show
            jmatt J Matt Peterson [X] (Inactive) added a comment - Use the older version of conda-build to avoid the TaP package problem created by the new version of conda-build.
            Hide
            tjenness Tim Jenness added a comment -

            Looks okay as a stop gap but I would have preferred the real fix. It should have only taken an hour to fix all the thirdparty eups packages. Is there a ticket for the real fix?

            Show
            tjenness Tim Jenness added a comment - Looks okay as a stop gap but I would have preferred the real fix. It should have only taken an hour to fix all the thirdparty eups packages. Is there a ticket for the real fix?
            Hide
            jmatt J Matt Peterson [X] (Inactive) added a comment -

            There is not. I will add one as soon as I have a new Conda epic.

            Show
            jmatt J Matt Peterson [X] (Inactive) added a comment - There is not. I will add one as soon as I have a new Conda epic.

              People

              • Assignee:
                jmatt J Matt Peterson [X] (Inactive)
                Reporter:
                jmatt J Matt Peterson [X] (Inactive)
                Reviewers:
                Tim Jenness
                Watchers:
                J Matt Peterson [X] (Inactive), Tim Jenness
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel