Bug #17976
ups v6_0_2 setups problem
Description
It seems that there is a problem with setups for ups v6_0_2. It removes an existing UPS_OVERRIDE variable instead of appending to it. This needs to be fixed before we can use ups v6 in a production environment.
The following is from my laptop. With new windows for each attempt.
mac-126940:~ garren$ export UPS_OVERRIDE="-H Darwin64bit+16" mac-126940:~ garren$ source unix/products/setup mac-126940:~ garren$ echo $UPS_OVERRIDE -H Darwin64bit+16 mac-126940:~ garren$ which ups /Users/garren/unix/products/ups/v6_0_2/Darwin64bit+16/bin/ups
mac-126940:~ garren$ export UPS_OVERRIDE="-H Darwin64bit+16" mac-126940:~ garren$ source unix/products/setups mac-126940:~ garren$ echo $UPS_OVERRIDE mac-126940:~ garren$ which ups /Users/garren/unix/products/ups/v6_0_2/Darwin64bit+16/bin/ups
For grins, I also tried export UPS_OVERRIDE="-H Dummy". The setup script failed, but setups happily ignored the override and reset it to empty.
FYI: These changes are not actually related to the ups v6 changes. Marc put them in some time ago, but we never made a new release with those changes.
Associated revisions
History
#1 Updated by Marc Mengel over 3 years ago
Just committed an initial patch for this...
Although we should also consider, if there is a ups_override.$hostname file, but UPS_OVERRIDE is already set, should we merge them, or override the override...
#2 Updated by Lynn Garren over 3 years ago
I just tested this. The existing behavior with ups v5_2_0 setup is that ups_override.$hostname takes precedence. Use of the file based hostname overrides should be rare, and I would expect them to take precedence.
#3 Updated by Lynn Garren over 3 years ago
- Status changed from New to Resolved
ups v6_0_3 is now tagged and built and available on SciSoft.
http://scisoft.fnal.gov/scisoft/packages/ups/v6_0_3/
SLF6 builds are also available in fnkits.
"ups" "v6_0_3" "Linux+2.6-2.12" "" "" "ups" "v6_0_3" "Linux64bit+2.6-2.12" "" ""
Marked as resolved, but it needs to be tested.
patch for issue #17976 -- don't stomp on UPS_OVERRIDE if we don't have a ups override file