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.

Sign up for GitHub

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

Jump to bottom

fix optional dependency #70

Closed
wants to merge 2 commits into from
Closed

fix optional dependency #70

wants to merge 2 commits into from

Conversation

sdepold
Copy link
Contributor

@sdepold sdepold commented Dec 17, 2013

This PR fixes the optional dependency between File["${plugin_dir}/${plugin}"] and Exec["download-${name}"].

@jenkinsadmin
Copy link

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

@sdepold
Copy link
Contributor Author

sdepold commented Dec 18, 2013

Hm I wonder if travis is failing because of my code change.

@jchristi jchristi mentioned this pull request Jan 6, 2014
Merged
@jchristi
Copy link
Contributor

jchristi commented Jan 7, 2014

@sdepold can you confirm #74 fixes this issue?

@jlambert121
Copy link
Contributor

@sdepold Please let us know if this wasn't resolved with the mentioned PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@sdepold @jenkinsadmin @jchristi @jlambert121