# ngmix is a TaP package

XMLWordPrintable

#### Details

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

#### Activity

Hide
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
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
John Swinbank added a comment -

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

Show
John Swinbank added a comment - Yeah. Already fixed on a branch, in fact, but I was documenting it too... Thanks, though!
Hide
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
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
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
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
John Swinbank added a comment - - edited

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

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

#### People

Assignee:
John Swinbank
Reporter:
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.