Details
-
Type:
Story
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: obs_cfht
-
Labels:None
-
Story Points:1
-
Epic Link:
-
Sprint:DRP S17-2
-
Team:Data Release Production
Description
obs_cfht.table includes:
envAppend(DYLD_LIBRARY_PATH, ${PRODUCT_DIR}/lib)
|
envAppend(PYTHONPATH, ${PRODUCT_DIR}/python)
|
envAppend(PATH, ${PRODUCT_DIR}/bin)
|
These envAppend calls should be envPrepend.
Attachments
Activity
I do not see any problem with this change --> Merge
Dominique Boutigny
added a comment - I do not see any problem with this change --> Merge Thanks Dominique Boutigny!
Merged to master.
Paul Price
added a comment - Thanks Dominique Boutigny !
Merged to master. 
Dominique Boutigny, are you comfortable reviewing this? I think it's straightforward.
price@price-laptop:~/LSST/obs/cfht (tickets/DM-8972=) $ git sub-patchcommit f17453cc83e06b3390e7f4a2bc1e12ecc2897679Author: Paul Price <price@astro.princeton.edu>Date: Wed Jan 11 13:16:23 2017 -0500ups: prepend to environment instead of appendPrepending ensures that a newly 'setup' package overridesthe system defaults.diff --git a/ups/obs_cfht.table b/ups/obs_cfht.tableindex 22a9b16..69d2512 100644--- a/ups/obs_cfht.table+++ b/ups/obs_cfht.table@@ -5,6 +5,6 @@ setupRequired(pipe_tasks)setupOptional(testdata_cfht)envPrepend(LD_LIBRARY_PATH, ${PRODUCT_DIR}/lib)-envAppend(DYLD_LIBRARY_PATH, ${PRODUCT_DIR}/lib)-envAppend(PYTHONPATH, ${PRODUCT_DIR}/python)-envAppend(PATH, ${PRODUCT_DIR}/bin)+envPrepend(DYLD_LIBRARY_PATH, ${PRODUCT_DIR}/lib)+envPrepend(PYTHONPATH, ${PRODUCT_DIR}/python)+envPrepend(PATH, ${PRODUCT_DIR}/bin)