-
Notifications
You must be signed in to change notification settings - Fork 38
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
Update build parent to 3.4 #10
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Once the Core version issue is fixed, I think it can be merged
<version>1.4.8</version> | ||
<packaging>hpi</packaging> | ||
<properties> | ||
<jenkins.version>1.625</jenkins.version> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<jenkins.version>1.625</jenkins.version> | |
<jenkins.version>2.60.3</jenkins.version> |
If you want to support Java 8 only, we need to use the core version which supports Java 8 only
@@ -0,0 +1,10 @@ | |||
<!-- FindBugs exclude file --> | |||
<!-- This is just a (currently unused, as-yet invalid) template. If you want |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hm? Do you need it then?
// JENKINS-7999 is fixed, so now use Jenkins FilePath instead of the | ||
// custom MyFilePath included in this plugin | ||
rootFilePathOnMaster.copyRecursiveTo(includes, excludes, | ||
projectWorkspaceOnSlave); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This removes silently the flatten
feature from https://issues.jenkins-ci.org/browse/JENKINS-8220
|
||
<scm> | ||
<connection>scm:git:ssh://github.com:jmcgeheeiv/copy-to-slave-plugin.git</connection> | ||
<developerConnection>scm:git:ssh://[email protected]/jmcgeheeiv/copy-to-slave-plugin.git</developerConnection> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we want this change in the published pom.xml?
|
||
<issueManagement> | ||
<url>https://github.com/jenkinsci/copy-to-slave-plugin/issues</url> | ||
<system>GitHub</system> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Issues for this plugin are currently managed on the Jenkins JIRA, not on GitHub; in fact, GitHub issues are disabled as of 2020-05-03
<organization> | ||
<name>Wave Computing, Inc.</name> | ||
<url>http://www.wavecomp.ai</url> | ||
</organization> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is Wave Computing now supporting the plugin...?
This PR contains mainly a modernized build:
MyFilePath
with the standardFilePath
. Original author Romain Seguy usedMyFilePath
to work around some problems he found in the standardFilePath
. Then he fixed the standardFilePath
per JENKINS-7999.IMPORTANTLY, THE SECURITY PROBLEM WITH THIS PLUGIN REMAINS.