From 1f3058cad06c26f662c0f1f34ff707a29fd6bb2c Mon Sep 17 00:00:00 2001 From: Joris van Lieshout Date: Tue, 1 May 2018 12:22:00 +0200 Subject: [PATCH] Make the classpath seperator in jmxfetch adjust to the os Linux uses a colon (:) and Windows a semicolon (;). os.pathsep knows this. --- jmxfetch.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/jmxfetch.py b/jmxfetch.py index 7aab863100..3072734ae5 100644 --- a/jmxfetch.py +++ b/jmxfetch.py @@ -240,11 +240,11 @@ def _start(self, path_to_java, java_run_opts, jmx_checks, command, reporter, too classpath = path_to_jmxfetch if tools_jar_path is not None: - classpath = r"%s:%s" % (tools_jar_path, classpath) + classpath = r"%s%s%s" % (tools_jar_path, os.pathsep, classpath) if custom_jar_paths: - classpath = r"%s:%s" % (':'.join(custom_jar_paths), classpath) + classpath = r"%s%s%s" % (os.pathsep.join(custom_jar_paths), os.pathsep, classpath) if self.config_jar_path: - classpath = r"%s:%s" % (self.config_jar_path, classpath) + classpath = r"%s%s%s" % (self.config_jar_path, os.pathsep, classpath) subprocess_args = [ path_to_java, # Path to the java bin