Review Request: JMC-6308 JMC 7.0.0 update ide and rcp landing page with correct url's
Sharath Ballal
sharath.ballal at oracle.com
Tue Jan 8 11:34:06 UTC 2019
Hi Prem,
My comments below.
1. In all the files, Copyright year should not be replaced to 2019 instead should be changed to:
Copyright (c) 2018, 2019, Oracle and/or its affiliates. All rights reserved.
File application/org.openjdk.jmc.updatesite.ide/src/main/resources/index.html
2. Not sure about this. I think JMC still works with Eclipse 4.7, can you pls verify?
- <li>Requires Eclipse 4.7 or later.</li>
+ <li>Requires Eclipse 4.8 or later.</li>
- JDK Mission Control is a set of plug-ins for Eclipse 4.7 or later.<br><br>
+ JDK Mission Control is a set of plug-ins for Eclipse 4.8 or later.<br><br>
3. No need to change the OTN link to http://jdk.java.net/jmc/ as the final GA bundle will be available in OTN.
- Release Notes can be found on the <a href="https://www.oracle.com/technetwork/java/javaseproducts/mission-control/index.html"> JDK Mission Control home page</a><br>
+ Release Notes can be found on the <a href="http://jdk.java.net/jmc/">JDK Mission Control home page</a><br>
4. No need to remove these lines
- <b>Download and Install</b><br>
- JDK Mission Control is available free of charge for development.
- For more information, see the <a href="https://www.oracle.com/technetwork/java/javaseproducts/mission-control/index.html">JDK Mission Control home page</a>.<br><br>
5. No need to remove these lines. Updatesite will be available in this link going forward.
- <img src="images/bullet-arrow-red.gif" alt="Bullet">
- <a href="http://download.oracle.com/technology/products/missioncontrol/updatesites/openjdk/7.0.0/eclipse/jmc-updatesite-7.0.0.zip">
- Download for Offline Installation
- </a><br>
File application/org.openjdk.jmc.updatesite.ide/src/main/resources/update-site-instructions/index.html
6. The new link doesn't seem to work
- <b>http://download.oracle.com/technology/products/missioncontrol/updatesites/openjdk/7.0.0/eclipse/</b><br><br>
+ <b>https://download.oracle.com/technology/products/missioncontrol/updatesites/openjdk/7.0.0/ide/</b><br><br>
7. Same as earlier comment, pls check if 4.7 is supported or not.
- Before starting, make sure that you have downloaded and installed Eclipse 4.7 or later.
+ Before starting, make sure that you have downloaded and installed Eclipse 4.8 or later.
File application/org.openjdk.jmc.updatesite.rcp/src/main/resources/index.html
8. No need to change the OTN link to http://jdk.java.net/jmc/ as the final GA bundle will be available in OTN.
- <a href="https://www.oracle.com/technetwork/java/javaseproducts/mission-control/index.html">
+ <a href="http://jdk.java.net/jmc/">
Thanks,
Sharath
-----Original Message-----
From: Prem Balakrishnan
Sent: Thursday, January 03, 2019 6:41 PM
To: jmc-dev at openjdk.java.net
Subject: Review Request: JMC-6308 JMC 7.0.0 update ide and rcp landing page with correct url's
Hi All,
Please review the fix for HYPERLINK "https://bugs.openjdk.java.net/browse/JMC-6308"JMC-6308
Webrev: http://cr.openjdk.java.net/~pkbalakr/jmc/6308/webrev.00/
updated "Mission control" to "JDK Mission control" and JMC download location.
Regards,
Prem
More information about the jmc-dev
mailing list