Changeset 7318 in josm for trunk/src/org/openstreetmap/josm/tools

Timestamp:

2014-07-19T21:12:28+02:00 (11 years ago)

Author:

stoecker

Message:

see #10272 - improve version checks for RPM based distros

File:

• trunk/src/org/openstreetmap/josm/tools/PlatformHookUnixoid.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/tools/PlatformHookUnixoid.java

@@ -133 +133 @@
         try {
             boolean dpkg = Files.exists(Paths.get("/usr/bin/dpkg-query"));
-            boolean rpm  = Files.exists(Paths.get("/usr/bin/rpm"));
+            boolean rpm  = Files.exists(Paths.get("/bin/rpm"));
             if (dpkg || rpm) {
                 for (String packageName : packageNames) {
@@ -143 +143 @@
                     }
                     String version = Utils.execOutput(Arrays.asList(args));
-                    if (version != null) {
+                    if (version != null && !version.contains("not installed")) {
                         return packageName + ":" + version;
                     }
@@ -163 +163 @@
      */
     public String getJavaPackageDetails() {
-        switch(System.getProperty("java.home")) {
-        case "/usr/lib/jvm/java-7-openjdk-amd64/jre":
-        case "/usr/lib/jvm/java-7-openjdk-i386/jre":
-        case "/usr/lib64/jvm/java-1.7.0-openjdk-1.7.0/jre":
-        case "/usr/lib/jvm/java-1.7.0-openjdk-1.7.0/jre":
-            return getPackageDetails("openjdk-7-jre", "java-1_7_0-openjdk");
+        String home = System.getProperty("java.home");
+        if(home.contains("java-7-openjdk") || home.contains("java-1.7.0-openjdk")) {
+            return getPackageDetails("openjdk-7-jre", "java-1_7_0-openjdk", "java-1.7.0-openjdk");
         }
         return null;