I updated my original pull-request to fix only this.

https://github.com/jenkinsci/git-plugin/pull/293

Hi Jean

I had some issues and was wondering about other scenarios that could be affected by this.

Like if you wanted to build one branch (with polling) and the actual branch parameter used for checking was another branch (like master) so that all changes to master triggered rebuilds..when you just wanted that on a parameterized branch.

would see as detected changes when there weren't any on the desired branch?

just wondering.

-kevin

Code changed in jenkins
User: Jean Blanchard
Path:
src/main/java/hudson/plugins/git/GitSCM.java
src/main/java/hudson/plugins/git/util/GitUtils.java
src/test/java/hudson/plugins/git/GitSCMTest.java
http://jenkins-ci.org/commit/git-plugin/d56fbee601d5c46d35fc5c05601ead12d66fba10
Log:
JENKINS-27349 Polling for parametrized branchspec should resolve to default param values

Code changed in jenkins
User: Nicolas De loof
Path:
src/main/java/hudson/plugins/git/GitSCM.java
src/main/java/hudson/plugins/git/util/GitUtils.java
src/test/java/hudson/plugins/git/GitSCMTest.java
http://jenkins-ci.org/commit/git-plugin/30766fe249c2512872c52ad96ceab1356510eba2
Log:
Merge pull request #293 from jeanblanchard/fix-notify

JENKINS-27349 Fix polling for parametrized branchspec

Compare: https://github.com/jenkinsci/git-plugin/compare/7af820e61615...30766fe249c2

Verified it is working as intended in the 2.3.6 SNAPSHOT build from 6 May 2015. Should be included in the 2.3.6 final build when that is released.

The git plugin and git client plugin are being tested in hopes of releasing new versions before the end of June. If you're willing to assist with the testing, please download and install a pre-release build of the git client plugin and the git plugin. Problems detected in the pre-release should be e-mailed to MarkEWaite and ndeloof.

I wrote some test ideas if you would like suggestions of areas that need testing. The git plugin supports many different use cases and its automated tests only evaluate a very few of those use cases.

Is there any way this pre-release could be updated to include the fix for JENKINS-27352? I installed the pre-release last week, but had to back it out. We're back to running a patched version that includes all of Jean's fixes.