Don't overwrite LD_LIBRARY_PATH variable

coin-or / coinbrew

COIN-OR build and installation script

29 stars 15 forks source link

Don't overwrite LD_LIBRARY_PATH variable #36

Closed ianbeyst closed 4 years ago

ianbeyst commented 4 years ago

Breaking people's preexisting setup is easily avoided by appending to LD_LIBRARY_PATH instead of overwriting it. In my case, this caused my linker to fail finding my OpenBLAS installation in /opt/OpenBLAS causing the unit tests at the end of the Ipopt installation to fail.

tkralphs commented 4 years ago

OK, I was thinking there would be no harm in this, but I didn't think about the particular case you're describing. Thanks!

ianbeyst commented 4 years ago

More than welcome. Cheers!