You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by js...@apache.org on 2008/03/31 16:30:25 UTC

svn commit: r643003 - /activemq/camel/trunk/tooling/maven/camel-maven-plugin/src/main/java/org/apache/camel/maven/DotMojo.java

Author: jstrachan
Date: Mon Mar 31 07:30:16 2008
New Revision: 643003

URL: http://svn.apache.org/viewvc?rev=643003&view=rev
Log:
applied patch for CAMEL-412 with thanks!

Modified:
    activemq/camel/trunk/tooling/maven/camel-maven-plugin/src/main/java/org/apache/camel/maven/DotMojo.java

Modified: activemq/camel/trunk/tooling/maven/camel-maven-plugin/src/main/java/org/apache/camel/maven/DotMojo.java
URL: http://svn.apache.org/viewvc/activemq/camel/trunk/tooling/maven/camel-maven-plugin/src/main/java/org/apache/camel/maven/DotMojo.java?rev=643003&r1=643002&r2=643003&view=diff
==============================================================================
--- activemq/camel/trunk/tooling/maven/camel-maven-plugin/src/main/java/org/apache/camel/maven/DotMojo.java (original)
+++ activemq/camel/trunk/tooling/maven/camel-maven-plugin/src/main/java/org/apache/camel/maven/DotMojo.java Mon Mar 31 07:30:16 2008
@@ -375,9 +375,14 @@
     }
 
     protected String convertFile(File file, String format) throws CommandLineException {
-        String generatedFileName = removeFileExtension(file.getAbsolutePath()) + "." + format;
         Log log = getLog();
+        if (this.executable == null || this.executable.length() == 0) {
+
+            log.warn( "Parameter <executable/> was not set in the pom.xml.  Skipping conversion." );
+            return null;
+        }
 
+        String generatedFileName = removeFileExtension(file.getAbsolutePath()) + "." + format;
         Commandline cl = new Commandline();
         cl.setExecutable(executable);
         cl.createArgument().setValue("-T" + format);