Branch: refs/heads/master
  Home:   https://github.com/jenkinsci/ant-plugin
  Commit: c57e18f6ac9c01ccce927930aed0221853d38ff6
      
https://github.com/jenkinsci/ant-plugin/commit/c57e18f6ac9c01ccce927930aed0221853d38ff6
  Author: Matt Doran <matt.do...@papercut.com>
  Date:   2017-02-07 (Tue, 07 Feb 2017)

  Changed paths:
    M src/main/java/hudson/tasks/Ant.java
    M src/test/java/hudson/tasks/AntTest.java

  Log Message:
  -----------
  JENKINS-41801: Allow empty build parameters to be replaced in ant properties.

By using EnvVars.overrideAll() to add all build variable to EnvVars, any blank 
("") build parameters are not added (or if they existed would be removed) from 
EnvVars.    This means any reference to a blank variable is not replaced 
instead of being set to empty.

The tests check the log line where the ant command line is printed.  It's 
needed to check here for reliable tests, because on unix the $parameter will be 
replaced by the shell, but not in windows.   By testing the command line params 
we ensure the replacements occur before going to the OS to run ant.

The tests try to be resilient to the command line and param formatting on 
windows.


  Commit: 7025e65f06ee3934db303960a74846cc52440a26
      
https://github.com/jenkinsci/ant-plugin/commit/7025e65f06ee3934db303960a74846cc52440a26
  Author: Armando Fernández <armfer...@gmail.com>
  Date:   2017-05-09 (Tue, 09 May 2017)

  Changed paths:
    M src/main/java/hudson/tasks/Ant.java
    M src/test/java/hudson/tasks/AntTest.java

  Log Message:
  -----------
  Merge pull request #25 from mattdoran/master

JENKINS-41801: Allow empty build parameters to be replaced in ant properties


Compare: 
https://github.com/jenkinsci/ant-plugin/compare/d1c5e13f88ac...7025e65f06ee

-- 
You received this message because you are subscribed to the Google Groups 
"Jenkins Commits" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to jenkinsci-commits+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to