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

Put work directory inside project directory #113

Merged
merged 1 commit into from
Jun 22, 2019

Conversation

britter
Copy link

@britter britter commented Jun 20, 2019

Previously the rootDir was used as parent for the work directory. This
led to the situation where a subproject containing a jenkins plugin put
the work directory at the top of the project hierarchy. This becomes
especially problematic when several jenkins plugins are managed from one
root projects.

This change uses project.projectDir instead which will be the same as
project.rootDir for projects without sub projcts but will be the
actual project directory for nested projects.

Previously the rootDir was used as parent for the work directory. This
led to the situation where a subproject containing a jenkins plugin put
the work directory at the top of the project hierarchy. This becomes
especially problematic when several jenkins plugins are managed from one
root projects.

This change uses `project.projectDir` instead which will be the same as
`project.rootDir` for projects without sub projcts but will be the
actual project directory for nested projects.
@britter
Copy link
Author

britter commented Jun 22, 2019

@sghill is this project still active? I have some more, bigger changes I'd like to propose :-)

@sghill sghill merged commit 875b027 into jenkinsci:master Jun 22, 2019
@sghill
Copy link

sghill commented Jun 22, 2019

@britter it’s still active. Open to larger changes, but strong preference for backwards compatibility.

Thanks for the contribution!

@sghill
Copy link

sghill commented Jun 28, 2019

This change is now released in v0.33.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants