Address Ivy-plugin issue jenkinsci#4 - don't ignore explicit revision…

…s for matching
noise · May 1, 2012 · d6177ae · d6177ae
1 parent 781670a
commit d6177ae
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/README-fork.markdown b/README-fork.markdown
@@ -0,0 +1,4 @@
+This fork is for the sole purpose of overriding the behavior in IvyBuildTrigger.java that ignores dependencies w/fixed (i.e. non-dynamic) revisions. 
+
+See: https://github.com/jenkinsci/ivy-plugin/issues/4
+
diff --git a/src/main/java/hudson/ivy/IvyBuildTrigger.java b/src/main/java/hudson/ivy/IvyBuildTrigger.java
@@ -491,7 +491,7 @@ private boolean captures(ModuleRevisionId depRevId, Project candidate) {
             VersionMatcher matcher = versionMatcher;
 
             // A null VersionMatcher means something is really wrong and we should not proceed to add the dependency
-            captures = (matcher != null && matcher.isDynamic(depRevId));
+            captures = matcher != null;
             if (captures) {
                 String dbranch = depRevId.getBranch();
                 String cbranch = cmrid.getBranch();
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,4 @@
		This fork is for the sole purpose of overriding the behavior in IvyBuildTrigger.java that ignores dependencies w/fixed (i.e. non-dynamic) revisions.

		See: https://github.com/jenkinsci/ivy-plugin/issues/4