Skip to content
This repository has been archived by the owner on Jan 25, 2022. It is now read-only.

Fix lock release when unexpected errors happen during the build #32

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

joshughes
Copy link

When I use the rbenv plugin there are many times where if a build fails in an unexpected way the .rbenv_lock file is left behind and other builds fail on the slave node because the .rbenv_lock is still present on the machine.

This PR uses the teardown function to ensure that if a build set the .rbenv_lock that it cleans it up despite any errors that happened during the build.

I also added --no-rdoc and --no-ri to the gem installation command to speed up installation of required gems.

@cloudbees-pull-request-builder

plugins » rbenv-plugin #17 SUCCESS
This pull request looks good

@cloudbees-pull-request-builder

plugins » rbenv-plugin #18 SUCCESS
This pull request looks good

@jenkinsadmin
Copy link
Member

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

@cloudbees-pull-request-builder

plugins » rbenv-plugin #19 SUCCESS
This pull request looks good

@cloudbees-pull-request-builder

plugins » rbenv-plugin #20 SUCCESS
This pull request looks good

@cloudbees-pull-request-builder

plugins » rbenv-plugin #21 SUCCESS
This pull request looks good

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

Successfully merging this pull request may close these issues.

3 participants