# Some pure python packages add to LD_LIBRARY_PATH

XMLWordPrintable

#### Details

• Type: Bug
• Status: Done
• Resolution: Done
• Fix Version/s: None
• Component/s:
• Labels:
• Story Points:
0.5
• Sprint:
AP S18-6
• Team:

#### Description

The pure python packages listed in Components have ups tables that add themselves to LD_LIBRARY_PATH and two related library paths. They should not do this. Remove the following from their ups table files:

 envPrepend(LD_LIBRARY_PATH, ${PRODUCT_DIR}/lib) envPrepend(DYLD_LIBRARY_PATH,${PRODUCT_DIR}/lib) envPrepend(LSST_LIBRARY_PATH, \${PRODUCT_DIR}/lib) 

#### Activity

Hide
Krzysztof Findeisen added a comment - - edited

Perhaps we need to add instructions for configuring ups/ to the dev guide. I think the table and config files are mostly cargo-culted right now.

Show
Krzysztof Findeisen added a comment - - edited Perhaps we need to add instructions for configuring ups/ to the dev guide. I think the table and config files are mostly cargo-culted right now.
Hide
Russell Owen added a comment -

The templates package is another reasonable source of this information (and the most likely source for new packages). That said, it could use some refinement of its description of pure python packages (based on my trying to figure out the right configuration for cbp recently).

Show
Russell Owen added a comment - The templates package is another reasonable source of this information (and the most likely source for new packages). That said, it could use some refinement of its description of pure python packages (based on my trying to figure out the right configuration for cbp recently).
Hide
Russell Owen added a comment - - edited

The templates package does indeed contain this info in the ups table file. I hope that will suffice.

I also updated obs_cfht as follows:

• Remove python 2 compatibility code.
• Move those bin scripts that have shbang lines into bin.src and enabled the usual processing of those, so they run on macOS
• Fixed flake8 warnings (not many) and enabled automatic flake8 testing
Show
Russell Owen added a comment - - edited The templates package does indeed contain this info in the ups table file. I hope that will suffice. I also updated obs_cfht as follows: Remove python 2 compatibility code. Move those bin scripts that have shbang lines into bin.src and enabled the usual processing of those, so they run on macOS Fixed flake8 warnings (not many) and enabled automatic flake8 testing
Hide
Tim Jenness added a comment -

FYI Your fixes to the shebangs in obs_lsstSim were reported on DM-13339.

Show
Tim Jenness added a comment - FYI Your fixes to the shebangs in obs_lsstSim were reported on DM-13339 .
Hide
Russell Owen added a comment -

Tim Jenness do you have time to look at this one?

Show
Russell Owen added a comment - Tim Jenness do you have time to look at this one?
Hide
Tim Jenness added a comment -

Looks good. I had reviewed the individual PRs this morning.

Show
Tim Jenness added a comment - Looks good. I had reviewed the individual PRs this morning.

#### People

Assignee:
Russell Owen
Reporter:
Russell Owen
Reviewers:
Tim Jenness
Watchers:
Krzysztof Findeisen, Russell Owen, Tim Jenness