Skip to content

Commit

Permalink
[FIXED JENKINS-42775] Fix check for 32 bit installer
Browse files Browse the repository at this point in the history
  • Loading branch information
nfalco79 committed Mar 15, 2017
1 parent 976e100 commit d692ef3
Show file tree
Hide file tree
Showing 2 changed files with 438 additions and 306 deletions.
Expand Up @@ -59,7 +59,7 @@ public String resolvePathFor(String version, Platform platform, CPU cpu) {

switch (cpu) {
case i386:
if (NodeJSVersion.parseVersion(version).compareTo(new NodeJSVersion(4, 0, 0)) >= 0) {
if (platform == Platform.OSX && NodeJSVersion.parseVersion(version).compareTo(new NodeJSVersion(4, 0, 0)) >= 0) {
throw new IllegalArgumentException("Unresolvable nodeJS installer for version=" + version + ", cpu=" + cpu.name() + ", platform=" + platform.name());
}
arch = "x86";
Expand Down

0 comments on commit d692ef3

Please sign in to comment.