[PATCH] Add post custom extension hooks to two launchers

Erik Joelsson erik.joelsson at oracle.com
Fri Feb 16 19:22:35 UTC 2018


This looks ok. I can sponsor it.

/Erik


On 2018-02-15 03:32, Ben Walsh wrote:
> This patch simply adds a hook to two launchers to call a post custom
> extension
> if it is available.
>
> I would like to pair with a sponsor to contribute this patch ...
>
> ------------------------------------------------------------------------------------
>
> diff -r 85c27aabf312 make/launcher/Launcher-jdk.jcmd.gmk
> --- a/make/launcher/Launcher-jdk.jcmd.gmk       Thu Feb 08 10:06:57 2018
> -0500
> +++ b/make/launcher/Launcher-jdk.jcmd.gmk       Thu Feb 15 11:16:28 2018
> +0000
> @@ -1,5 +1,5 @@
>   #
> -# Copyright (c) 2011, 2015, Oracle and/or its affiliates. All rights
> reserved.
> +# Copyright (c) 2011, 2018, Oracle and/or its affiliates. All rights
> reserved.
>   # DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
>   #
>   # This code is free software; you can redistribute it and/or modify it
> @@ -60,3 +60,6 @@
>   $(eval $(call SetupBuildLauncher, jcmd, \
>       MAIN_CLASS := sun.tools.jcmd.JCmd, \
>   ))
> +
> +# Hook to include the corresponding custom file, if present.
> +$(eval $(call IncludeCustomExtension,
> launcher/Launcher-jdk.jcmd-post.gmk))
> diff -r 85c27aabf312 make/launcher/Launcher-jdk.jstatd.gmk
> --- a/make/launcher/Launcher-jdk.jstatd.gmk     Thu Feb 08 10:06:57 2018
> -0500
> +++ b/make/launcher/Launcher-jdk.jstatd.gmk     Thu Feb 15 11:16:28 2018
> +0000
> @@ -1,5 +1,5 @@
>   #
> -# Copyright (c) 2011, 2016, Oracle and/or its affiliates. All rights
> reserved.
> +# Copyright (c) 2011, 2018, Oracle and/or its affiliates. All rights
> reserved.
>   # DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
>   #
>   # This code is free software; you can redistribute it and/or modify it
> @@ -28,3 +28,6 @@
>   $(eval $(call SetupBuildLauncher, jstatd, \
>       MAIN_CLASS := sun.tools.jstatd.Jstatd, \
>   ))
> +
> +# Hook to include the corresponding custom file, if present.
> +$(eval $(call IncludeCustomExtension,
> launcher/Launcher-jdk.jstatd-post.gmk))
>
> ------------------------------------------------------------------------------------
>
> Regards,
> Ben Walsh
>
>
> Unless stated otherwise above:
> IBM United Kingdom Limited - Registered in England and Wales with number
> 741598.
> Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU
>



More information about the build-dev mailing list