From 95ad2904b3309003349ff476627906a6896cb1d2 Mon Sep 17 00:00:00 2001 From: CentOS Sources Date: Jul 10 2020 01:31:12 +0000 Subject: import jmc-7.1.1-5.module+el8.2.1+6408+a7ca4662 --- diff --git a/.gitignore b/.gitignore new file mode 100644 index 0000000..097bf6c --- /dev/null +++ b/.gitignore @@ -0,0 +1 @@ +SOURCES/e67446b5fc9d.tar.gz diff --git a/.jmc.metadata b/.jmc.metadata new file mode 100644 index 0000000..6908e7e --- /dev/null +++ b/.jmc.metadata @@ -0,0 +1 @@ +676ac85e3a33296288129c09e1b898a9d460d36f SOURCES/e67446b5fc9d.tar.gz diff --git a/SOURCES/0-remove-twitter.patch b/SOURCES/0-remove-twitter.patch new file mode 100644 index 0000000..f7c6839 --- /dev/null +++ b/SOURCES/0-remove-twitter.patch @@ -0,0 +1,225 @@ +diff --git a/application/org.openjdk.jmc.rjmx/META-INF/MANIFEST.MF b/application/org.openjdk.jmc.rjmx/META-INF/MANIFEST.MF +--- a/application/org.openjdk.jmc.rjmx/META-INF/MANIFEST.MF ++++ b/application/org.openjdk.jmc.rjmx/META-INF/MANIFEST.MF +@@ -54,7 +54,6 @@ + org.openjdk.jmc.rjmx.triggers; + x-friends:="org.openjdk.jmc.console.ui.notification, + org.openjdk.jmc.alert, +- org.openjdk.jmc.console.twitter, + org.openjdk.jmc.rjmx.test", + org.openjdk.jmc.rjmx.triggers.actions.internal;x-friends:="org.openjdk.jmc.console.ui.notification", + org.openjdk.jmc.rjmx.triggers.condition.internal;x-friends:="org.openjdk.jmc.console.ui.notification,org.openjdk.jmc.test.jemmy", +diff --git a/application/pom.xml b/application/pom.xml +--- a/application/pom.xml ++++ b/application/pom.xml +@@ -50,7 +50,6 @@ + org.openjdk.jmc.console.jconsole + org.openjdk.jmc.console.pde + org.openjdk.jmc.console.persistence +- org.openjdk.jmc.console.twitter + org.openjdk.jmc.console.ui + org.openjdk.jmc.console.ui.diagnostic + org.openjdk.jmc.console.ui.mbeanbrowser +@@ -72,7 +71,6 @@ + org.openjdk.jmc.feature.pde + org.openjdk.jmc.feature.rcp + org.openjdk.jmc.feature.rcp.update +- org.openjdk.jmc.feature.twitter + org.openjdk.jmc.flightrecorder.configuration + org.openjdk.jmc.flightrecorder.controlpanel.ui + org.openjdk.jmc.flightrecorder.controlpanel.ui.configuration +diff --git a/license/THIRDPARTYREADME.txt b/license/THIRDPARTYREADME.txt +--- a/license/THIRDPARTYREADME.txt ++++ b/license/THIRDPARTYREADME.txt +@@ -296,191 +296,6 @@ + ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED + OF THE POSSIBILITY OF SUCH DAMAGE. + +- +-%% The following notice is provided with respect to Twitter4J 4.0.7, +-which may be included with this product. +- +- +- Apache License +- Version 2.0, January 2004 +- http://www.apache.org/licenses/ +- +- TERMS AND CONDITIONS FOR USE, REPRODUCTION, AND DISTRIBUTION +- +- 1. Definitions. +- +- "License" shall mean the terms and conditions for use, reproduction, +- and distribution as defined by Sections 1 through 9 of this document. +- +- "Licensor" shall mean the copyright owner or entity authorized by +- the copyright owner that is granting the License. +- +- "Legal Entity" shall mean the union of the acting entity and all +- other entities that control, are controlled by, or are under common +- control with that entity. For the purposes of this definition, +- "control" means (i) the power, direct or indirect, to cause the +- direction or management of such entity, whether by contract or +- otherwise, or (ii) ownership of fifty percent (50%) or more of the +- outstanding shares, or (iii) beneficial ownership of such entity. +- +- "You" (or "Your") shall mean an individual or Legal Entity +- exercising permissions granted by this License. +- +- "Source" form shall mean the preferred form for making modifications, +- including but not limited to software source code, documentation +- source, and configuration files. +- +- "Object" form shall mean any form resulting from mechanical +- transformation or translation of a Source form, including but +- not limited to compiled object code, generated documentation, +- and conversions to other media types. +- +- "Work" shall mean the work of authorship, whether in Source or +- Object form, made available under the License, as indicated by a +- copyright notice that is included in or attached to the work +- (an example is provided in the Appendix below). +- +- "Derivative Works" shall mean any work, whether in Source or Object +- form, that is based on (or derived from) the Work and for which the +- editorial revisions, annotations, elaborations, or other modifications +- represent, as a whole, an original work of authorship. For the purposes +- of this License, Derivative Works shall not include works that remain +- separable from, or merely link (or bind by name) to the interfaces of, +- the Work and Derivative Works thereof. +- +- "Contribution" shall mean any work of authorship, including +- the original version of the Work and any modifications or additions +- to that Work or Derivative Works thereof, that is intentionally +- submitted to Licensor for inclusion in the Work by the copyright owner +- or by an individual or Legal Entity authorized to submit on behalf of +- the copyright owner. For the purposes of this definition, "submitted" +- means any form of electronic, verbal, or written communication sent +- to the Licensor or its representatives, including but not limited to +- communication on electronic mailing lists, source code control systems, +- and issue tracking systems that are managed by, or on behalf of, the +- Licensor for the purpose of discussing and improving the Work, but +- excluding communication that is conspicuously marked or otherwise +- designated in writing by the copyright owner as "Not a Contribution." +- +- "Contributor" shall mean Licensor and any individual or Legal Entity +- on behalf of whom a Contribution has been received by Licensor and +- subsequently incorporated within the Work. +- +- 2. Grant of Copyright License. Subject to the terms and conditions of +- this License, each Contributor hereby grants to You a perpetual, +- worldwide, non-exclusive, no-charge, royalty-free, irrevocable +- copyright license to reproduce, prepare Derivative Works of, +- publicly display, publicly perform, sublicense, and distribute the +- Work and such Derivative Works in Source or Object form. +- +- 3. Grant of Patent License. Subject to the terms and conditions of +- this License, each Contributor hereby grants to You a perpetual, +- worldwide, non-exclusive, no-charge, royalty-free, irrevocable +- (except as stated in this section) patent license to make, have made, +- use, offer to sell, sell, import, and otherwise transfer the Work, +- where such license applies only to those patent claims licensable +- by such Contributor that are necessarily infringed by their +- Contribution(s) alone or by combination of their Contribution(s) +- with the Work to which such Contribution(s) was submitted. If You +- institute patent litigation against any entity (including a +- cross-claim or counterclaim in a lawsuit) alleging that the Work +- or a Contribution incorporated within the Work constitutes direct +- or contributory patent infringement, then any patent licenses +- granted to You under this License for that Work shall terminate +- as of the date such litigation is filed. +- +- 4. Redistribution. You may reproduce and distribute copies of the +- Work or Derivative Works thereof in any medium, with or without +- modifications, and in Source or Object form, provided that You +- meet the following conditions: +- +- (a) You must give any other recipients of the Work or +- Derivative Works a copy of this License; and +- +- (b) You must cause any modified files to carry prominent notices +- stating that You changed the files; and +- +- (c) You must retain, in the Source form of any Derivative Works +- that You distribute, all copyright, patent, trademark, and +- attribution notices from the Source form of the Work, +- excluding those notices that do not pertain to any part of +- the Derivative Works; and +- +- (d) If the Work includes a "NOTICE" text file as part of its +- distribution, then any Derivative Works that You distribute must +- include a readable copy of the attribution notices contained +- within such NOTICE file, excluding those notices that do not +- pertain to any part of the Derivative Works, in at least one +- of the following places: within a NOTICE text file distributed +- as part of the Derivative Works; within the Source form or +- documentation, if provided along with the Derivative Works; or, +- within a display generated by the Derivative Works, if and +- wherever such third-party notices normally appear. The contents +- of the NOTICE file are for informational purposes only and +- do not modify the License. You may add Your own attribution +- notices within Derivative Works that You distribute, alongside +- or as an addendum to the NOTICE text from the Work, provided +- that such additional attribution notices cannot be construed +- as modifying the License. +- +- You may add Your own copyright statement to Your modifications and +- may provide additional or different license terms and conditions +- for use, reproduction, or distribution of Your modifications, or +- for any such Derivative Works as a whole, provided Your use, +- reproduction, and distribution of the Work otherwise complies with +- the conditions stated in this License. +- +- 5. Submission of Contributions. Unless You explicitly state otherwise, +- any Contribution intentionally submitted for inclusion in the Work +- by You to the Licensor shall be under the terms and conditions of +- this License, without any additional terms or conditions. +- Notwithstanding the above, nothing herein shall supersede or modify +- the terms of any separate license agreement you may have executed +- with Licensor regarding such Contributions. +- +- 6. Trademarks. This License does not grant permission to use the trade +- names, trademarks, service marks, or product names of the Licensor, +- except as required for reasonable and customary use in describing the +- origin of the Work and reproducing the content of the NOTICE file. +- +- 7. Disclaimer of Warranty. Unless required by applicable law or +- agreed to in writing, Licensor provides the Work (and each +- Contributor provides its Contributions) on an "AS IS" BASIS, +- WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or +- implied, including, without limitation, any warranties or conditions +- of TITLE, NON-INFRINGEMENT, MERCHANTABILITY, or FITNESS FOR A +- PARTICULAR PURPOSE. You are solely responsible for determining the +- appropriateness of using or redistributing the Work and assume any +- risks associated with Your exercise of permissions under this License. +- +- 8. Limitation of Liability. In no event and under no legal theory, +- whether in tort (including negligence), contract, or otherwise, +- unless required by applicable law (such as deliberate and grossly +- negligent acts) or agreed to in writing, shall any Contributor be +- liable to You for damages, including any direct, indirect, special, +- incidental, or consequential damages of any character arising as a +- result of this License or out of the use or inability to use the +- Work (including but not limited to damages for loss of goodwill, +- work stoppage, computer failure or malfunction, or any and all +- other commercial damages or losses), even if such Contributor +- has been advised of the possibility of such damages. +- +- 9. Accepting Warranty or Additional Liability. While redistributing +- the Work or Derivative Works thereof, You may choose to offer, +- and charge a fee for, acceptance of support, warranty, indemnity, +- or other liability obligations and/or rights consistent with this +- License. However, in accepting such obligations, You may act only +- on Your own behalf and on Your sole responsibility, not on behalf +- of any other Contributor, and only if You agree to indemnify, +- defend, and hold each Contributor harmless for any liability +- incurred by, or claims asserted against, such Contributor by reason +- of your accepting any such warranty or additional liability. +- +-Twitter4J SUBCOMPONENTS: +- +-Twitter4J includes software from JSON.org to parse JSON response from the Twitter API. You can see the license term at http://www.JSON.org/license.html +- +- + %% The following notice is provided with respect to JavaBeans Activation + Framework (JAF) 1.2.1, which may be included with this product. + diff --git a/SOURCES/1-fix-javamail.patch b/SOURCES/1-fix-javamail.patch new file mode 100644 index 0000000..476a274 --- /dev/null +++ b/SOURCES/1-fix-javamail.patch @@ -0,0 +1,29 @@ +# HG changeset patch +# Parent 9696af3e434734700a8c37680ce3742ab940eccb + +diff --git a/application/org.openjdk.jmc.feature.core/feature.xml b/application/org.openjdk.jmc.feature.core/feature.xml +--- a/application/org.openjdk.jmc.feature.core/feature.xml ++++ b/application/org.openjdk.jmc.feature.core/feature.xml +@@ -161,13 +161,6 @@ + unpack="false"/> + + +- +- + + + + +- + + + +diff -r 37a493c3f60b -r bbf0648beb5c application/org.openjdk.jmc.flightrecorder.flameview/build.properties +--- a/application/org.openjdk.jmc.flightrecorder.flameview/build.properties Tue Nov 12 16:25:24 2019 +0100 ++++ b/application/org.openjdk.jmc.flightrecorder.flameview/build.properties Mon Nov 25 17:57:23 2019 -0500 +@@ -1,5 +1,4 @@ +-source.. = src/main/java/,\ +- src/main/resources/ ++source.. = src/main/java/ + output.. = target/classes/ + bin.includes = META-INF/,\ + .,\ +diff -r 37a493c3f60b -r bbf0648beb5c application/org.openjdk.jmc.flightrecorder.flameview/pom.xml +--- a/application/org.openjdk.jmc.flightrecorder.flameview/pom.xml Tue Nov 12 16:25:24 2019 +0100 ++++ b/application/org.openjdk.jmc.flightrecorder.flameview/pom.xml Mon Nov 25 17:57:23 2019 -0500 +@@ -41,94 +41,4 @@ + + org.openjdk.jmc.flightrecorder.flameview + eclipse-plugin +- +- +- 1.4.2 +- process-resources +- ${project.basedir}/src/main/resources/jslibs +- +- +- +- +- +- com.googlecode.maven-download-plugin +- download-maven-plugin +- ${download.maven.plugin.version} +- +- +- d3-flamegraph-css +- ${download-maven-plugin.phase} +- +- wget +- +- +- https://cdn.jsdelivr.net/gh/spiermar/d3-flame-graph@2.0.3/dist/d3-flamegraph.css +- false +- ${download-maven-plugin.output} +- +- +- +- ie-html5shiv-js +- ${download-maven-plugin.phase} +- +- wget +- +- +- https://oss.maxcdn.com/html5shiv/3.7.2/html5shiv.min.js +- false +- ${download-maven-plugin.output} +- +- +- +- ie-respond-js +- ${download-maven-plugin.phase} +- +- wget +- +- +- https://oss.maxcdn.com/respond/1.4.2/respond.min.js +- false +- ${download-maven-plugin.output} +- +- +- +- d3-v4-js +- ${download-maven-plugin.phase} +- +- wget +- +- +- https://d3js.org/d3.v4.min.js +- false +- ${download-maven-plugin.output} +- +- +- +- d3-tip-js +- ${download-maven-plugin.phase} +- +- wget +- +- +- https://cdnjs.cloudflare.com/ajax/libs/d3-tip/0.9.1/d3-tip.min.js +- false +- ${download-maven-plugin.output} +- +- +- +- d3-flamegraph-js +- ${download-maven-plugin.phase} +- +- wget +- +- +- https://cdn.jsdelivr.net/gh/spiermar/d3-flame-graph@2.0.3/dist/d3-flamegraph.min.js +- false +- ${download-maven-plugin.output} +- +- +- +- +- +- + +diff -r 37a493c3f60b -r bbf0648beb5c application/org.openjdk.jmc.flightrecorder.flameview/src/main/java/org/openjdk/jmc/flightrecorder/flameview/views/FlameGraphView.java +--- a/application/org.openjdk.jmc.flightrecorder.flameview/src/main/java/org/openjdk/jmc/flightrecorder/flameview/views/FlameGraphView.java Tue Nov 12 16:25:24 2019 +0100 ++++ b/application/org.openjdk.jmc.flightrecorder.flameview/src/main/java/org/openjdk/jmc/flightrecorder/flameview/views/FlameGraphView.java Mon Nov 25 17:57:23 2019 -0500 +@@ -34,14 +34,11 @@ + package org.openjdk.jmc.flightrecorder.flameview.views; + + import java.io.IOException; +- +-import java.text.MessageFormat; + import java.util.concurrent.CancellationException; + import java.util.concurrent.CompletableFuture; + import java.util.concurrent.ExecutorService; + import java.util.concurrent.Executors; + import java.util.logging.Level; +-import java.util.stream.Collectors; + import java.util.stream.Stream; + + import org.eclipse.jface.action.Action; +@@ -77,32 +74,7 @@ + import org.openjdk.jmc.ui.misc.DisplayToolkit; + + public class FlameGraphView extends ViewPart implements ISelectionListener { +- private static final String HTML_PAGE; +- static { +- // from: https://cdn.jsdelivr.net/gh/spiermar/d3-flame-graph@2.0.3/dist/d3-flamegraph.css +- String cssD3Flamegraph = "jslibs/d3-flamegraph.css"; +- // from: https://oss.maxcdn.com/html5shiv/3.7.2/html5shiv.min.js +- String jsHtml5shiv = "jslibs/html5shiv.min.js"; +- // from: https://oss.maxcdn.com/respond/1.4.2/respond.min.js +- String jsRespond = "jslibs/respond.min.js"; +- // from: https://d3js.org/d3.v4.min.js +- String jsD3V4 = "jslibs/d3.v4.min.js"; +- // from: https://cdnjs.cloudflare.com/ajax/libs/d3-tip/0.9.1/d3-tip.min.js +- String jsD3Tip = "jslibs/d3-tip.min.js"; +- // from: https://cdn.jsdelivr.net/gh/spiermar/d3-flame-graph@2.0.3/dist/d3-flamegraph.min.js +- String jsD3FlameGraph = "jslibs/d3-flamegraph.min.js"; +- +- String jsIeLibraries = loadLibraries(jsHtml5shiv, jsRespond); +- String jsD3Libraries = loadLibraries(jsD3V4, jsD3Tip, jsD3FlameGraph); +- +- // formatter arguments for the template: %1 - CSSs, %2 - IE9 specific scripts, %3 - 3rd party scripts +- HTML_PAGE = String.format(fileContent("page.template"), +- fileContent(cssD3Flamegraph), +- jsIeLibraries, +- jsD3Libraries); +- } +- +- private static final ExecutorService MODEL_EXECUTOR = Executors.newFixedThreadPool(1); ++ private static ExecutorService MODEL_EXECUTOR = Executors.newFixedThreadPool(1); + private FrameSeparator frameSeparator; + + private Browser browser; +@@ -207,6 +179,13 @@ + }, MODEL_EXECUTOR); + } + ++ private static Void handleModelBuildException(Throwable ex) { ++ if (!(ex.getCause() instanceof CancellationException)) { ++ FlightRecorderUI.getDefault().getLogger().log(Level.SEVERE, "Failed to build stacktrace view model", ex); //$NON-NLS-1$ ++ } ++ return null; ++ } ++ + private void setModel(TraceNode root) { + if (!browser.isDisposed() && !root.equals(currentRoot)) { + currentRoot = root; +@@ -215,21 +194,19 @@ + } + + private void setViewerInput(TraceNode root) { +- browser.setText(HTML_PAGE); +- browser.addProgressListener(new ProgressAdapter() { +- @Override +- public void completed(ProgressEvent event) { +- browser.removeProgressListener(this); +- browser.execute(String.format("processGraph(%s);", toJSon(root))); +- } +- }); +- } +- +- private static Void handleModelBuildException(Throwable ex) { +- if (!(ex.getCause() instanceof CancellationException)) { +- FlightRecorderUI.getDefault().getLogger().log(Level.SEVERE, "Failed to build stacktrace view model", ex); //$NON-NLS-1$ ++ try { ++ browser.setText(StringToolkit.readString(FlameGraphView.class.getResourceAsStream("page.html"))); ++ browser.addProgressListener(new ProgressAdapter() { ++ @Override ++ public void completed(ProgressEvent event) { ++ browser.removeProgressListener(this); ++ browser.execute(String.format("processGraph(%s);", toJSon(root))); ++ } ++ }); ++ } catch (IOException e) { ++ browser.setText(e.getMessage()); ++ e.printStackTrace(); + } +- return null; + } + + private static String toJSon(TraceNode root) { +@@ -261,23 +238,4 @@ + private static String toJSonKeyValue(String key, String value) { + return "\"" + key + "\": " + "\"" + value + "\""; + } +- +- private static String loadLibraries(String... libs) { +- if(libs == null || libs.length == 0) { +- return ""; +- } else { +- return Stream.of(libs).map(FlameGraphView::fileContent).collect(Collectors.joining("\n")); +- } +- } +- +- private static String fileContent(String fileName){ +- try { +- return StringToolkit.readString(FlameGraphView.class.getClassLoader().getResourceAsStream(fileName)); +- } catch (IOException e) { +- FlightRecorderUI.getDefault().getLogger() +- .log(Level.WARNING, MessageFormat +- .format("Could not load script \"{0}\",\"{1}\"", fileName, e.getMessage())); //$NON-NLS-1$ +- return ""; +- } +- } + } +diff -r 37a493c3f60b -r bbf0648beb5c application/org.openjdk.jmc.flightrecorder.flameview/src/main/java/org/openjdk/jmc/flightrecorder/flameview/views/page.html +--- /dev/null Thu Jan 01 00:00:00 1970 +0000 ++++ b/application/org.openjdk.jmc.flightrecorder.flameview/src/main/java/org/openjdk/jmc/flightrecorder/flameview/views/page.html Mon Nov 25 17:57:23 2019 -0500 +@@ -0,0 +1,54 @@ ++ ++ ++ ++ ++ ++ ++ ++ ++ ++ ++ ++
++ ++ ++ ++ ++ ++ ++ ++ +diff -r 37a493c3f60b -r bbf0648beb5c application/org.openjdk.jmc.flightrecorder.flameview/src/main/resources/page.template +--- a/application/org.openjdk.jmc.flightrecorder.flameview/src/main/resources/page.template Tue Nov 12 16:25:24 2019 +0100 ++++ /dev/null Thu Jan 01 00:00:00 1970 +0000 +@@ -1,49 +0,0 @@ +- +- +- +- +- +- +- +- +- +- +- +-
+- +- +- +- +\ No newline at end of file diff --git a/SOURCES/11-6554.patch b/SOURCES/11-6554.patch new file mode 100644 index 0000000..6707d30 --- /dev/null +++ b/SOURCES/11-6554.patch @@ -0,0 +1,697 @@ +diff --git a/application/org.openjdk.jmc.browser.attach/src/main/java/org/openjdk/jmc/browser/attach/LocalJVMToolkit.java b/application/org.openjdk.jmc.browser.attach/src/main/java/org/openjdk/jmc/browser/attach/LocalJVMToolkit.java +--- a/application/org.openjdk.jmc.browser.attach/src/main/java/org/openjdk/jmc/browser/attach/LocalJVMToolkit.java ++++ b/application/org.openjdk.jmc.browser.attach/src/main/java/org/openjdk/jmc/browser/attach/LocalJVMToolkit.java +@@ -1,5 +1,5 @@ + /* +- * Copyright (c) 2018, Oracle and/or its affiliates. All rights reserved. ++ * Copyright (c) 2018, 2019, Oracle and/or its affiliates. All rights reserved. + * + * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. + * +@@ -199,6 +199,8 @@ + String address = null; + String version = null; + String jvmArgs = null; ++ String jvmVendor = null; ++ + try { + // This used to leak one \BaseNamedObjects\hsperfdata_* Section handle on Windows + MonitoredVm mvm = host.getMonitoredVm(new VmIdentifier(name)); +@@ -235,6 +237,12 @@ + if (sm != null) { + isDebug = isDebug(sm.stringValue()); + } ++ ++ sm = (StringMonitor) mvm.findByName("java.property.java.vm.vendor"); //$NON-NLS-1$ ++ if (sm != null) { ++ jvmVendor = sm.stringValue(); ++ } ++ + // NOTE: isAttachable seems to return true even if a real attach is not possible. + // attachable = MonitoredVmUtil.isAttachable(mvm); + +@@ -260,7 +268,7 @@ + } catch (Exception x) { + // ignore + } +- connDesc = createDescriptor(name, jvmArgs, vmid, connectable, type, jvmArch, address, version, isDebug); ++ connDesc = createDescriptor(name, jvmArgs, jvmVendor, vmid, connectable, type, jvmArch, address, version, isDebug); + return connDesc; + } + }); +@@ -359,6 +367,7 @@ + String javaArgs = null; + String jvmArgs = null; + String jvmVersion = null; ++ String jvmVendor = null; + VirtualMachine vm = null; + try { + // Attach creates one process handle on Windows. +@@ -379,6 +388,7 @@ + jvmType = getJVMType(vmName); + version = props.getProperty("java.version"); //$NON-NLS-1$ + jvmVersion = props.getProperty("java.vm.version"); //$NON-NLS-1$ ++ jvmVendor = props.getProperty("java.vm.vendor"); + isDebug = isDebug(jvmVersion); + jvmArch = JVMArch.getJVMArch(props); + } +@@ -396,7 +406,7 @@ + } + } + if (connectable.isAttachable()) { +- connDesc = createDescriptor(javaArgs, jvmArgs, Integer.parseInt(vmd.id()), connectable, jvmType, jvmArch, ++ connDesc = createDescriptor(javaArgs, jvmArgs, jvmVendor, Integer.parseInt(vmd.id()), connectable, jvmType, jvmArch, + address, version, isDebug); + } + BrowserAttachPlugin.getPluginLogger().info("Done resolving PID " + vmd); //$NON-NLS-1$ +@@ -461,9 +471,10 @@ + } + + private static DiscoveryEntry createDescriptor( +- String javaCommand, String jvmArgs, int pid, Connectable connectable, JVMType type, JVMArch arch, +- String address, String version, boolean isDebug) { +- JVMDescriptor jvmInfo = new JVMDescriptor(version, type, arch, javaCommand, jvmArgs, pid, isDebug, connectable); ++ String javaCommand, String jvmArgs, String jvmVendor, int pid, Connectable connectable, JVMType type, ++ JVMArch arch, String address, String version, boolean isDebug) { ++ JVMDescriptor jvmInfo = new JVMDescriptor(version, type, arch, javaCommand, jvmArgs, jvmVendor, pid, isDebug, ++ connectable); + LocalConnectionDescriptor lcd = new LocalConnectionDescriptor(pid, address, connectable == ATTACHABLE); + String guid = "Local-[PID:" + pid + ", seq:" + (SEQ_NUMBER++) + "]"; //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$ + IServerDescriptor sd = IServerDescriptor.create(guid, null, jvmInfo); +diff --git a/application/org.openjdk.jmc.browser.jdp/src/main/java/org/openjdk/jmc/browser/jdp/JDPDescriptorProvider.java b/application/org.openjdk.jmc.browser.jdp/src/main/java/org/openjdk/jmc/browser/jdp/JDPDescriptorProvider.java +--- a/application/org.openjdk.jmc.browser.jdp/src/main/java/org/openjdk/jmc/browser/jdp/JDPDescriptorProvider.java ++++ b/application/org.openjdk.jmc.browser.jdp/src/main/java/org/openjdk/jmc/browser/jdp/JDPDescriptorProvider.java +@@ -1,5 +1,5 @@ + /* +- * Copyright (c) 2018, Oracle and/or its affiliates. All rights reserved. ++ * Copyright (c) 2018, 2019, Oracle and/or its affiliates. All rights reserved. + * + * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. + * +@@ -89,9 +89,9 @@ + String url = map.get(JMXDataKeys.KEY_JMX_SERVICE_URL); + String commandLine = map.get(JMXDataKeys.KEY_JAVA_COMMAND); + String pid = map.get(JMXDataKeys.KEY_PID); +- // NOTE: We would like to have the JVM type and architecture included in the JDP payload. We should probably file an enhancement request on JDK for this. ++ // NOTE: We would like to have the JVM type, architecture and vendor included in the JDP payload. We should probably file an enhancement request on JDK for this. + JVMDescriptor jvmInfo = new JVMDescriptor(null, JVMType.UNKNOWN, JVMArch.UNKNOWN, commandLine, null, +- pid == null ? null : Integer.parseInt(pid), false, Connectable.MGMNT_AGENT_STARTED); ++ null, pid == null ? null : Integer.parseInt(pid), false, Connectable.MGMNT_AGENT_STARTED); + String path = null; + if (name == null) { + } else if (name.endsWith(PATH_SEPARATOR)) { +diff --git a/application/org.openjdk.jmc.rjmx.services.jfr/src/main/java/org/openjdk/jmc/rjmx/services/jfr/internal/FlightRecorderServiceV2.java b/application/org.openjdk.jmc.rjmx.services.jfr/src/main/java/org/openjdk/jmc/rjmx/services/jfr/internal/FlightRecorderServiceV2.java +--- a/application/org.openjdk.jmc.rjmx.services.jfr/src/main/java/org/openjdk/jmc/rjmx/services/jfr/internal/FlightRecorderServiceV2.java ++++ b/application/org.openjdk.jmc.rjmx.services.jfr/src/main/java/org/openjdk/jmc/rjmx/services/jfr/internal/FlightRecorderServiceV2.java +@@ -1,5 +1,5 @@ + /* +- * Copyright (c) 2018, Oracle and/or its affiliates. All rights reserved. ++ * Copyright (c) 2018, 2019, Oracle and/or its affiliates. All rights reserved. + * + * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. + * +@@ -104,17 +104,14 @@ + } + + private boolean isDynamicFlightRecorderSupported(IConnectionHandle handle) { +- return ConnectionToolkit.isHotSpot(handle) +- && ConnectionToolkit.isJavaVersionAboveOrEqual(handle, JavaVersionSupport.DYNAMIC_JFR_SUPPORTED); +- } +- +- private boolean isFlightRecorderCommercial() { +- return ConnectionToolkit.isHotSpot(connection) +- && !ConnectionToolkit.isJavaVersionAboveOrEqual(connection, JavaVersionSupport.JFR_NOT_COMMERCIAL); ++ // All OpenJDK versions of JFR support dynamic enablement of JFR, so if there are no commercial features in play ++ // all is A-OK. ++ return !cfs.hasCommercialFeatures() || (ConnectionToolkit.isHotSpot(handle) ++ && ConnectionToolkit.isJavaVersionAboveOrEqual(handle, JavaVersionSupport.DYNAMIC_JFR_SUPPORTED)); + } + + private boolean isFlightRecorderDisabled(IConnectionHandle handle) { +- if (cfs != null && isFlightRecorderCommercial()) { ++ if (cfs != null && cfs.hasCommercialFeatures()) { + return !cfs.isCommercialFeaturesEnabled() || JVMSupportToolkit.isFlightRecorderDisabled(handle, false); + } else { + return JVMSupportToolkit.isFlightRecorderDisabled(handle, false); +@@ -127,6 +124,7 @@ + + public FlightRecorderServiceV2(IConnectionHandle handle) throws ConnectionException, ServiceNotAvailableException { + cfs = handle.getServiceOrThrow(ICommercialFeaturesService.class); ++ + if (!isDynamicFlightRecorderSupported(handle) && isFlightRecorderDisabled(handle)) { + throw new ServiceNotAvailableException(""); //$NON-NLS-1$ + } +@@ -481,7 +479,7 @@ + @Override + public boolean isEnabled() { + if (!wasEnabled) { +- boolean isEnabled = isFlightRecorderCommercial() ? cfs.isCommercialFeaturesEnabled() ++ boolean isEnabled = cfs.hasCommercialFeatures() ? cfs.isCommercialFeaturesEnabled() + : isAvailable(connection); + if (isEnabled) { + wasEnabled = true; +diff --git a/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/ConnectionToolkit.java b/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/ConnectionToolkit.java +--- a/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/ConnectionToolkit.java ++++ b/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/ConnectionToolkit.java +@@ -1,5 +1,5 @@ + /* +- * Copyright (c) 2018, Oracle and/or its affiliates. All rights reserved. ++ * Copyright (c) 2018, 2019, Oracle and/or its affiliates. All rights reserved. + * + * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. + * +@@ -56,6 +56,7 @@ + import org.openjdk.jmc.common.version.JavaVMVersionToolkit; + import org.openjdk.jmc.common.version.JavaVersion; + import org.openjdk.jmc.rjmx.internal.RJMXConnection; ++import org.openjdk.jmc.ui.common.jvm.JVMDescriptor; + + /** + * Toolkit providing utility methods to retrieve MBean proxy objects, invoke JMX operations and +@@ -346,7 +347,6 @@ + * otherwise. + */ + public static boolean isJRockit(IConnectionHandle connectionHandle) { +- + String vmName = getVMName(connectionHandle); + return JavaVMVersionToolkit.isJRockitJVMName(vmName); + } +@@ -366,6 +366,38 @@ + } + + /** ++ * Returns {@code true} if the connection handle is associated with an Oracle built JVM, ++ * {@code false} otherwise. If the information is already present in the {@link JVMDescriptor}, ++ * this method will not cause any JMXRMI calls. If the information is lacking, an attempt will ++ * be made to look it up in the connected JVM. If the attempt fails, false will be returned. ++ * ++ * @return {@code true} if the connection handle describes an Oracle JVM, or {@code false} ++ * otherwise or if it could not be determined. ++ */ ++ public static boolean isOracle(IConnectionHandle handle) { ++ JVMDescriptor descriptor = handle.getServerDescriptor().getJvmInfo(); ++ // This should normally not happen for discovered JVMs, but users can create custom connections ++ String vendor = null; ++ if (descriptor != null) { ++ vendor = descriptor.getJvmVendor(); ++ } else { ++ // We try checking if connected ++ if (handle.isConnected()) { ++ MBeanServerConnection connection = handle.getServiceOrNull(MBeanServerConnection.class); ++ if (connection != null) { ++ try { ++ vendor = getRuntimeBean(connection).getVmVendor(); ++ } catch (IOException e) { ++ // Worst case we classify JVM vendor wrong ++ RJMXPlugin.getDefault().getLogger().log(Level.WARNING, "Could not check if Oracle JVM", e); ++ } ++ } ++ } ++ } ++ return vendor != null && vendor.contains("Oracle"); ++ } ++ ++ /** + * This will return true if the java version is above or equal the supplied value. (For example + * 1.7.0_40). + * +@@ -411,5 +443,4 @@ + } + return null; + } +- + } +diff --git a/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/JVMSupportToolkit.java b/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/JVMSupportToolkit.java +--- a/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/JVMSupportToolkit.java ++++ b/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/JVMSupportToolkit.java +@@ -1,5 +1,5 @@ + /* +- * Copyright (c) 2018, Oracle and/or its affiliates. All rights reserved. ++ * Copyright (c) 2018, 2019, Oracle and/or its affiliates. All rights reserved. + * + * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. + * +@@ -137,7 +137,7 @@ + } + + /** +- * Returns information about whether to server denoted by the handle supports Flight Recorder ++ * Returns information about whether the server supports Flight Recorder. + * + * @param handle + * the server to check +diff --git a/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/services/ICommercialFeaturesService.java b/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/services/ICommercialFeaturesService.java +--- a/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/services/ICommercialFeaturesService.java ++++ b/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/services/ICommercialFeaturesService.java +@@ -1,5 +1,5 @@ + /* +- * Copyright (c) 2018, Oracle and/or its affiliates. All rights reserved. ++ * Copyright (c) 2018, 2019, Oracle and/or its affiliates. All rights reserved. + * + * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. + * +@@ -52,4 +52,9 @@ + */ + void enableCommercialFeatures() throws Exception; + ++ /** ++ * @return true if there are commercial features available, or false if this JVM doesn't have ++ * commercial features. ++ */ ++ boolean hasCommercialFeatures(); + } +diff --git a/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/services/internal/CommercialFeaturesServiceFactory.java b/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/services/internal/CommercialFeaturesServiceFactory.java +--- a/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/services/internal/CommercialFeaturesServiceFactory.java ++++ b/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/services/internal/CommercialFeaturesServiceFactory.java +@@ -1,5 +1,5 @@ + /* +- * Copyright (c) 2018, Oracle and/or its affiliates. All rights reserved. ++ * Copyright (c) 2018, 2019, Oracle and/or its affiliates. All rights reserved. + * + * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. + * +@@ -34,6 +34,7 @@ + + import org.openjdk.jmc.common.version.JavaVersion; + import org.openjdk.jmc.rjmx.ConnectionException; ++import org.openjdk.jmc.rjmx.ConnectionToolkit; + import org.openjdk.jmc.rjmx.IConnectionHandle; + import org.openjdk.jmc.rjmx.ServiceNotAvailableException; + import org.openjdk.jmc.rjmx.services.ICommercialFeaturesService; +@@ -52,10 +53,16 @@ + if (descriptor != null) { + JavaVersion version = new JavaVersion(descriptor.getJavaVersion()); + if (version.getMajorVersion() >= 11) { +- return new Jdk11CommercialFeaturesService(); ++ return new NoCommercialFeaturesService(); + } + } +- return new HotSpot23CommercialFeaturesService(handle); ++ ++ // Funnily enough, OpenJDK built JVMs for unknown reasons also have the unlock commercial features flag, ++ // so we'll just check if Oracle is the JVM vendor. Any other vendor will not have JFR protected by commercial flags. ++ if (ConnectionToolkit.isOracle(handle)) { ++ return new HotSpot23CommercialFeaturesService(handle); ++ } ++ return new NoCommercialFeaturesService(); + } + + @Override +diff --git a/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/services/internal/HotSpot23CommercialFeaturesService.java b/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/services/internal/HotSpot23CommercialFeaturesService.java +--- a/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/services/internal/HotSpot23CommercialFeaturesService.java ++++ b/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/services/internal/HotSpot23CommercialFeaturesService.java +@@ -1,5 +1,5 @@ + /* +- * Copyright (c) 2018, Oracle and/or its affiliates. All rights reserved. ++ * Copyright (c) 2018, 2019, Oracle and/or its affiliates. All rights reserved. + * + * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. + * +@@ -43,7 +43,7 @@ + import javax.management.ObjectName; + + public class HotSpot23CommercialFeaturesService implements ICommercialFeaturesService { +- private final static String VM_FLAG = "UnlockCommercialFeatures"; //$NON-NLS-1$ ++ private final static String UNLOCK_COMMERCIAL_FEATURES_FLAG = "UnlockCommercialFeatures"; //$NON-NLS-1$ + private final static String UNLOCK_COMMAND = "VM.unlock_commercial_features"; //$NON-NLS-1$ + private final MBeanServerConnection server; + private final IDiagnosticCommandService dcs; +@@ -54,7 +54,7 @@ + server = handle.getServiceOrThrow(MBeanServerConnection.class); + dcs = handle.getServiceOrNull(IDiagnosticCommandService.class); + try { +- HotspotManagementToolkit.getVMOption(server, VM_FLAG); // Will fail if option is not available ++ HotspotManagementToolkit.getVMOption(server, UNLOCK_COMMERCIAL_FEATURES_FLAG); // Will fail if option is not available + } catch (Exception e) { + // Commercial Feature option is not available but Flight Recorder is. + if (!isJfrMBeanAvailable()) { +@@ -66,7 +66,8 @@ + @Override + public boolean isCommercialFeaturesEnabled() { + try { +- return ((String) HotspotManagementToolkit.getVMOption(server, VM_FLAG)).contains("true"); //$NON-NLS-1$ ++ return ((String) HotspotManagementToolkit.getVMOption(server, UNLOCK_COMMERCIAL_FEATURES_FLAG)) ++ .contains("true"); //$NON-NLS-1$ + } catch (Exception e) { + return false; + } +@@ -78,7 +79,7 @@ + dcs.runCtrlBreakHandlerWithResult(UNLOCK_COMMAND); + } + if (!isCommercialFeaturesEnabled()) { +- HotspotManagementToolkit.setVMOption(server, VM_FLAG, "true"); //$NON-NLS-1$ ++ HotspotManagementToolkit.setVMOption(server, UNLOCK_COMMERCIAL_FEATURES_FLAG, "true"); //$NON-NLS-1$ + } + } + +@@ -96,4 +97,9 @@ + server.getMBeanInfo(candidateObjectName); + return candidateObjectName; + } ++ ++ @Override ++ public boolean hasCommercialFeatures() { ++ return true; ++ } + } +diff --git a/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/services/internal/Jdk11CommercialFeaturesService.java b/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/services/internal/Jdk11CommercialFeaturesService.java +deleted file mode 100644 +--- a/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/services/internal/Jdk11CommercialFeaturesService.java ++++ /dev/null +@@ -1,48 +0,0 @@ +-/* +- * Copyright (c) 2018, Oracle and/or its affiliates. All rights reserved. +- * +- * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. +- * +- * The contents of this file are subject to the terms of either the Universal Permissive License +- * v 1.0 as shown at http://oss.oracle.com/licenses/upl +- * +- * or the following license: +- * +- * Redistribution and use in source and binary forms, with or without modification, are permitted +- * provided that the following conditions are met: +- * +- * 1. Redistributions of source code must retain the above copyright notice, this list of conditions +- * and the following disclaimer. +- * +- * 2. Redistributions in binary form must reproduce the above copyright notice, this list of +- * conditions and the following disclaimer in the documentation and/or other materials provided with +- * the distribution. +- * +- * 3. Neither the name of the copyright holder nor the names of its contributors may be used to +- * endorse or promote products derived from this software without specific prior written permission. +- * +- * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND ANY EXPRESS OR +- * IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND +- * FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR +- * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL +- * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, +- * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, +- * WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY +- * WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. +- */ +-package org.openjdk.jmc.rjmx.services.internal; +- +-import org.openjdk.jmc.rjmx.services.ICommercialFeaturesService; +- +-public class Jdk11CommercialFeaturesService implements ICommercialFeaturesService { +- +- @Override +- public boolean isCommercialFeaturesEnabled() { +- return true; +- } +- +- @Override +- public void enableCommercialFeatures() throws Exception { +- // Noop +- } +-} +diff --git a/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/services/internal/NoCommercialFeaturesService.java b/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/services/internal/NoCommercialFeaturesService.java +new file mode 100644 +--- /dev/null ++++ b/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/services/internal/NoCommercialFeaturesService.java +@@ -0,0 +1,56 @@ ++/* ++ * Copyright (c) 2018, 2019, Oracle and/or its affiliates. All rights reserved. ++ * ++ * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. ++ * ++ * The contents of this file are subject to the terms of either the Universal Permissive License ++ * v 1.0 as shown at http://oss.oracle.com/licenses/upl ++ * ++ * or the following license: ++ * ++ * Redistribution and use in source and binary forms, with or without modification, are permitted ++ * provided that the following conditions are met: ++ * ++ * 1. Redistributions of source code must retain the above copyright notice, this list of conditions ++ * and the following disclaimer. ++ * ++ * 2. Redistributions in binary form must reproduce the above copyright notice, this list of ++ * conditions and the following disclaimer in the documentation and/or other materials provided with ++ * the distribution. ++ * ++ * 3. Neither the name of the copyright holder nor the names of its contributors may be used to ++ * endorse or promote products derived from this software without specific prior written permission. ++ * ++ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND ANY EXPRESS OR ++ * IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND ++ * FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR ++ * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL ++ * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, ++ * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, ++ * WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY ++ * WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. ++ */ ++package org.openjdk.jmc.rjmx.services.internal; ++ ++import org.openjdk.jmc.rjmx.services.ICommercialFeaturesService; ++ ++/** ++ * Used by JVMs with no commercial features, e.g. OpenJDK 8 and JDK 11+ JVMs. ++ */ ++public class NoCommercialFeaturesService implements ICommercialFeaturesService { ++ ++ @Override ++ public boolean isCommercialFeaturesEnabled() { ++ return true; ++ } ++ ++ @Override ++ public void enableCommercialFeatures() throws Exception { ++ // Noop ++ } ++ ++ @Override ++ public boolean hasCommercialFeatures() { ++ return false; ++ } ++} +diff --git a/application/org.openjdk.jmc.ui.common/src/main/java/org/openjdk/jmc/ui/common/jvm/JVMDescriptor.java b/application/org.openjdk.jmc.ui.common/src/main/java/org/openjdk/jmc/ui/common/jvm/JVMDescriptor.java +--- a/application/org.openjdk.jmc.ui.common/src/main/java/org/openjdk/jmc/ui/common/jvm/JVMDescriptor.java ++++ b/application/org.openjdk.jmc.ui.common/src/main/java/org/openjdk/jmc/ui/common/jvm/JVMDescriptor.java +@@ -1,5 +1,5 @@ + /* +- * Copyright (c) 2018, Oracle and/or its affiliates. All rights reserved. ++ * Copyright (c) 2018, 2019, Oracle and/or its affiliates. All rights reserved. + * + * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. + * +@@ -38,6 +38,7 @@ + public class JVMDescriptor { + private final String javaVersion; + private final JVMType jvmType; ++ private final String jvmVendor; + private final JVMArch jvmArch; + private final String javaCommand; + private final String jvmArguments; +@@ -46,13 +47,14 @@ + private final Connectable connectable; + + public JVMDescriptor(String javaVersion, JVMType jvmType, JVMArch jvmArch, String javaCommand, String jvmArguments, +- Integer pid, boolean debug, Connectable attachable) { ++ String jvmVendor, Integer pid, boolean debug, Connectable attachable) { + super(); + this.javaVersion = javaVersion; + this.jvmType = jvmType; + this.jvmArch = jvmArch; + this.javaCommand = javaCommand; + this.jvmArguments = jvmArguments; ++ this.jvmVendor = jvmVendor; + this.pid = pid; + this.debug = debug; + connectable = attachable; +@@ -78,6 +80,10 @@ + return jvmArguments; + } + ++ public String getJvmVendor() { ++ return jvmVendor; ++ } ++ + public Integer getPid() { + return pid; + } +diff --git a/application/tests/org.openjdk.jmc.rjmx.test/src/test/java/org/openjdk/jmc/rjmx/test/JVMSupportToolkitTest.java b/application/tests/org.openjdk.jmc.rjmx.test/src/test/java/org/openjdk/jmc/rjmx/test/JVMSupportToolkitTest.java +--- a/application/tests/org.openjdk.jmc.rjmx.test/src/test/java/org/openjdk/jmc/rjmx/test/JVMSupportToolkitTest.java ++++ b/application/tests/org.openjdk.jmc.rjmx.test/src/test/java/org/openjdk/jmc/rjmx/test/JVMSupportToolkitTest.java +@@ -1,5 +1,5 @@ + /* +- * Copyright (c) 2018, Oracle and/or its affiliates. All rights reserved. ++ * Copyright (c) 2018, 2019, Oracle and/or its affiliates. All rights reserved. + * + * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. + * +@@ -33,6 +33,7 @@ + package org.openjdk.jmc.rjmx.test; + + import static org.junit.Assert.assertEquals; ++import static org.junit.Assert.assertNotNull; + + import org.junit.Test; + import org.openjdk.jmc.rjmx.ConnectionDescriptorBuilder; +@@ -46,7 +47,8 @@ + + @SuppressWarnings("nls") + public class JVMSupportToolkitTest { +- ++ private static final String VENDOR_OPEN_JDK = "OpenJDK"; ++ private static final String VENDOR_ORACLE = "Oracle"; + // FIXME: Add tests for the methods that take IConnectionHandle as a parameter. + + private static final String SUPPORTED_MESSAGE = null; +@@ -63,7 +65,8 @@ + public void testJfr17U40HotSpotSupported() { + ServerHandle server = new ServerHandle( + new ServerDescriptor(null, null, +- new JVMDescriptor("1.7.0_40", JVMType.HOTSPOT, JVMArch.UNKNOWN, null, null, null, false, null)), ++ new JVMDescriptor("1.7.0_40", JVMType.HOTSPOT, JVMArch.UNKNOWN, VENDOR_ORACLE, null, null, null, ++ false, null)), + new ConnectionDescriptorBuilder().hostName("localhost").port(0).build(), null); + String errorMessage = JVMSupportToolkit.checkFlightRecorderSupport(server, false); + assertEquals(SUPPORTED_MESSAGE, errorMessage); +@@ -73,7 +76,8 @@ + public void testJfr17U4HotSpotNotFullySupported() { + ServerHandle server = new ServerHandle( + new ServerDescriptor(null, null, +- new JVMDescriptor("1.7.0_04", JVMType.HOTSPOT, JVMArch.UNKNOWN, null, null, null, false, null)), ++ new JVMDescriptor("1.7.0_04", JVMType.HOTSPOT, JVMArch.UNKNOWN, VENDOR_ORACLE, null, null, null, ++ false, null)), + new ConnectionDescriptorBuilder().hostName("localhost").port(0).build(), null); + String errorMessage = JVMSupportToolkit.checkFlightRecorderSupport(server, false); + assertEquals(Messages.JVMSupport_FLIGHT_RECORDER_NOT_FULLY_SUPPORTED_OLD_HOTSPOT, errorMessage); +@@ -83,7 +87,8 @@ + public void testJfr17HotSpotNotSupported() { + ServerHandle server = new ServerHandle( + new ServerDescriptor(null, null, +- new JVMDescriptor("1.7.0", JVMType.HOTSPOT, JVMArch.UNKNOWN, null, null, null, false, null)), ++ new JVMDescriptor("1.7.0", JVMType.HOTSPOT, JVMArch.UNKNOWN, VENDOR_ORACLE, null, null, null, ++ false, null)), + new ConnectionDescriptorBuilder().hostName("localhost").port(0).build(), null); + String errorMessage = JVMSupportToolkit.checkFlightRecorderSupport(server, false); + assertEquals(Messages.JVMSupport_FLIGHT_RECORDER_NOT_SUPPORTED_OLD_HOTSPOT, errorMessage); +@@ -93,7 +98,8 @@ + public void testJfrJRockitNotSupported() { + ServerHandle server = new ServerHandle( + new ServerDescriptor(null, null, +- new JVMDescriptor("1.6", JVMType.JROCKIT, JVMArch.UNKNOWN, null, null, null, false, null)), ++ new JVMDescriptor("1.6", JVMType.JROCKIT, JVMArch.UNKNOWN, VENDOR_ORACLE, null, null, null, ++ false, null)), + new ConnectionDescriptorBuilder().hostName("localhost").port(0).build(), null); + String errorMessage = JVMSupportToolkit.checkFlightRecorderSupport(server, false); + assertEquals(Messages.JVMSupport_JROCKIT_NO_LONGER_SUPPORTED, errorMessage); +@@ -103,7 +109,8 @@ + public void testJfrOldHotSpotNotSupported() { + ServerHandle server = new ServerHandle( + new ServerDescriptor(null, null, +- new JVMDescriptor("1.6", JVMType.HOTSPOT, JVMArch.UNKNOWN, null, null, null, false, null)), ++ new JVMDescriptor("1.6", JVMType.HOTSPOT, JVMArch.UNKNOWN, VENDOR_ORACLE, null, null, null, ++ false, null)), + new ConnectionDescriptorBuilder().hostName("localhost").port(0).build(), null); + String errorMessage = JVMSupportToolkit.checkFlightRecorderSupport(server, false); + assertEquals(Messages.JVMSupport_FLIGHT_RECORDER_NOT_SUPPORTED_OLD_HOTSPOT, errorMessage); +@@ -112,8 +119,8 @@ + @Test + public void testJfrNonHotSpotNotSupported() { + ServerHandle server = new ServerHandle( +- new ServerDescriptor(null, null, +- new JVMDescriptor("1.7", JVMType.OTHER, JVMArch.UNKNOWN, null, null, null, false, null)), ++ new ServerDescriptor(null, null, new JVMDescriptor("1.7", JVMType.OTHER, JVMArch.UNKNOWN, VENDOR_ORACLE, ++ null, null, null, false, null)), + new ConnectionDescriptorBuilder().hostName("localhost").port(0).build(), null); + String errorMessage = JVMSupportToolkit.checkFlightRecorderSupport(server, false); + assertEquals(Messages.JVMSupport_FLIGHT_RECORDER_NOT_SUPPORTED_NOT_HOTSPOT, errorMessage); +@@ -123,10 +130,32 @@ + public void testJfrUnknownNoWarning() { + ServerHandle server = new ServerHandle( + new ServerDescriptor(null, null, +- new JVMDescriptor("1.7", JVMType.UNKNOWN, JVMArch.UNKNOWN, null, null, null, false, null)), ++ new JVMDescriptor("1.7", JVMType.UNKNOWN, JVMArch.UNKNOWN, VENDOR_ORACLE, null, null, null, ++ false, null)), + new ConnectionDescriptorBuilder().hostName("localhost").port(0).build(), null); + String errorMessage = JVMSupportToolkit.checkFlightRecorderSupport(server, false); + assertEquals(SUPPORTED_MESSAGE, errorMessage); + } + ++ @Test ++ public void testJfr8HotSpotOpenJDKSupported() { ++ ServerHandle server = new ServerHandle( ++ new ServerDescriptor(null, null, ++ new JVMDescriptor("1.8.0", JVMType.HOTSPOT, JVMArch.UNKNOWN, VENDOR_OPEN_JDK, null, null, null, ++ false, null)), ++ new ConnectionDescriptorBuilder().hostName("localhost").port(0).build(), null); ++ String errorMessage = JVMSupportToolkit.checkFlightRecorderSupport(server, false); ++ assertEquals(SUPPORTED_MESSAGE, errorMessage); ++ } ++ ++ @Test ++ public void testJdk7HotSpotOpenJDKNotSupported() { ++ ServerHandle server = new ServerHandle( ++ new ServerDescriptor(null, null, ++ new JVMDescriptor("1.7.0", JVMType.HOTSPOT, JVMArch.UNKNOWN, VENDOR_OPEN_JDK, null, null, null, ++ false, null)), ++ new ConnectionDescriptorBuilder().hostName("localhost").port(0).build(), null); ++ String errorMessage = JVMSupportToolkit.checkFlightRecorderSupport(server, false); ++ assertNotNull(errorMessage); ++ } + } +diff --git a/application/tests/org.openjdk.jmc.rjmx.test/src/test/java/org/openjdk/jmc/rjmx/test/services/CommercialFeaturesServiceTest.java b/application/tests/org.openjdk.jmc.rjmx.test/src/test/java/org/openjdk/jmc/rjmx/test/services/CommercialFeaturesServiceTest.java +--- a/application/tests/org.openjdk.jmc.rjmx.test/src/test/java/org/openjdk/jmc/rjmx/test/services/CommercialFeaturesServiceTest.java ++++ b/application/tests/org.openjdk.jmc.rjmx.test/src/test/java/org/openjdk/jmc/rjmx/test/services/CommercialFeaturesServiceTest.java +@@ -1,5 +1,5 @@ + /* +- * Copyright (c) 2018, Oracle and/or its affiliates. All rights reserved. ++ * Copyright (c) 2018, 2019, Oracle and/or its affiliates. All rights reserved. + * + * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. + * +@@ -36,7 +36,6 @@ + import static org.junit.Assert.assertTrue; + + import org.junit.Test; +- + import org.openjdk.jmc.rjmx.ConnectionException; + import org.openjdk.jmc.rjmx.IConnectionHandle; + import org.openjdk.jmc.rjmx.services.ICommercialFeaturesService; +@@ -63,18 +62,17 @@ + public void testSetCommercialFeaturesState() throws Exception { + ICommercialFeaturesService service = getCommercialFeaturesService(); + // Check state. Any state is okay, but we want to catch exceptions. +- if (!service.isCommercialFeaturesEnabled()) { ++ if (service.hasCommercialFeatures() && !service.isCommercialFeaturesEnabled()) { + service.enableCommercialFeatures(); + } +- assertTrue("Commercial features should now be enabled!", service.isCommercialFeaturesEnabled()); ++ if (service.hasCommercialFeatures()) { ++ assertTrue("Commercial features should now be enabled!", service.isCommercialFeaturesEnabled()); ++ } + } + + private ICommercialFeaturesService getCommercialFeaturesService() throws ConnectionException { +- IConnectionHandle handle = getConnectionHandle(); +- +- // LocalRJMXTestToolkit.createDefaultConnectionHandle(getConnectionManager()); ++ IConnectionHandle handle = getDefaultServer().connect("Connection handle for test"); + assumeHotSpot7u4OrLater(handle); +- + ICommercialFeaturesService service = handle.getServiceOrNull(ICommercialFeaturesService.class); + + assertNotNull( +diff --git a/configuration/spotbugs/spotbugs-exclude.xml b/configuration/spotbugs/spotbugs-exclude.xml +--- a/configuration/spotbugs/spotbugs-exclude.xml ++++ b/configuration/spotbugs/spotbugs-exclude.xml +@@ -566,6 +566,13 @@ + + + ++ ++ ++ ++ ++ ++ + + diff --git a/SOURCES/12-6692.patch b/SOURCES/12-6692.patch new file mode 100644 index 0000000..0abab10 --- /dev/null +++ b/SOURCES/12-6692.patch @@ -0,0 +1,326 @@ +diff --git a/application/org.openjdk.jmc.browser.attach/src/main/java/org/openjdk/jmc/browser/attach/LocalJVMToolkit.java b/application/org.openjdk.jmc.browser.attach/src/main/java/org/openjdk/jmc/browser/attach/LocalJVMToolkit.java +--- a/application/org.openjdk.jmc.browser.attach/src/main/java/org/openjdk/jmc/browser/attach/LocalJVMToolkit.java ++++ b/application/org.openjdk.jmc.browser.attach/src/main/java/org/openjdk/jmc/browser/attach/LocalJVMToolkit.java +@@ -199,6 +199,7 @@ + String address = null; + String version = null; + String jvmArgs = null; ++ String jvmName = null; + String jvmVendor = null; + + try { +@@ -210,6 +211,7 @@ + jvmArgs = MonitoredVmUtil.jvmArgs(mvm); + StringMonitor sm = (StringMonitor) mvm.findByName("java.property.java.vm.name"); //$NON-NLS-1$ + if (sm != null) { ++ jvmName = sm.stringValue(); + type = getJVMType(sm.stringValue()); + } + +@@ -268,7 +270,7 @@ + } catch (Exception x) { + // ignore + } +- connDesc = createDescriptor(name, jvmArgs, jvmVendor, vmid, connectable, type, jvmArch, address, version, isDebug); ++ connDesc = createDescriptor(name, jvmArgs, jvmName, jvmVendor, vmid, connectable, type, jvmArch, address, version, isDebug); + return connDesc; + } + }); +@@ -367,6 +369,7 @@ + String javaArgs = null; + String jvmArgs = null; + String jvmVersion = null; ++ String jvmName = null; + String jvmVendor = null; + VirtualMachine vm = null; + try { +@@ -384,8 +387,8 @@ + + vmd + ": " + e.getMessage()); //$NON-NLS-1$ + } + if (props != null) { +- String vmName = props.getProperty("java.vm.name"); //$NON-NLS-1$ +- jvmType = getJVMType(vmName); ++ jvmName = props.getProperty("java.vm.name"); //$NON-NLS-1$ ++ jvmType = getJVMType(jvmName); + version = props.getProperty("java.version"); //$NON-NLS-1$ + jvmVersion = props.getProperty("java.vm.version"); //$NON-NLS-1$ + jvmVendor = props.getProperty("java.vm.vendor"); +@@ -406,7 +409,7 @@ + } + } + if (connectable.isAttachable()) { +- connDesc = createDescriptor(javaArgs, jvmArgs, jvmVendor, Integer.parseInt(vmd.id()), connectable, jvmType, jvmArch, ++ connDesc = createDescriptor(javaArgs, jvmArgs, jvmName, jvmVendor, Integer.parseInt(vmd.id()), connectable, jvmType, jvmArch, + address, version, isDebug); + } + BrowserAttachPlugin.getPluginLogger().info("Done resolving PID " + vmd); //$NON-NLS-1$ +@@ -471,10 +474,10 @@ + } + + private static DiscoveryEntry createDescriptor( +- String javaCommand, String jvmArgs, String jvmVendor, int pid, Connectable connectable, JVMType type, +- JVMArch arch, String address, String version, boolean isDebug) { +- JVMDescriptor jvmInfo = new JVMDescriptor(version, type, arch, javaCommand, jvmArgs, jvmVendor, pid, isDebug, +- connectable); ++ String javaCommand, String jvmArgs, String jvmName, String jvmVendor, int pid, Connectable connectable, ++ JVMType type, JVMArch arch, String address, String version, boolean isDebug) { ++ JVMDescriptor jvmInfo = new JVMDescriptor(version, type, arch, javaCommand, jvmArgs, jvmName, jvmVendor, pid, ++ isDebug, connectable); + LocalConnectionDescriptor lcd = new LocalConnectionDescriptor(pid, address, connectable == ATTACHABLE); + String guid = "Local-[PID:" + pid + ", seq:" + (SEQ_NUMBER++) + "]"; //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$ + IServerDescriptor sd = IServerDescriptor.create(guid, null, jvmInfo); +diff --git a/application/org.openjdk.jmc.browser.jdp/src/main/java/org/openjdk/jmc/browser/jdp/JDPDescriptorProvider.java b/application/org.openjdk.jmc.browser.jdp/src/main/java/org/openjdk/jmc/browser/jdp/JDPDescriptorProvider.java +--- a/application/org.openjdk.jmc.browser.jdp/src/main/java/org/openjdk/jmc/browser/jdp/JDPDescriptorProvider.java ++++ b/application/org.openjdk.jmc.browser.jdp/src/main/java/org/openjdk/jmc/browser/jdp/JDPDescriptorProvider.java +@@ -91,7 +91,7 @@ + String pid = map.get(JMXDataKeys.KEY_PID); + // NOTE: We would like to have the JVM type, architecture and vendor included in the JDP payload. We should probably file an enhancement request on JDK for this. + JVMDescriptor jvmInfo = new JVMDescriptor(null, JVMType.UNKNOWN, JVMArch.UNKNOWN, commandLine, null, +- null, pid == null ? null : Integer.parseInt(pid), false, Connectable.MGMNT_AGENT_STARTED); ++ null, null, pid == null ? null : Integer.parseInt(pid), false, Connectable.MGMNT_AGENT_STARTED); + String path = null; + if (name == null) { + } else if (name.endsWith(PATH_SEPARATOR)) { +diff --git a/application/org.openjdk.jmc.rjmx.services.jfr/src/main/java/org/openjdk/jmc/rjmx/services/jfr/internal/FlightRecorderServiceV1.java b/application/org.openjdk.jmc.rjmx.services.jfr/src/main/java/org/openjdk/jmc/rjmx/services/jfr/internal/FlightRecorderServiceV1.java +--- a/application/org.openjdk.jmc.rjmx.services.jfr/src/main/java/org/openjdk/jmc/rjmx/services/jfr/internal/FlightRecorderServiceV1.java ++++ b/application/org.openjdk.jmc.rjmx.services.jfr/src/main/java/org/openjdk/jmc/rjmx/services/jfr/internal/FlightRecorderServiceV1.java +@@ -131,6 +131,9 @@ + + public FlightRecorderServiceV1(IConnectionHandle handle) throws ConnectionException, ServiceNotAvailableException { + cfs = handle.getServiceOrThrow(ICommercialFeaturesService.class); ++ if (!JVMSupportToolkit.hasFlightRecorder(handle)) { ++ throw new ServiceNotAvailableException(""); //$NON-NLS-1$ ++ } + if (!isDynamicFlightRecorderSupported(handle) && isFlightRecorderDisabled(handle)) { + throw new ServiceNotAvailableException(""); //$NON-NLS-1$ + } +diff --git a/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/ConnectionToolkit.java b/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/ConnectionToolkit.java +--- a/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/ConnectionToolkit.java ++++ b/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/ConnectionToolkit.java +@@ -377,24 +377,24 @@ + public static boolean isOracle(IConnectionHandle handle) { + JVMDescriptor descriptor = handle.getServerDescriptor().getJvmInfo(); + // This should normally not happen for discovered JVMs, but users can create custom connections +- String vendor = null; ++ String name = null; + if (descriptor != null) { +- vendor = descriptor.getJvmVendor(); ++ name = descriptor.getJvmName(); + } else { + // We try checking if connected + if (handle.isConnected()) { + MBeanServerConnection connection = handle.getServiceOrNull(MBeanServerConnection.class); + if (connection != null) { + try { +- vendor = getRuntimeBean(connection).getVmVendor(); ++ name = getRuntimeBean(connection).getVmName(); + } catch (IOException e) { +- // Worst case we classify JVM vendor wrong ++ // Worst case we classify JVM name wrong + RJMXPlugin.getDefault().getLogger().log(Level.WARNING, "Could not check if Oracle JVM", e); + } + } + } + } +- return vendor != null && vendor.contains("Oracle"); ++ return name != null && (name.contains("Java HotSpot")); + } + + /** +diff --git a/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/JVMSupportToolkit.java b/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/JVMSupportToolkit.java +--- a/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/JVMSupportToolkit.java ++++ b/application/org.openjdk.jmc.rjmx/src/main/java/org/openjdk/jmc/rjmx/JVMSupportToolkit.java +@@ -85,6 +85,27 @@ + } + + /** ++ * Checks if Flight Recorder is available for use ++ * ++ * @param connection ++ * @return If it is an Oracle JVM or there is a FlightRecorder VM option, then return true. ++ * Otherwise, return false. This is used for verifying JDK 8 JVMs that are not built ++ * with JFR enabled, e.g., OpenJDK 8 ++ */ ++ public static boolean hasFlightRecorder(IConnectionHandle connection) { ++ if (ConnectionToolkit.isOracle(connection)) { ++ return true; ++ } ++ MBeanServerConnection server = connection.getServiceOrNull(MBeanServerConnection.class); ++ try { ++ HotspotManagementToolkit.getVMOption(server, "FlightRecorder"); ++ return true; ++ } catch (Exception e) { // RuntimeMBeanException thrown if FlightRecorder is not present ++ return false; ++ } ++ } ++ ++ /** + * Checks if Flight Recorder is disabled. + * + * @param connection +diff --git a/application/org.openjdk.jmc.rjmx/src/main/resources/org/openjdk/jmc/rjmx/messages/internal/messages.properties b/application/org.openjdk.jmc.rjmx/src/main/resources/org/openjdk/jmc/rjmx/messages/internal/messages.properties +--- a/application/org.openjdk.jmc.rjmx/src/main/resources/org/openjdk/jmc/rjmx/messages/internal/messages.properties ++++ b/application/org.openjdk.jmc.rjmx/src/main/resources/org/openjdk/jmc/rjmx/messages/internal/messages.properties +@@ -54,7 +54,7 @@ + JVMSupport_FLIGHT_RECORDER_NOT_FULLY_SUPPORTED_OLD_HOTSPOT_SHORT=Flight Recorder is not fully supported for JVMs with Java versions below 7u40 + JVMSupport_FLIGHT_RECORDER_NOT_SUPPORTED_NOT_HOTSPOT=Flight Recorder is not supported for non HotSpot JVMs + JVMSupport_FLIGHT_RECORDER_NOT_SUPPORTED_NOT_HOTSPOT_SHORT=Flight Recorder is not supported for non HotSpot JVMs +-JVMSupport_FLIGHT_RECORDER_NOT_ENABLED=Flight Recorder features are not enabled. To enable this you need to use an Oracle JDK 7u4 or later JVM started with -XX:+UnlockCommercialFeatures -XX:+FlightRecorder or an OpenJDK 11+ JVM. ++JVMSupport_FLIGHT_RECORDER_NOT_ENABLED=Flight Recorder features are not enabled. To enable this you need to use an Oracle JDK 7u4 or later JVM started with -XX:+UnlockCommercialFeatures -XX:+FlightRecorder, an OpenJDK 8 JFR-enabled JVM, or an OpenJDK 11+ JVM. + JVMSupport_FLIGHT_RECORDER_NOT_ENABLED_SHORT=Flight Recorder features are not enabled + JVMSupport_FLIGHT_RECORDER_DISABLED=Flight Recorder is explicitly disabled. To enable the Flight Recorder, remove -XX:-FlightRecorder from the command line. + JVMSupport_FLIGHT_RECORDER_DISABLED_SHORT=Flight Recorder is explicitly disabled +diff --git a/application/org.openjdk.jmc.ui.common/src/main/java/org/openjdk/jmc/ui/common/jvm/JVMDescriptor.java b/application/org.openjdk.jmc.ui.common/src/main/java/org/openjdk/jmc/ui/common/jvm/JVMDescriptor.java +--- a/application/org.openjdk.jmc.ui.common/src/main/java/org/openjdk/jmc/ui/common/jvm/JVMDescriptor.java ++++ b/application/org.openjdk.jmc.ui.common/src/main/java/org/openjdk/jmc/ui/common/jvm/JVMDescriptor.java +@@ -38,6 +38,7 @@ + public class JVMDescriptor { + private final String javaVersion; + private final JVMType jvmType; ++ private final String jvmName; + private final String jvmVendor; + private final JVMArch jvmArch; + private final String javaCommand; +@@ -47,13 +48,14 @@ + private final Connectable connectable; + + public JVMDescriptor(String javaVersion, JVMType jvmType, JVMArch jvmArch, String javaCommand, String jvmArguments, +- String jvmVendor, Integer pid, boolean debug, Connectable attachable) { ++ String jvmName, String jvmVendor, Integer pid, boolean debug, Connectable attachable) { + super(); + this.javaVersion = javaVersion; + this.jvmType = jvmType; + this.jvmArch = jvmArch; + this.javaCommand = javaCommand; + this.jvmArguments = jvmArguments; ++ this.jvmName = jvmName; + this.jvmVendor = jvmVendor; + this.pid = pid; + this.debug = debug; +@@ -80,6 +82,10 @@ + return jvmArguments; + } + ++ public String getJvmName() { ++ return jvmName; ++ } ++ + public String getJvmVendor() { + return jvmVendor; + } +diff --git a/application/tests/org.openjdk.jmc.rjmx.test/src/test/java/org/openjdk/jmc/rjmx/test/JVMSupportToolkitTest.java b/application/tests/org.openjdk.jmc.rjmx.test/src/test/java/org/openjdk/jmc/rjmx/test/JVMSupportToolkitTest.java +--- a/application/tests/org.openjdk.jmc.rjmx.test/src/test/java/org/openjdk/jmc/rjmx/test/JVMSupportToolkitTest.java ++++ b/application/tests/org.openjdk.jmc.rjmx.test/src/test/java/org/openjdk/jmc/rjmx/test/JVMSupportToolkitTest.java +@@ -47,8 +47,10 @@ + + @SuppressWarnings("nls") + public class JVMSupportToolkitTest { +- private static final String VENDOR_OPEN_JDK = "OpenJDK"; +- private static final String VENDOR_ORACLE = "Oracle"; ++ private static final String NAME_OPEN_JDK = "OpenJDK 64-Bit Server VM"; ++ private static final String NAME_ORACLE = "Java HotSpot(TM) 64-Bit Server VM"; ++ private static final String VENDOR_OPEN_JDK = "Oracle Corporation"; ++ private static final String VENDOR_ORACLE = "Oracle Corporation"; + // FIXME: Add tests for the methods that take IConnectionHandle as a parameter. + + private static final String SUPPORTED_MESSAGE = null; +@@ -65,8 +67,8 @@ + public void testJfr17U40HotSpotSupported() { + ServerHandle server = new ServerHandle( + new ServerDescriptor(null, null, +- new JVMDescriptor("1.7.0_40", JVMType.HOTSPOT, JVMArch.UNKNOWN, VENDOR_ORACLE, null, null, null, +- false, null)), ++ new JVMDescriptor("1.7.0_40", JVMType.HOTSPOT, JVMArch.UNKNOWN, null, null, NAME_ORACLE, ++ VENDOR_ORACLE, null, false, null)), + new ConnectionDescriptorBuilder().hostName("localhost").port(0).build(), null); + String errorMessage = JVMSupportToolkit.checkFlightRecorderSupport(server, false); + assertEquals(SUPPORTED_MESSAGE, errorMessage); +@@ -76,8 +78,8 @@ + public void testJfr17U4HotSpotNotFullySupported() { + ServerHandle server = new ServerHandle( + new ServerDescriptor(null, null, +- new JVMDescriptor("1.7.0_04", JVMType.HOTSPOT, JVMArch.UNKNOWN, VENDOR_ORACLE, null, null, null, +- false, null)), ++ new JVMDescriptor("1.7.0_04", JVMType.HOTSPOT, JVMArch.UNKNOWN, null, null, NAME_ORACLE, ++ VENDOR_ORACLE, null, false, null)), + new ConnectionDescriptorBuilder().hostName("localhost").port(0).build(), null); + String errorMessage = JVMSupportToolkit.checkFlightRecorderSupport(server, false); + assertEquals(Messages.JVMSupport_FLIGHT_RECORDER_NOT_FULLY_SUPPORTED_OLD_HOTSPOT, errorMessage); +@@ -87,8 +89,8 @@ + public void testJfr17HotSpotNotSupported() { + ServerHandle server = new ServerHandle( + new ServerDescriptor(null, null, +- new JVMDescriptor("1.7.0", JVMType.HOTSPOT, JVMArch.UNKNOWN, VENDOR_ORACLE, null, null, null, +- false, null)), ++ new JVMDescriptor("1.7.0", JVMType.HOTSPOT, JVMArch.UNKNOWN, null, null, NAME_ORACLE, ++ VENDOR_ORACLE, null, false, null)), + new ConnectionDescriptorBuilder().hostName("localhost").port(0).build(), null); + String errorMessage = JVMSupportToolkit.checkFlightRecorderSupport(server, false); + assertEquals(Messages.JVMSupport_FLIGHT_RECORDER_NOT_SUPPORTED_OLD_HOTSPOT, errorMessage); +@@ -98,8 +100,8 @@ + public void testJfrJRockitNotSupported() { + ServerHandle server = new ServerHandle( + new ServerDescriptor(null, null, +- new JVMDescriptor("1.6", JVMType.JROCKIT, JVMArch.UNKNOWN, VENDOR_ORACLE, null, null, null, +- false, null)), ++ new JVMDescriptor("1.6", JVMType.JROCKIT, JVMArch.UNKNOWN, null, null, NAME_ORACLE, ++ VENDOR_ORACLE, null, false, null)), + new ConnectionDescriptorBuilder().hostName("localhost").port(0).build(), null); + String errorMessage = JVMSupportToolkit.checkFlightRecorderSupport(server, false); + assertEquals(Messages.JVMSupport_JROCKIT_NO_LONGER_SUPPORTED, errorMessage); +@@ -109,8 +111,8 @@ + public void testJfrOldHotSpotNotSupported() { + ServerHandle server = new ServerHandle( + new ServerDescriptor(null, null, +- new JVMDescriptor("1.6", JVMType.HOTSPOT, JVMArch.UNKNOWN, VENDOR_ORACLE, null, null, null, +- false, null)), ++ new JVMDescriptor("1.6", JVMType.HOTSPOT, JVMArch.UNKNOWN, null, null, NAME_ORACLE, ++ VENDOR_ORACLE, null, false, null)), + new ConnectionDescriptorBuilder().hostName("localhost").port(0).build(), null); + String errorMessage = JVMSupportToolkit.checkFlightRecorderSupport(server, false); + assertEquals(Messages.JVMSupport_FLIGHT_RECORDER_NOT_SUPPORTED_OLD_HOTSPOT, errorMessage); +@@ -119,8 +121,9 @@ + @Test + public void testJfrNonHotSpotNotSupported() { + ServerHandle server = new ServerHandle( +- new ServerDescriptor(null, null, new JVMDescriptor("1.7", JVMType.OTHER, JVMArch.UNKNOWN, VENDOR_ORACLE, +- null, null, null, false, null)), ++ new ServerDescriptor(null, null, ++ new JVMDescriptor("1.7", JVMType.OTHER, JVMArch.UNKNOWN, null, null, NAME_ORACLE, VENDOR_ORACLE, ++ null, false, null)), + new ConnectionDescriptorBuilder().hostName("localhost").port(0).build(), null); + String errorMessage = JVMSupportToolkit.checkFlightRecorderSupport(server, false); + assertEquals(Messages.JVMSupport_FLIGHT_RECORDER_NOT_SUPPORTED_NOT_HOTSPOT, errorMessage); +@@ -130,8 +133,8 @@ + public void testJfrUnknownNoWarning() { + ServerHandle server = new ServerHandle( + new ServerDescriptor(null, null, +- new JVMDescriptor("1.7", JVMType.UNKNOWN, JVMArch.UNKNOWN, VENDOR_ORACLE, null, null, null, +- false, null)), ++ new JVMDescriptor("1.7", JVMType.UNKNOWN, JVMArch.UNKNOWN, null, null, NAME_ORACLE, ++ VENDOR_ORACLE, null, false, null)), + new ConnectionDescriptorBuilder().hostName("localhost").port(0).build(), null); + String errorMessage = JVMSupportToolkit.checkFlightRecorderSupport(server, false); + assertEquals(SUPPORTED_MESSAGE, errorMessage); +@@ -141,8 +144,8 @@ + public void testJfr8HotSpotOpenJDKSupported() { + ServerHandle server = new ServerHandle( + new ServerDescriptor(null, null, +- new JVMDescriptor("1.8.0", JVMType.HOTSPOT, JVMArch.UNKNOWN, VENDOR_OPEN_JDK, null, null, null, +- false, null)), ++ new JVMDescriptor("1.8.0", JVMType.HOTSPOT, JVMArch.UNKNOWN, null, null, NAME_OPEN_JDK, ++ VENDOR_OPEN_JDK, null, false, null)), + new ConnectionDescriptorBuilder().hostName("localhost").port(0).build(), null); + String errorMessage = JVMSupportToolkit.checkFlightRecorderSupport(server, false); + assertEquals(SUPPORTED_MESSAGE, errorMessage); +@@ -152,8 +155,8 @@ + public void testJdk7HotSpotOpenJDKNotSupported() { + ServerHandle server = new ServerHandle( + new ServerDescriptor(null, null, +- new JVMDescriptor("1.7.0", JVMType.HOTSPOT, JVMArch.UNKNOWN, VENDOR_OPEN_JDK, null, null, null, +- false, null)), ++ new JVMDescriptor("1.7.0", JVMType.HOTSPOT, JVMArch.UNKNOWN, null, null, NAME_OPEN_JDK, ++ VENDOR_OPEN_JDK, null, false, null)), + new ConnectionDescriptorBuilder().hostName("localhost").port(0).build(), null); + String errorMessage = JVMSupportToolkit.checkFlightRecorderSupport(server, false); + assertNotNull(errorMessage); diff --git a/SOURCES/2-javax.patch b/SOURCES/2-javax.patch new file mode 100644 index 0000000..e34f1ac --- /dev/null +++ b/SOURCES/2-javax.patch @@ -0,0 +1,36 @@ +diff --git a/application/org.openjdk.jmc.console.ui/META-INF/MANIFEST.MF b/application/org.openjdk.jmc.console.ui/META-INF/MANIFEST.MF +index a0bd643..926c1b3 100644 +--- a/application/org.openjdk.jmc.console.ui/META-INF/MANIFEST.MF ++++ b/application/org.openjdk.jmc.console.ui/META-INF/MANIFEST.MF +@@ -10,7 +10,7 @@ Bundle-Localization: plugin + Require-Bundle: org.openjdk.jmc.rjmx.ui;visibility:=reexport, + org.eclipse.e4.core.contexts;bundle-version="1.3.0", + org.eclipse.e4.core.di;bundle-version="1.3.0";visibility:=reexport, +- javax.annotation;bundle-version="1.0.0";visibility:=reexport, ++ javax.annotation-api;bundle-version="1.2.0";visibility:=reexport, + javax.inject;bundle-version="1.0.0";visibility:=reexport, + org.openjdk.jmc.commands + Bundle-ActivationPolicy: lazy +@@ -24,7 +24,7 @@ Export-Package: org.openjdk.jmc.console.ui.actions; + org.openjdk.jmc.console.ui.messages.internal;x-friends:="org.openjdk.jmc.console.uitest", + org.openjdk.jmc.console.ui.preferences;x-friends:="org.openjdk.jmc.console.uitest" + Bundle-ClassPath: . +-Import-Package: javax.annotation;version="1.0.0", ++Import-Package: javax.annotation;version="1.2.0", + javax.inject;version="1.0.0" + Automatic-Module-Name: org.openjdk.jmc.console.ui + +diff --git a/application/org.openjdk.jmc.feature.rcp/feature.xml b/application/org.openjdk.jmc.feature.rcp/feature.xml +index 828c3f2..cf4b462 100644 +--- a/application/org.openjdk.jmc.feature.rcp/feature.xml ++++ b/application/org.openjdk.jmc.feature.rcp/feature.xml +@@ -84,7 +84,8 @@ + + + +- ++ ++ + + + diff --git a/SOURCES/3-remove-profiles.patch b/SOURCES/3-remove-profiles.patch new file mode 100644 index 0000000..0a5622a --- /dev/null +++ b/SOURCES/3-remove-profiles.patch @@ -0,0 +1,141 @@ +diff -r c34e9deef71f pom.xml +--- a/pom.xml Fri Apr 17 15:53:00 2020 -0400 ++++ b/pom.xml Fri Apr 17 16:01:41 2020 -0400 +@@ -105,137 +105,6 @@ + ${snapshot.repo} + + +- +- +- 2019-09 +- +- true +- +- +- +- +- org.eclipse.tycho +- target-platform-configuration +- ${tycho.version} +- +- +- +- org.openjdk.jmc +- platform-definition-2019-09 +- 7.1.1-SNAPSHOT +- +- +- +- +- +- +- +- +- 2019-06 +- +- +- +- org.eclipse.tycho +- target-platform-configuration +- ${tycho.version} +- +- +- +- org.openjdk.jmc +- platform-definition-2019-06 +- 7.1.1-SNAPSHOT +- +- +- +- +- +- +- +- +- 2019-03 +- +- +- +- org.eclipse.tycho +- target-platform-configuration +- ${tycho.version} +- +- +- +- org.openjdk.jmc +- platform-definition-2019-03 +- 7.1.1-SNAPSHOT +- +- +- +- +- +- +- +- +- 2018-12 +- +- +- +- org.eclipse.tycho +- target-platform-configuration +- ${tycho.version} +- +- +- +- org.openjdk.jmc +- platform-definition-2018-12 +- 7.1.1-SNAPSHOT +- +- +- +- +- +- +- +- +- 2018-09 +- +- +- +- org.eclipse.tycho +- target-platform-configuration +- ${tycho.version} +- +- +- +- org.openjdk.jmc +- platform-definition-2018-09 +- 7.1.1-SNAPSHOT +- +- +- +- +- +- +- +- +- photon +- +- +- +- org.eclipse.tycho +- target-platform-configuration +- ${tycho.version} +- +- +- +- org.openjdk.jmc +- platform-definition-photon +- 7.1.1-SNAPSHOT +- +- +- +- +- +- +- +- + + + diff --git a/SOURCES/4-remove-localization.patch b/SOURCES/4-remove-localization.patch new file mode 100644 index 0000000..271e118 --- /dev/null +++ b/SOURCES/4-remove-localization.patch @@ -0,0 +1,48 @@ +diff --git a/application/org.openjdk.jmc.feature.rcp/feature.xml b/application/org.openjdk.jmc.feature.rcp/feature.xml +index 1e295ca..9687dd4 100644 +--- a/application/org.openjdk.jmc.feature.rcp/feature.xml ++++ b/application/org.openjdk.jmc.feature.rcp/feature.xml +@@ -51,18 +51,10 @@ + id="org.openjdk.jmc.feature.console" + version="0.0.0"/> + +- +- + + +- +- + +@@ -104,24 +96,6 @@ + version="0.0.0" + unpack="false"/> + +- +- +- +- + +- +- + +\ No newline at end of file +diff --git a/application/pom.xml b/application/pom.xml +index d185d9d..89fd033 100644 +--- a/application/pom.xml ++++ b/application/pom.xml +@@ -77,7 +77,6 @@ + org.openjdk.jmc.flightrecorder.ui + org.openjdk.jmc.greychart + org.openjdk.jmc.greychart.ui +- org.openjdk.jmc.ide.jdt + org.openjdk.jmc.ide.launch + org.openjdk.jmc.ide.ui + org.openjdk.jmc.jdp diff --git a/SOURCES/6-remove-buchen-repo.patch b/SOURCES/6-remove-buchen-repo.patch new file mode 100644 index 0000000..6e60bee --- /dev/null +++ b/SOURCES/6-remove-buchen-repo.patch @@ -0,0 +1,16 @@ +diff --git a/application/org.openjdk.jmc.rcp.product/pom.xml b/application/org.openjdk.jmc.rcp.product/pom.xml +index 8e6b303..8060da9 100644 +--- a/application/org.openjdk.jmc.rcp.product/pom.xml ++++ b/application/org.openjdk.jmc.rcp.product/pom.xml +@@ -141,11 +141,4 @@ + + ../../target/ + +- +- +- buchen-maven-repo +- http://buchen.github.io/maven-repo +- default +- +- + diff --git a/SOURCES/7-add-hamcrest.patch b/SOURCES/7-add-hamcrest.patch new file mode 100644 index 0000000..6940f7d --- /dev/null +++ b/SOURCES/7-add-hamcrest.patch @@ -0,0 +1,16 @@ +diff --git a/application/tests/org.openjdk.jmc.rjmx.services.jfr.test/META-INF/MANIFEST.MF b/application/tests/org.openjdk.jmc.rjmx.services.jfr.test/META-INF/MANIFEST.MF +index bf02b43..a4d5a67 100644 +--- a/application/tests/org.openjdk.jmc.rjmx.services.jfr.test/META-INF/MANIFEST.MF ++++ b/application/tests/org.openjdk.jmc.rjmx.services.jfr.test/META-INF/MANIFEST.MF +@@ -12,7 +12,9 @@ Require-Bundle: org.junit, + org.openjdk.jmc.rjmx.services.jfr, + org.openjdk.jmc.rjmx, + org.openjdk.jmc.common, +- org.openjdk.jmc.flightrecorder.configuration ++ org.openjdk.jmc.flightrecorder.configuration, ++ org.hamcrest.core + Export-Package: org.openjdk.jmc.rjmx.services.jfr.test + Automatic-Module-Name: org.openjdk.jmc.rjmx.services.jfr.test +-Import-Package: org.openjdk.jmc.flightrecorder.jdk ++Import-Package: org.openjdk.jmc.flightrecorder.jdk, ++ org.hamcrest.core diff --git a/SOURCES/8-remove-arch.patch b/SOURCES/8-remove-arch.patch new file mode 100644 index 0000000..dae259d --- /dev/null +++ b/SOURCES/8-remove-arch.patch @@ -0,0 +1,26 @@ +diff --git a/pom.xml b/pom.xml +index 59f292a..2985869 100644 +--- a/pom.xml ++++ b/pom.xml +@@ -117,21 +117,11 @@ + ${tycho.version} + + +- +- win32 +- win32 +- x86_64 +- + + linux + gtk + x86_64 + +- +- macosx +- cocoa +- x86_64 +- + + +
diff --git a/SOURCES/9-remove-jacoco-dep.patch b/SOURCES/9-remove-jacoco-dep.patch new file mode 100644 index 0000000..b282ea6 --- /dev/null +++ b/SOURCES/9-remove-jacoco-dep.patch @@ -0,0 +1,15 @@ +diff -r 07b99eac9056 application/pom.xml +--- a/application/pom.xml Fri Apr 17 17:57:23 2020 -0400 ++++ b/application/pom.xml Fri Apr 17 17:58:31 2020 -0400 +@@ -160,11 +160,6 @@ + flightrecorder.rules.jdk + 7.1.1 + +- +- org.jacoco +- jacoco-maven-plugin +- 0.8.3 +- + + + diff --git a/SOURCES/jmc.1 b/SOURCES/jmc.1 new file mode 100644 index 0000000..ecc2d3c --- /dev/null +++ b/SOURCES/jmc.1 @@ -0,0 +1,24 @@ +.TH JMC 1 +.SH NAME +jmc \- Java profiling and diagnostics tool +.SH SYNOPSIS +.B jmc +[\fB\-vm\fR \fIJVM\fR] +.SH DESCRIPTION +JDK Mission Control is a powerful profiler for HotSpot JVMs and has an +advanced set of tools that enables efficient and detailed analysis of the +extensive data collected by JDK Flight Recorder. +The tool chain enables developers and administrators to collect and analyze data +from Java applications running locally or deployed in production environments. +.SH OPTIONS +.TP +.BR \-vm \fIJVM\fR +Specify path to the JVM you wish to use for running JDK Mission Control. +Defaults to system Java. +.SH FILES +.RS +Startup options are specified here. +The -vm option can be set here as well to avoid specifying at startup. +Options specified at startup will override values in this file. +.SH BUGS +Report bugs to JMC mailing list: diff --git a/SOURCES/jmc.desktop b/SOURCES/jmc.desktop new file mode 100644 index 0000000..93e9f74 --- /dev/null +++ b/SOURCES/jmc.desktop @@ -0,0 +1,8 @@ +[Desktop Entry] +Name=JDK Mission Control +Comment=Analyze performance of Java applications +Exec=jmc +Icon=jmc +Terminal=false +Type=Application +Categories=Development;Java;Profiling; diff --git a/SOURCES/symlink_libs.sh b/SOURCES/symlink_libs.sh new file mode 100644 index 0000000..8aa7f1c --- /dev/null +++ b/SOURCES/symlink_libs.sh @@ -0,0 +1,59 @@ +#!/bin/bash + +# This script replaces plugins with symlinks +# $1 : folder to replace with symlinks + +set -e + +SCL_JAVA_DIRS=${@:2} + +pushd () { + command pushd "$@" > /dev/null +} + +popd () { + command popd "$@" > /dev/null +} + +function _sym { + if [ -f $1 ]; then + echo "linking $1 to $2" + rm -rf $1 + ln -s $2 $1 + else + echo "Failed to find $1" + ls -l + exit 1 + fi +} + +function _symlink { + _f=$(ls | grep -e "^$1" || :) + if [ -n "$_f" ] ; then + rm -rf $_f + for SCL_JAVA_DIR in ${SCL_JAVA_DIRS}; do + if [ -f ${SCL_JAVA_DIR}/$2 ]; then + echo "linking ${_f%.jar}.jar to ${SCL_JAVA_DIR}/$2" + ln -s ${SCL_JAVA_DIR}/$2 ${_f%.jar}.jar + return 0 + fi + done + echo "not found $2 in any of ${SCL_JAVA_DIRS}" + exit 1 + fi +} + +pushd $1 + # owasp-java-encoder + _sym org.owasp.encoder_1.2.2.jar /usr/share/java/owasp-java-encoder/encoder.jar + # jaf + _sym com.sun.activation.jakarta.activation_1.2.1.jar /usr/share/java/jaf/jakarta.activation.jar + _sym jakarta.activation-api_1.2.1.jar /usr/share/java/jaf/jakarta.activation-api.jar + # HdrHistogram + _sym org.hdrhistogram.HdrHistogram_2.1.11.jar /usr/share/java/HdrHistogram.jar + # jmc-core + _symlink org.openjdk.jmc.common_ common.jar + _symlink org.openjdk.jmc.flightrecorder_ flightrecorder.jar + _symlink org.openjdk.jmc.flightrecorder.rules_ flightrecorder.rules.jar + _symlink org.openjdk.jmc.flightrecorder.rules.jdk_ flightrecorder.rules.jdk.jar +popd \ No newline at end of file diff --git a/SPECS/jmc.spec b/SPECS/jmc.spec new file mode 100644 index 0000000..3a55391 --- /dev/null +++ b/SPECS/jmc.spec @@ -0,0 +1,242 @@ +# Version +%global major 7 +%global minor 1 +%global patchlevel 1 + +# Revision +%global revnum 5 +# set to 1 for hg snapshots, 0 for release +%global usesnapshot 0 + +# SNAPSHOT version +%global hgrevhash e67446b5fc9d +%global hgrevdate 20200422 + +%global tarball_name jmc7-%{hgrevhash} + +# Install jmc in /usr/lib/jmc (arch-specific and multilib exempt) +%global _jmcdir %{_prefix}/lib/%{name} + +%global debug_package %{nil} + +%if %{usesnapshot} + %global releasestr %{revnum}.%{hgrevdate}hg%{hgrevhash} +%else + %global releasestr %{revnum} +%endif + +%ifarch %{ix86} + %global eclipse_arch x86 +%endif +%ifarch %{arm} + %global eclipse_arch arm +%endif +%ifarch s390x x86_64 aarch64 ppc64le + %global eclipse_arch %{_arch} +%endif + +# Don't export Eclipse libraries +%global __provides_exclude_from ^%{_jmcdir}/plugins/org.eclipse.*$ +%global __requires_exclude_from ^%{_jmcdir}/plugins/org.eclipse.*$ + +%global __requires_exclude ^osgi\\((javax|org\\.apache|org\\.eclipse|org\\.sat4j).*$ +%global __provides_exclude ^osgi\\((com|javax|org\\.apache|org\\.glassfish|org\\.kxml2|org\\.sat4j|org\\.tukaani|org\\.w3c|org\\.xmlpull).*$ + +Name: jmc +Version: %{major}.%{minor}.%{patchlevel} +Release: %{releasestr}%{?dist} +Summary: JDK Mission Control is a profiling and diagnostics tool + +License: UPL +URL: http://openjdk.java.net/projects/jmc/ + +Source0: https://hg.openjdk.java.net/jmc/jmc7/archive/%{hgrevhash}.tar.gz +Source1: %{name}.desktop +Source2: %{name}.1 +Source3: symlink_libs.sh + +# Remove optional twitter related functionality +Patch0: 0-remove-twitter.patch +# Update javamail dependency names +Patch1: 1-fix-javamail.patch +# Update javax dependency names to match what is found in Fedora +Patch2: 2-javax.patch +# Remove maven build profiles that won't be used in local build +Patch3: 3-remove-profiles.patch +# Remove localization files that currently cannot be supported +# due to a packaging issue for Eclipse language packs +# eclipse-nls-ja and eclipse-nls-zh +# They currently provide multiple archs within the same package +# and the local build system cannot fulfill dependencies from them +Patch4: 4-remove-localization.patch +# Remove unused module org.openjdk.jmc.ide.jdt +Patch5: 5-remove-ide-jdt.patch +# Remove unused remote repository definition +Patch6: 6-remove-buchen-repo.patch +# Add dependency on org. hamcrest-core to provide class used in unit tests +Patch7: 7-add-hamcrest.patch +# Remove windows and mac arches +Patch8: 8-remove-arch.patch +# Remove unnecessary dependency +Patch9: 9-remove-jacoco-dep.patch +# Revert downloading of flameview assets from the web +Patch10: 10-revert-flameview.patch +# Backport of JMC-6554 to add jvm vendor information retrieval and commercial features checks +Patch11: 11-6554.patch +# Backport of JMC-6692 to prevent JMC from prompting the user +# to enable commercial features when using a JDK8 JFR-enabled JVM +Patch12: 12-6692.patch + +ExclusiveArch: x86_64 + +BuildRequires: desktop-file-utils +BuildRequires: maven-local +BuildRequires: osgi(javax.annotation-api) + +BuildRequires: eclipse-pde +BuildRequires: tycho + +BuildRequires: javamail +BuildRequires: mvn(org.commonjava.maven.plugins:directory-maven-plugin) +BuildRequires: mvn(com.sun.activation:jakarta.activation) +BuildRequires: mvn(org.openjdk.jmc:common) +BuildRequires: HdrHistogram >= 2.1.11 + +Requires: java-openjdk >= 1:1.8 + +Requires: osgi(com.sun.activation.jakarta.activation) +Requires: osgi(org.openjdk.jmc.common) +Requires: osgi(org.openjdk.jmc.flightrecorder) +Requires: osgi(org.openjdk.jmc.flightrecorder.rules) +Requires: osgi(org.openjdk.jmc.flightrecorder.rules.jdk) +Requires: osgi(org.owasp.encoder) +Requires: osgi(org.hdrhistogram.HdrHistogram) >= 2.1.11 + +Requires: gtk3 +Requires: webkitgtk4 +Requires: libGLU.so.1()(64bit) + +%description +JDK Mission Control is a powerful profiler for HotSpot JVMs and has an +advanced set of tools that enables efficient and detailed analysis of the +extensive data collected by JDK Flight Recorder. The tool chain enables +developers and administrators to collect and analyze data from Java +applications running locally or deployed in production environments. + +%prep +%setup -q -n %{tarball_name} + +%patch0 -p1 +%patch1 -p1 +%patch2 -p1 +%patch3 -p1 +%patch4 -p1 +%patch5 -p1 +%patch6 -p1 +%patch7 -p1 +%patch8 -p1 +%patch9 -p1 +%patch10 -p1 +%patch11 -p1 +%patch12 -p1 + +%pom_disable_module releng +%pom_disable_module l10n application +%pom_disable_module org.openjdk.jmc.updatesite.ide application +%pom_disable_module org.openjdk.jmc.updatesite.rcp application + +%pom_remove_plugin com.github.spotbugs:spotbugs-maven-plugin +%pom_remove_plugin :maven-enforcer-plugin + +%pom_remove_plugin :jacoco-maven-plugin application/tests +%pom_remove_plugin :jacoco-maven-plugin application/uitests +%pom_disable_module coverage application + +# Info.plist are mac files and we only build for Linux +%pom_remove_plugin name.abuchen:fix-info-plist-maven-plugin application/org.openjdk.jmc.rcp.product + +%pom_remove_plugin org.codehaus.mojo:buildnumber-maven-plugin +%pom_remove_plugin org.apache.maven.plugins:maven-deploy-plugin + +TYCHO_ENV="linuxgtk%{eclipse_arch}" +%pom_xpath_set "pom:configuration/pom:environments" "$TYCHO_ENV" + +%build +# some tests require large heap and fail with OOM +# depending on the builder resources +%mvn_build -j -- -Dmaven.test.failure.ignore=true + +%install + +# not using mvn_install macro because it installs JMC as an Eclipse plugin +# we want to install JMC as an RCP application + +# change jmc.ini to use system java (remove -vm option line) +sed -i '/^-vm$/d' %{_builddir}/%{tarball_name}/target/products/org.openjdk.jmc/%{_os}/gtk/%{eclipse_arch}/%{name}.ini +sed -i '/^..\/..\/bin\/$/d' %{_builddir}/%{tarball_name}/target/products/org.openjdk.jmc/%{_os}/gtk/%{eclipse_arch}/%{name}.ini + +# add IgnoreUnrecognizedVMOptions flag to allow running on OpenJDK 8 without 'Unrecognized VM option' error +sed -i '/^-vmargs$/a -XX:+IgnoreUnrecognizedVMOptions' %{_builddir}/%{tarball_name}/target/products/org.openjdk.jmc/%{_os}/gtk/%{eclipse_arch}/%{name}.ini + +# move contents of target/products/org.openjdk.jmc/${_os}/gtk/%{eclipse_arch}/ to /usr/lib/jmc/ +install -d -m 755 %{buildroot}%{_jmcdir} +cp -p -r %{_builddir}/%{tarball_name}/target/products/org.openjdk.jmc/%{_os}/gtk/%{eclipse_arch}/* %{buildroot}%{_jmcdir}/ + +# move jmc.ini to /etc/jmc.ini +install -d -m 755 %{buildroot}%{_sysconfdir} +mv %{buildroot}%{_jmcdir}/%{name}.ini %{buildroot}%{_sysconfdir}/%{name}.ini +ln -s %{_sysconfdir}/%{name}.ini %{buildroot}%{_jmcdir}/%{name}.ini + +# create symlink to jmc in /usr/bin/ +install -d -m 755 %{buildroot}%{_bindir} +ln -s %{_jmcdir}/%{name} %{buildroot}%{_bindir}/%{name} + +# replace jars with symlinks to installed libraries +bash %{SOURCE3} %{buildroot}%{_jmcdir}/plugins %{_javadir} %{_jnidir} %{_javadir_maven} %{_jnidir_maven} %{_javadir}/jmc-core + +# create application launcher in desktop menu +install -d -m 755 %{buildroot}%{_datadir}/pixmaps +mv %{buildroot}%{_jmcdir}/icon.xpm %{buildroot}%{_datadir}/pixmaps/%{name}.xpm +chmod 644 %{buildroot}%{_datadir}/pixmaps/%{name}.xpm +desktop-file-install --dir=%{buildroot}%{_datadir}/applications %{SOURCE1} + +# install pom file +install -d -m 755 %{buildroot}%{_datadir}/maven-poms/%{name} +install -p -m 644 %{_builddir}/%{tarball_name}/pom.xml %{buildroot}%{_datadir}/maven-poms/%{name}/%{name}.pom + +# install manpage and insert location of config file +install -d -m 755 %{buildroot}%{_mandir}/man1 +install -p -m 644 %{SOURCE2} %{buildroot}%{_mandir}/man1/%{name}.1 +sed -i "/.SH FILES/a .I %{_sysconfdir}/%{name}.ini" %{buildroot}%{_mandir}/man1/%{name}.1 + +%files +%license license/LICENSE.txt +%license license/THIRDPARTYREADME.txt +%doc README.md +%config(noreplace) %{_sysconfdir}/%{name}.ini +%{_jmcdir} +%{_mandir}/man1/%{name}.1* +%{_bindir}/%{name} +%{_datadir}/maven-poms/%{name} +%{_datadir}/pixmaps/%{name}.xpm +%{_datadir}/applications/%{name}.desktop + +%changelog +* Thu Apr 23 2020 Alex Macdonald - 7.1.1-5 +- Update jacoco removal patch +* Thu Apr 23 2020 Alex Macdonald - 7.1.1-4 +- Update to latest commit e67446b5fc9d +* Fri Apr 17 2020 Alex Macdonald - 7.1.1-3 +- include jmc-core in the directories to search in symlink_libs.sh +* Fri Apr 17 2020 Alex Macdonald - 7.1.1-2 +- Remove jacoco from the pom and disable coverage +* Thu Apr 16 2020 Alex Macdonald - 7.1.1-1 +- Updated to version 7.1.1 +* Thu Nov 14 2019 Jie Kang - 7.0.0-4 +- Fix requires +* Wed Nov 13 2019 Jie Kang - 7.0.0-3 +- Fix exclusions +* Wed Nov 13 2019 Jie Kang - 7.0.0-2 +- Update require and provide exclusions +* Tue Mar 12 2019 Jie Kang - 7.0.0-1 +- Initial package