You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@uima.apache.org by bu...@apache.org on 2014/01/07 15:26:12 UTC
svn commit: r1556224 -
/uima/sandbox/uima-ducc/trunk/uima-ducc-cli/src/main/java/org/apache/uima/ducc/cli/CliBase.java
Author: burn
Date: Tue Jan 7 14:26:11 2014
New Revision: 1556224
URL: http://svn.apache.org/r1556224
Log:
UIMA-3525 Safer fix is to trim & only drop the empty service_dependency option that the SM chokes on
Modified:
uima/sandbox/uima-ducc/trunk/uima-ducc-cli/src/main/java/org/apache/uima/ducc/cli/CliBase.java
Modified: uima/sandbox/uima-ducc/trunk/uima-ducc-cli/src/main/java/org/apache/uima/ducc/cli/CliBase.java
URL: http://svn.apache.org/viewvc/uima/sandbox/uima-ducc/trunk/uima-ducc-cli/src/main/java/org/apache/uima/ducc/cli/CliBase.java?rev=1556224&r1=1556223&r2=1556224&view=diff
==============================================================================
--- uima/sandbox/uima-ducc/trunk/uima-ducc-cli/src/main/java/org/apache/uima/ducc/cli/CliBase.java (original)
+++ uima/sandbox/uima-ducc/trunk/uima-ducc-cli/src/main/java/org/apache/uima/ducc/cli/CliBase.java Tue Jan 7 14:26:11 2014
@@ -364,11 +364,12 @@ public abstract class CliBase
}
}
val = val.trim();
- if (val.length() > 0) {
+ // SM cannot handle an empty list of service dependencies
+ if (val.length() == 0 && opt.getLongOpt().equals(UiOption.ServiceDependency.pname())) {
+ if (debug) System.out.println("CLI dropped empty option " + opt.getLongOpt());
+ } else {
cli_props.put(opt.getLongOpt(), val);
if (debug) System.out.println("CLI set " + opt.getLongOpt() + " = '" + val + "'");
- } else {
- if (debug) System.out.println("CLI dropped empty option " + opt.getLongOpt());
}
}
}