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


Reply via email to