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

Embed and install native libraries #51

Merged
merged 7 commits into from
Aug 20, 2015

Conversation

olivierdagenais
Copy link
Member

In preparation for unpacking the native libraries on agents, switch to unpacking them on the master, as well. This has the added benefit that it should no longer be necessary to restart the whole Java process (i.e. just reloading Jenkins itself, from, say, Tomcat) when upgrading the plugin, because the native libraries will be loaded from a location NOT under the tfs sub-folder.

Manual testing

Uploaded the tfs.hpi to my Jenkins server, restarted it for what should be the last time and ran a job: the Microsoft/TFS_SDK/14.0.1/ folder structure was created and the appropriate native libraries were loaded.
Mission accomplished!

Oli Dagenais added 7 commits August 18, 2015 20:09
This will be used to help extract the native libraries out of the JAR
and into a suitable spot for semi-permanent storage, on both the Jenkins
master and agents/nodes/slaves.
Verification was made possible with the NativeLibraryExtractor.
This pretty much completes the minimally viable implementation.
@jenkinsadmin
Copy link
Member

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

olivierdagenais added a commit that referenced this pull request Aug 20, 2015
@olivierdagenais olivierdagenais merged commit d423236 into master Aug 20, 2015
@olivierdagenais olivierdagenais deleted the embed_and_install_native_libraries branch August 20, 2015 03:02
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.

2 participants