JMeter on Mac OS X (jmeter directory not found)

[asfimport]

Holger Isenberg (Bug 41414):
Jmeter on Mac OS X does not find its directory as Java on Mac adds an
additional entry to the classpath while Jmeter expects only 1 JAR as
classpath entry.

Error message:

test$ bin/jmeter
Jmeter-Dir: /Users/test/workspace
java.lang.Throwable: Could not access /Users/test/workspace/lib
at org.apache.jmeter.NewDriver.<clinit>(NewDriver.java:80)
java.lang.Throwable: Could not access /Users/test/workspace/lib/ext
at org.apache.jmeter.NewDriver.<clinit>(NewDriver.java:80)
java.lang.Throwable: Could not access /Users/test/workspace/lib/junit
at org.apache.jmeter.NewDriver.<clinit>(NewDriver.java:80)
java.lang.ClassNotFoundException: org.apache.jmeter.JMeter
at java.net.URLClassLoader$1.run(URLClassLoader.java:200)
at java.security.AccessController.doPrivileged(Native Method)
at java.net.URLClassLoader.findClass(URLClassLoader.java:188)
at java.lang.ClassLoader.loadClass(ClassLoader.java:306)
at java.lang.ClassLoader.loadClass(ClassLoader.java:251)
at org.apache.jmeter.NewDriver.main(NewDriver.java:175)

Severity: normal
OS: Mac OS X 10.4

Blocks:

• Regular Expression Extractor does not work anymore #1850

[asfimport]

Holger Isenberg (migrated from Bugzilla):
Created attachment macosx-classpath.patch: fix to start on Mac OS X

macosx-classpath.patch

Index: src/core/org/apache/jmeter/NewDriver.java
===================================================================
--- src/core/org/apache/jmeter/NewDriver.java	(revision 497856)
+++ src/core/org/apache/jmeter/NewDriver.java	(working copy)
@@ -47,7 +47,7 @@
 
 		// Find JMeter home dir
 		StringTokenizer tok = new StringTokenizer(cp, File.pathSeparator);
-		if (tok.countTokens() == 1) {
+		if (tok.countTokens() == 1 || System.getProperty("os.name").toLowerCase().startsWith("mac os x") && tok.countTokens()  == 2) {
 			File jar = new File(tok.nextToken());
 			try {
 				jmDir = jar.getCanonicalFile().getParentFile().getParent();

[asfimport]

Sebb (migrated from Bugzilla):
Thanks, now fixed in SVN, so will appear in the next nightly build in:

http://people.apache.org/builds/jakarta-jmeter/nightly/