Project

General

Profile

Bug #17976

ups v6_0_2 setups problem

Added by Lynn Garren almost 2 years ago. Updated almost 2 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
10/19/2017
Due date:
% Done:

0%

Estimated time:
Spent time:
0.50 h
Experiment:
-
Duration:

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

Revision 2887 (diff)
Added by Marc Mengel almost 2 years ago

patch for issue #17976 -- don't stomp on UPS_OVERRIDE if we don't have a ups override file

History

#1 Updated by Marc Mengel almost 2 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 almost 2 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 almost 2 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.

Also available in: Atom PDF