JDK 9 RFR of JDK-8067099: Add deprecation lint warning to build of jdk repository

Erik Joelsson erik.joelsson at oracle.com
Wed Dec 10 06:39:57 UTC 2014


First, congratulations on almost being done fixing all the warnings!

When removing the last warning exception, perhaps it's time to change 
the comment next to this line too?

/Erik

On 2014-12-10 02:59, joe darcy wrote:
> Hello,
>
> The time approaches when the one remaining category of lint warning 
> still present in the jdk repo, deprecation, will be eliminated. In 
> anticipation of that happy day, please review this simple patch to 
> enable the warning in the build:
>
> diff -r f7c11da0b048 make/common/SetupJavaCompilers.gmk
> --- a/make/common/SetupJavaCompilers.gmk    Thu Dec 04 15:21:09 2014 
> -0800
> +++ b/make/common/SetupJavaCompilers.gmk    Tue Dec 09 17:30:32 2014 
> -0800
> @@ -32,7 +32,7 @@
>
>  # To build with all warnings enabled, do the following:
>  # make JAVAC_WARNINGS="-Xlint:all -Xmaxwarns 10000"
> -JAVAC_WARNINGS := -Xlint:all,-deprecation -Werror
> +JAVAC_WARNINGS := -Xlint:all -Werror
>
>  # The BOOT_JAVAC setup uses the boot jdk compiler to compile the tools
>  # and the interim javac, to be run by the boot jdk.
>
> As usual for this kind of change,  a jprt job will be used to verify 
> the fix is valid on all platforms, etc.
>
> Thanks,
>
> -Joe




More information about the build-dev mailing list