On Wednesday 26 March 2008 05:33, juiceman at freenetproject.org wrote: > Author: juiceman > Date: 2008-03-26 05:33:55 +0000 (Wed, 26 Mar 2008) > New Revision: 18793 > > Modified: > trunk/freenet/src/freenet/clients/http/StatisticsToadlet.java > trunk/freenet/src/freenet/node/Node.java > trunk/freenet/src/freenet/node/TextModeClientInterface.java > trunk/freenet/src/freenet/pluginmanager/PluginManager.java > Log: > Use System.getProperty("java.version") instead of System.getProperty("java.vm.version") to handle Sun's recent decision to change the naming convention of the client JVM: > C:\Documents and Settings\user>java -version > java version "1.6.0_05" > Java(TM) SE Runtime Environment (build 1.6.0_05-b13) > Java HotSpot(TM) Client VM (build 10.0-b19, mixed mode, sharing) > > Tested with Sun JVM, needs to be tested on other vendors JVMs
Does this work with older JVMs? > > Modified: trunk/freenet/src/freenet/clients/http/StatisticsToadlet.java > =================================================================== > --- trunk/freenet/src/freenet/clients/http/StatisticsToadlet.java > 2008-03-25 19:05:32 UTC (rev 18792) > +++ trunk/freenet/src/freenet/clients/http/StatisticsToadlet.java > 2008-03-26 05:33:55 UTC (rev 18793) > @@ -379,8 +379,8 @@ > jvmStatsList.addChild("li", l10n("threads", new String[] { "running", "max" }, > new String[] { > thousendPoint.format(threadCount), Integer.toString(stats.getThreadLimit()) })); > jvmStatsList.addChild("li", l10n("cpus", "count", Integer.toString(availableCpus))); > - jvmStatsList.addChild("li", l10n("jvmVendor", "vendor", System.getProperty("java.vm.vendor"))); > - jvmStatsList.addChild("li", l10n("jvmVersion", "version", System.getProperty("java.vm.version"))); > + jvmStatsList.addChild("li", l10n("jvmVendor", "vendor", System.getProperty("java.vendor"))); > + jvmStatsList.addChild("li", l10n("jvmVersion", "version", System.getProperty("java.version"))); > jvmStatsList.addChild("li", l10n("osName", "name", System.getProperty("os.name"))); > jvmStatsList.addChild("li", l10n("osVersion", "version", System.getProperty("os.version"))); > jvmStatsList.addChild("li", l10n("osArch", "arch", System.getProperty("os.arch"))); > > Modified: trunk/freenet/src/freenet/node/Node.java > =================================================================== > --- trunk/freenet/src/freenet/node/Node.java 2008-03-25 19:05:32 UTC (rev 18792) > +++ trunk/freenet/src/freenet/node/Node.java 2008-03-26 05:33:55 UTC (rev 18793) > @@ -599,7 +599,7 @@ > Node(PersistentConfig config, RandomSource random, > LoggingConfigHandler lc, NodeStarter ns, Executor executor) throws NodeInitException { > // Easy stuff > logMINOR = Logger.shouldLog(Logger.MINOR, this); > - String tmp = "Initializing Node using Freenet Build #"+Version.buildNumber()+" r"+Version.cvsRevision+" and freenet-ext Build #"+NodeStarter.extBuildNumber+" r"+NodeStarter.extRevisionNumber+" with "+System.getProperty("java.vm.vendor")+" JVM version "+System.getProperty("java.vm.version")+" running on "+System.getProperty("os.arch")+' '+System.getProperty("os.name")+' '+System.getProperty("os.version"); > + String tmp = "Initializing Node using Freenet Build #"+Version.buildNumber()+" r"+Version.cvsRevision+" and freenet-ext Build #"+NodeStarter.extBuildNumber+" r"+NodeStarter.extRevisionNumber+" with "+System.getProperty("java.vendor")+" JVM version "+System.getProperty("java.version")+" running on "+System.getProperty("os.arch")+' '+System.getProperty("os.name")+' '+System.getProperty("os.version"); > Logger.normal(this, tmp); > System.out.println(tmp); > collector = new IOStatisticCollector(); > @@ -1760,8 +1760,8 @@ > // Now check whether we are likely to get the EvilJVMBug. > // If we are running a Sun or Blackdown JVM, on Linux, and LD_ASSUME_KERNEL is not set, then we are. > > - String jvmVendor = System.getProperty("java.vm.vendor"); > - String jvmVersion = System.getProperty("java.vm.version"); > + String jvmVendor = System.getProperty("java.vendor"); > + String jvmVersion = System.getProperty("java.version"); > String osName = System.getProperty("os.name"); > String osVersion = System.getProperty("os.version"); > > @@ -1810,12 +1810,12 @@ > L10n.addL10nSubstitution(n, > "Node.buggyJVMWithLink", > new String[] { > "link", "/link", "version" }, > new String[] { > "<a href=\"/?_CHECKED_HTTP_=http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=4855795\">", > - "</a>", HTMLEncoder.encode(System.getProperty("java.vm.version")) }); > + "</a>", > HTMLEncoder.encode(System.getProperty("java.version")) }); > return n; > } > > public String getText() { > - return l10n("buggyJVM", > "version", System.getProperty("java.vm.version")); > + return l10n("buggyJVM", > "version", System.getProperty("java.version")); > } > > public String getTitle() { > @@ -1834,7 +1834,7 @@ > } else { > if(jvmVendor.startsWith("Free Software Foundation")) { > try { > - jvmVersion = > System.getProperty("java.vm.version").split(" ") [0].replaceAll("[.]",""); > + jvmVersion = > System.getProperty("java.version").split(" ") [0].replaceAll("[.]",""); > int jvmVersionInt = > Integer.parseInt(jvmVersion); > > if(jvmVersionInt <= 422 && > jvmVersionInt >= 100) // make sure that no bogus values cause true > > Modified: trunk/freenet/src/freenet/node/TextModeClientInterface.java > =================================================================== > --- trunk/freenet/src/freenet/node/TextModeClientInterface.java > 2008-03-25 19:05:32 UTC (rev 18792) > +++ trunk/freenet/src/freenet/node/TextModeClientInterface.java > 2008-03-26 05:33:55 UTC (rev 18793) > @@ -425,8 +425,8 @@ > sb.append("Maximum Java memory:\u00a0" + > SizeUtil.formatSize(maxJavaMem, true)+"\r\n"); > sb.append("Running threads:\u00a0" + thousendPoint.format(threadCount)+"\r\n"); > sb.append("Available CPUs:\u00a0" + availableCpus+"\r\n"); > - sb.append("JVM Vendor:\u00a0" + System.getProperty("java.vm.vendor")+"\r\n"); > - sb.append("JVM Version:\u00a0" + System.getProperty("java.vm.version")+"\r\n"); > + sb.append("JVM Vendor:\u00a0" + System.getProperty("java.vendor")+"\r\n"); > + sb.append("JVM Version:\u00a0" + System.getProperty("java.version")+"\r\n"); > sb.append("OS Name:\u00a0" + > System.getProperty("os.name")+"\r\n"); > sb.append("OS Version:\u00a0" + > System.getProperty("os.version")+"\r\n"); > sb.append("OS Architecture:\u00a0" + System.getProperty("os.arch")+"\r\n"); > > Modified: trunk/freenet/src/freenet/pluginmanager/PluginManager.java > =================================================================== > --- trunk/freenet/src/freenet/pluginmanager/PluginManager.java > 2008-03-25 19:05:32 UTC (rev 18792) > +++ trunk/freenet/src/freenet/pluginmanager/PluginManager.java > 2008-03-26 05:33:55 UTC (rev 18793) > @@ -163,7 +163,7 @@ > Logger.error(this, "Could not load > plugin " + filename + " : " + e, e); > System.err.println("Could not load > plugin " + filename + " : " + e); > e.printStackTrace(); > - String jvmVersion = > System.getProperty("java.vm.version"); > + String jvmVersion = > System.getProperty("java.version"); > if (jvmVersion.startsWith("1.4.") || > jvmVersion.equals("1.4")) { > System.err.println("Plugin " + > filename + " appears to require a later JVM"); > Logger.error(this, "Plugin " + > filename + " appears to require a later JVM"); > > _______________________________________________ > cvs mailing list > cvs at freenetproject.org > http://emu.freenetproject.org/cgi-bin/mailman/listinfo/cvs > > -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 189 bytes Desc: not available URL: <https://emu.freenetproject.org/pipermail/devl/attachments/20080328/3334804f/attachment.pgp>