Fix to langtools/make/launcher.sh-template for the new -Xpatch
Jonathan Gibbons
jonathan.gibbons at oracle.com
Fri Apr 29 23:43:43 UTC 2016
Wow, someone uses that; thanks for the update.
-- Jon
On 04/29/2016 04:27 PM, mark.reinhold at oracle.com wrote:
> If this looks okay to you then I'll push it.
>
> (It also enables -ea for jdk.javadoc code, which I suspect was always
> the intent.)
>
> Thanks,
> - Mark
>
>
> # HG changeset patch
> # Parent 84d1306b39efaebe4dbeff2b9c84fc7ec4189556
> Fix launcher script for new -Xpatch
>
> diff --git a/make/launcher.sh-template b/make/launcher.sh-template
> --- a/make/launcher.sh-template
> +++ b/make/launcher.sh-template
> @@ -33,11 +33,8 @@
> esac
> mylib="$mydir/../modules"
>
> -# patch langtools modules
> -bcp=-Xpatch:"$mylib"
> -
> # tools currently assumes that assertions are enabled in the launcher
> -ea=-ea:com.sun.tools...
> +ea=-ea:com.sun.tools...:jdk.javadoc...
>
> # Any parameters starting with -J are passed to the JVM.
> # All other parameters become parameters of #PROGRAM#.
> @@ -58,5 +55,10 @@
> done
> unset DUALCASE
>
> +# patch langtools modules
> +mods='java.compiler jdk.compiler jdk.javadoc jdk.jdeps jdk.jshell'
> +patches=
> +for m in $mods; do patches="$patches$nl-Xpatch:$m=$mylib/$m"; done
> +
> IFS=$nl
> -"#TARGET_JAVA#" $bcp ${ea} ${javaOpts} #PROGRAM# ${toolOpts}
> +"#TARGET_JAVA#" ${patches} ${ea} ${javaOpts} #PROGRAM# ${toolOpts}
More information about the jigsaw-dev
mailing list