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