[ 
https://issues.apache.org/activemq/browse/CAMEL-2549?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Mitko Kolev updated CAMEL-2549:
-------------------------------

    Attachment: camel-exec-patch2.diff

Hi Martin, 

here is something that could be relevant for you when you evaluate the code. 
In the tests in patch "camel-exec-patch.diff" i assumed, that "java" is 
available on the system path. I realized yesterday, that it would make sense to 
assume, that the JAVA_HOME environment variable is set, because JAVA_HOME is 
used in the mvn scripts anyway. 

Something else: in one of the tests i execute a test-script. Because JDK 1.5 
does not have the canExecute() method, the test could, at least in theory, 
break the build if there are not enough priviledges to execute the test-script. 
Therefore I set @Ignore on this test. 

I am sending a new patch that fixes those two issues. 
Thanks again.

> Camel Exec component
> --------------------
>
>                 Key: CAMEL-2549
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-2549
>             Project: Apache Camel
>          Issue Type: New Feature
>    Affects Versions: 2.3.0
>         Environment: Tested on Windows XP and Linux
>            Reporter: Mitko Kolev
>            Assignee: Martin Krasser
>             Fix For: 2.3.0
>
>         Attachments: camel-exec-patch.diff, camel-exec-patch2.diff
>
>


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to