bminstall script for mac

[rafaelnferreira]

Hello, an amendment has been made on the bminstall.sh script which was not anymore compatible with current OS. By testing with Jdks from 1.6 to 1.8 I could not understand why a switch was introduced in an attempt to load Classes.jar (maybe that was the case in really old jdk releases prior to 1.6).

My reference installation structure for the test:

ls /Library/Java/JavaVirtualMachines/1.6.0.jdk/Contents/Home/
bin bundle lib man

ls /Library/Java/JavaVirtualMachines/jdk1.7.0_67.jdk/Contents/Home/
COPYRIGHT THIRDPARTYLICENSEREADME-JAVAFX.txt db lib src.zip
LICENSE THIRDPARTYLICENSEREADME.txt include man
README.html bin jre release

ls /Library/Java/JavaVirtualMachines/jdk1.8.0_60.jdk/Contents/Home/
COPYRIGHT THIRDPARTYLICENSEREADME-JAVAFX.txt db jre release
LICENSE THIRDPARTYLICENSEREADME.txt include lib src.zip
README.html bin javafx-src.zip man

[adinn]

Hi rafael,

I'm not sure exactly what this pull request is meant to do. The script attempts to make sure that classes needed for the install operation to work are available in the classpath. These classes were in classes.jar on releases of Java for Mac where the script was tested. If your release does not contain that jar then it will still need these classes to be available in order for the script to work.

Are you suggesting that all current releases of Java for Mac have relocated these classes to another jar? If so then can you please identify for me where your release provides class com.sun.tools.attach.VirtualMachine.

If your release does not contain these classes then that may well be because it is a JRE release (i.e. a Java runtime) rather than a full JDK release (i.e. a development release). You need a full JDK release in order to be able to use the install script.

[adinn]

I cannot proceed with this pull without further input. Given the lack of any response I am rejecting it.