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

ngmix is a TaP package

    XMLWordPrintable

    Details

    • Story Points:
      0.2
    • Epic Link:
    • Sprint:
      DRP X16-2
    • Team:
      Data Release Production

      Description

      Adding apparently innocent .gitignore and README.rst files to ngmix has broken it:

      [2016-04-26T22:01:09.614069eupspkg.prep (fatal): files found in root directory; guessing this is not a TaP package.
      		 
      Z] + export EUPS_PATH=/Users/jds/Projects/Astronomy/LSST/lsstsw/stack

      Please fix it.

        Attachments

          Issue Links

          relates to

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-2253 add third-party package builds for ngmix dependencies

          • Done

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-5386 Please document "TAP packages" and associated policy

          • Invalid
          mentioned in

          Page Loading...

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            I am guessing you know how to fix it. TAP_PACKAGE=1 in the eupspkg.cfg.sh and a .tap_package zero-length file in the root directory.

            Show
            tjenness Tim Jenness added a comment - I am guessing you know how to fix it. TAP_PACKAGE=1 in the eupspkg.cfg.sh and a .tap_package zero-length file in the root directory.
            Hide
            Permalink
            swinbank John Swinbank added a comment -

            Yeah. Already fixed on a branch, in fact, but I was documenting it too... Thanks, though!

            Show
            swinbank John Swinbank added a comment - Yeah. Already fixed on a branch, in fact, but I was documenting it too... Thanks, though!
            Hide
            Permalink
            swinbank John Swinbank added a comment - - edited

            Hey Jonathan Sick – could you review this please? Most of the effort here went into documentation rather than code changes. I think they are also an adequate solution for DM-5386; if you agree, let's mark that as a duplicate and kill two birds with one stone.

            Work is on tickets/DM-5867 on both ngmix and dm_dev_guide.

            Show
            swinbank John Swinbank added a comment - - edited Hey Jonathan Sick – could you review this please? Most of the effort here went into documentation rather than code changes. I think they are also an adequate solution for DM-5386 ; if you agree, let's mark that as a duplicate and kill two birds with one stone. Work is on tickets/ DM-5867 on both ngmix and dm_dev_guide .
            Hide
            Permalink
            jsick Jonathan Sick added a comment -

            I commented on the doc addition. Looks great! I was unaware of the DM-5386 ticket until now, but it sounds like this fixes it.

            Show
            jsick Jonathan Sick added a comment - I commented on the doc addition. Looks great! I was unaware of the DM-5386 ticket until now, but it sounds like this fixes it.
            Hide
            Permalink
            swinbank John Swinbank added a comment - - edited

            Thanks for the speedy turnaround! Updated as suggested & merged.

            Show
            swinbank John Swinbank added a comment - - edited Thanks for the speedy turnaround! Updated as suggested & merged.

              People

              Assignee:
              swinbank John Swinbank
              Reporter:
              swinbank John Swinbank
              Reviewers:
              Jonathan Sick
              Watchers:
              John Swinbank, Jonathan Sick, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.