Skip to content

Commit

Permalink
JENKINS-10831 maven submodule build fails doing mkdir on master.
Browse files Browse the repository at this point in the history
Remove the shadow copy of buildOn from MavenBuild.java and move the setter to
AbstractBuild to allow MavenModuleSetBuild.java to continue to set the
build node.

Originally-Committed-As: 67262e455f818bf693d1bd7a5987c9268b29c52e
  • Loading branch information
oldelvet authored and kohsuke committed Aug 28, 2011
1 parent 0aea1c0 commit 61cac17
Showing 1 changed file with 0 additions and 27 deletions.
27 changes: 0 additions & 27 deletions src/main/java/hudson/maven/MavenBuild.java
Expand Up @@ -95,13 +95,6 @@ public class MavenBuild extends AbstractMavenBuild<MavenModule,MavenBuild> {
* @since 1.98.
*/
private List<ExecutedMojo> executedMojos;

/**
* Name of the slave this project was built on.
* Null or "" if built by the master. (null happens when we read old record that didn't have this information.)
* @since 1.394
*/
private String builtOn;

public MavenBuild(MavenModule job) throws IOException {
super(job);
Expand Down Expand Up @@ -293,26 +286,6 @@ public MavenModule getParent() {// don't know why, but javac wants this
return super.getParent();
}

/**
* @see hudson.model.AbstractBuild#getBuiltOn()
* @since 1.394
*/
public Node getBuiltOn() {
if(builtOn==null || builtOn.equals(""))
return Jenkins.getInstance();
else
return Jenkins.getInstance().getNode(builtOn);
}

/**
* @param builtOn
* @since 1.394
*/
public void setBuiltOnStr( String builtOn )
{
this.builtOn = builtOn;
}

/**
* Runs Maven and builds the project.
*/
Expand Down

0 comments on commit 61cac17

Please sign in to comment.