Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Correction in CreateContainerCommand - Command with multiple args #34

Merged
merged 1 commit into from Aug 20, 2015

Conversation

Projects
None yet
3 participants
@BorX
Copy link
Contributor

commented Aug 18, 2015

Hi,
I couldn't create and start a container for Maven like specified in the description of the official image.
I had this result :

[Docker] ERROR: command 'Start container(s)' failed: Cannot start container my-maven-project: [8] System error: exec: "mvn clean install": executable file not found in $PATH

I think the simplest way to correct it is to consider the command string like an array splited by spaces.

In a second time, it will be a good idea to consider syntax like ["mvn","clean","install"].

Regards,

BorX

Correction in CreateContainerCommand - Command with multiple args
Instead of considering mvn clean install like one command (not found in $PATH), considers it like one command with 2 args
@jenkinsadmin

This comment has been minimized.

Copy link
Member

commented Aug 18, 2015

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

@vjuranek vjuranek merged commit fa97ea4 into jenkinsci:master Aug 20, 2015

1 check passed

Jenkins This pull request looks good
Details
@vjuranek

This comment has been minimized.

Copy link
Member

commented Aug 20, 2015

Merged, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.