Skip to content

Commit

Permalink
[FIXED JENKINS-17668] Windows phing is broken, jenkins tries to execu…
Browse files Browse the repository at this point in the history
…te phing.bat with php.exe
  • Loading branch information
ssogabe committed Apr 20, 2013
1 parent aefdb9c commit ffb569e
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/main/java/hudson/plugins/phing/PhingBuilder.java
Expand Up @@ -161,7 +161,9 @@ public boolean perform(AbstractBuild<?, ?> build, Launcher launcher, BuildListen
}

// PHP Command
args.add(computePhpCommand(pi, env));
if (launcher.isUnix()) {
args.add(computePhpCommand(pi, env));
}
// Phing Command
args.add(computePhingCommand(pi, launcher));

Expand Down

1 comment on commit ffb569e

@schill32
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am seeing a problem to where if my phing.bat folder is in my path environment window it will cause a problem. In the system configuration I have updated the global properties to supply the path for the home field (c:\php in my case). And have also updated the Phing section field of Phing HOME with c:\php. Is there another section that I should be updating?

Please sign in to comment.