Looks fine. Mandy
> On Apr 1, 2016, at 11:41 AM, Tim Bell <tim.b...@oracle.com> 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
