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

obs_cfht table file uses envAppend

    XMLWordPrintable

Details

    • Story
    • Status: Done
    • Resolution: Done
    • None
    • obs_cfht
    • None
    • 1
    • DRP S17-2
    • 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

          price Paul Price added a comment -

          boutigny, are you comfortable reviewing this? I think it's straightforward.

          price@price-laptop:~/LSST/obs/cfht (tickets/DM-8972=) $ git sub-patch
          commit f17453cc83e06b3390e7f4a2bc1e12ecc2897679
          Author: Paul Price <price@astro.princeton.edu>
          Date:   Wed Jan 11 13:16:23 2017 -0500
           
              ups: prepend to environment instead of append
              
              Prepending ensures that a newly 'setup' package overrides
              the system defaults.
           
          diff --git a/ups/obs_cfht.table b/ups/obs_cfht.table
          index 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)
          

          price Paul Price added a comment - boutigny , are you comfortable reviewing this? I think it's straightforward. price@price-laptop:~/LSST/obs/cfht (tickets/DM-8972=) $ git sub-patch commit f17453cc83e06b3390e7f4a2bc1e12ecc2897679 Author: Paul Price <price@astro.princeton.edu> Date: Wed Jan 11 13:16:23 2017 -0500   ups: prepend to environment instead of append Prepending ensures that a newly 'setup' package overrides the system defaults.   diff --git a/ups/obs_cfht.table b/ups/obs_cfht.table index 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)

          I do not see any problem with this change --> Merge

          boutigny Dominique Boutigny added a comment - I do not see any problem with this change --> Merge
          price Paul Price added a comment -

          Thanks boutigny!

          Merged to master.

          price Paul Price added a comment - Thanks boutigny ! Merged to master.

          People

            price Paul Price
            price Paul Price
            Dominique Boutigny
            Dominique Boutigny, Paul Price
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Jenkins

                No builds found.