Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 368356 - Update of LML_da (-> 1.14)
Summary: Update of LML_da (-> 1.14)
Status: RESOLVED FIXED
Alias: None
Product: PTP
Classification: Tools
Component: RM (show other bugs)
Version: 5.0.4   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 5.0.5   Edit
Assignee: Wolfgang Frings CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-01-11 10:03 EST by Wolfgang Frings CLA
Modified: 2012-05-23 17:42 EDT (History)
0 users

See Also:

Attachments
Patch (version 1.12 -> 1.14) (308.81 KB, patch)
2012-01-11 10:34 EST, Wolfgang Frings CLA 		g.watson: iplog-
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Wolfgang Frings CLA 2012-01-11 10:03:50 EST 
Update of LML_da to version 1.14 

1.13:
   - improved reporting: option -verbose control debug output on stderr
     full debugging report will written $tmpdir/report.log
   - improved support for PBS (vnodes)
   - bug fixes

1.14:
   - Improvements for testing and demonstration purpose 
     - option -test: use existing workflow input in tmpdir
     - options specification file .LML_da_options 
       - will be read if exists in current directory
       - will overwrite command line option
   - incoporated support for new RMS (GridEngine, PE)
     (contributed by others)
Comment 1 Wolfgang Frings CLA 2012-01-11 10:34:42 EST 
Created attachment 209317 [details]
Patch (version 1.12 -> 1.14)
Comment 2 Greg Watson CLA 2012-01-23 09:19:02 EST 
I noticed that the LML_da_driver.pl contains this code:

			if(exists($driver_ref->{attr}->{name})) {
			    $rms=uc($driver_ref->{attr}->{name}); # upper case, except:  
			    $rms="GridEngine" if ($rms=~/GRIDENGINE/);
			    &report_if_verbose("$0: check_for rms, got hint from request ... ($rms)\n");
		    }

It looks like this is just to accommodate having a directory in mixed case, rather than all uppercase. Is there any need for this? It seems a bit arbitrary.
Comment 3 Greg Watson CLA 2012-02-08 14:45:17 EST 
Committed.