RFR: JDK-8004842 Unify values of boolean make variables set in configure to true/false
David Holmes
david.holmes at oracle.com
Mon Feb 6 03:47:25 UTC 2017
Hi Magnus,
On 3/02/2017 11:14 PM, Magnus Ihse Bursie wrote:
> Currently there are three different ways of expressing a boolean value
> in spec.gmk, true/false, yes/no and 1/0. We should only have one and
> since true/false is the most common one, we should stick to that.
>
> Since this bug was opened, most cases has been fixed but two remains:
> USE_PRECOMPILED_HEADER=1
One nit:
hotspot/src/share/vm/precompiled/precompiled.hpp
I don't think we normally document configure args in the source code,
and I would expect that we can still pass
USE_PRECOMPILED_HEADERS=true/false as a make arg. So I think this should
also just change 0 to false.
Thanks,
David
> ENABLE_INTREE_EC=yes
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8004842
> WebRev:
> http://cr.openjdk.java.net/~ihse/JDK-8004842-standardize-booleans-in-build/webrev.01
>
>
> /Magnus
>
More information about the build-dev
mailing list