Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 368356

Summary:

Update of LML_da (-> 1.14)

Product:

[Tools] PTP

Reporter:

Wolfgang Frings <W.Frings>

Component:

Assignee:

Wolfgang Frings <W.Frings>

Status:

RESOLVED FIXED

QA Contact:

Severity:

normal

Priority:

Version:

5.0.4

Target Milestone:

5.0.5

Hardware:

OS:

Linux

Whiteboard:

Attachments:

Description	Flags
Patch (version 1.12 -> 1.14)	g.watson: iplog-

Description Wolfgang Frings

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

2012-01-11 10:34:42 EST

Created attachment 209317 [details]
Patch (version 1.12 -> 1.14)

Comment 2 Greg Watson

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

2012-02-08 14:45:17 EST

Committed.