Looks good, thanks Tim!

/Erik

On 2016-04-01 20:41, Tim Bell wrote:
Please review this build fix. One ',' was omitted in the JDK-8153257 changes.

Bug report:
   https://bugs.openjdk.java.net/browse/JDK-8153303

Webrev:
   http://cr.openjdk.java.net/~tbell/8153303/webrev.00/index.html

This is small enough that I am including it here inline:

% hg diff common/conf/jib-profiles.js
diff --git a/common/conf/jib-profiles.js b/common/conf/jib-profiles.js
--- a/common/conf/jib-profiles.js
+++ b/common/conf/jib-profiles.js
@@ -250,7 +250,7 @@
             target_cpu: "x86",
             build_cpu: "x64",
             dependencies: concat(common.dependencies, "devkit"),
- configure_args: concat(common.configure_args, common.configure_args_32bit + configure_args: concat(common.configure_args, common.configure_args_32bit,
                 "--with-zlib=system"),
             make_args: common.make_args
         },


Thanks in advance-

Tim

Reply via email to