Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 210735 - WorkflowRunner ignores multiple -p arguments
Summary: WorkflowRunner ignores multiple -p arguments
Status: VERIFIED FIXED
Alias: None
Product: EMFT
Classification: Modeling
Component: MWE (show other bugs)
Version: 0.7   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Bernd Kolb CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-11-23 05:30 EST by Christian Pfeifhofer CLA
Modified: 2011-05-20 05:06 EDT (History)
2 users (show)

See Also:


Attachments
Patch (827 bytes, patch)
2008-01-16 01:00 EST, Karsten Thoms CLA
sven.efftinge: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Christian Pfeifhofer CLA 2007-11-23 05:30:33 EST
Build ID: I20070621-1340

Steps To Reproduce:
1. Call WorkflowRunner from the command line using more than one -p (set property value) argument.
2. WorkflowRunner fails to run the workflow, displaying an error message about multiple occurences of -p being unsupported


More information:
WorkflowRunner uses hasOption instead of hasOptions for -p, meaning that effectively its only possible to hand over a single parameter.


To fix this issue, the line

final Option paramOption = OptionBuilder.withArgName("key=value").withDescription("external property that is handled as workflow property").hasArg().create(PARAM); 

has to be replaced by

final Option paramOption = OptionBuilder.withArgName("key=value").withDescription("external property that is handled as workflow property").hasArgs().create(PARAM);
Comment 1 Karsten Thoms CLA 2008-01-16 01:00:28 EST
Created attachment 87016 [details]
Patch

Patch for MWE
Comment 2 Karsten Thoms CLA 2008-01-16 01:08:39 EST
I have applied this patch to the oAW4.2 branch.

Outstanding is applying it also to HEAD, which is MWE now. Moving bug to MWE.
Comment 3 Bernd Kolb CLA 2008-01-16 03:27:48 EST
applied

Thanks!
Comment 4 Nick Boldt CLA 2008-01-28 16:56:17 EST
Move to verified as per bug 206558.