Remove leading space in Ruby help message. (#145)

metalbear-co · Sep 11, 2023 · b74199b · b74199b
1 parent 6efc6f3
commit b74199b
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 18 deletions.
diff --git a/changelog.d/+leading-space-ruby-error.fixed.md b/changelog.d/+leading-space-ruby-error.fixed.md
@@ -0,0 +1 @@
+Remove leading space from non-RVM Ruby help message.
diff --git a/.../main/kotlin/com/metalbear/mirrord/products/rubymine/RubyMineRunConfigurationExtension.kt b/.../main/kotlin/com/metalbear/mirrord/products/rubymine/RubyMineRunConfigurationExtension.kt
@@ -57,24 +57,24 @@ class RubyMineRunConfigurationExtension : RubyRunConfigurationExtension() {
                 cmdLine.exePath = patched
             }
 
-            // TODO: would be nice to have a more robust RVM detection mechanism.
-            val isRvm = cmdLine.exePath.contains("/.rvm/rubies/")
-            if (isMac && !isRvm) {
-                val e = MirrordError(
-                    "At the moment, only RVM Rubies are supported by mirrord on RubyMine on macOS, due to SIP.",
-                    " Support for other Rubies is tracked on " +
-                        "https://github.com/metalbear-co/mirrord-intellij/issues/134."
-                )
-                e.showHelp(configuration.project)
-                throw e
-            }
-
-            if (isMac && isRvm) {
-                val path = createTempFile("mirrord-ruby-launcher-", ".sh")
-                // Using patched exe inside the launcher script.
-                path.writeText("DYLD_INSERT_LIBRARIES=${currentEnv["DYLD_INSERT_LIBRARIES"]} ${cmdLine.exePath} $@")
-                cmdLine.exePath = path.pathString
-                path.toFile().setExecutable(true)
+            if (isMac) {
+                // TODO: would be nice to have a more robust RVM detection mechanism.
+                val isRvm = cmdLine.exePath.contains("/.rvm/rubies/")
+                if (isRvm) {
+                    val path = createTempFile("mirrord-ruby-launcher-", ".sh")
+                    // Using patched exe inside the launcher script.
+                    path.writeText("DYLD_INSERT_LIBRARIES=${currentEnv["DYLD_INSERT_LIBRARIES"]} ${cmdLine.exePath} $@")
+                    cmdLine.exePath = path.pathString
+                    path.toFile().setExecutable(true)
+                } else {
+                    val e = MirrordError(
+                        "At the moment, only RVM Rubies are supported by mirrord on RubyMine on macOS, due to SIP.",
+                        "Support for other Rubies is tracked on " +
+                            "https://github.com/metalbear-co/mirrord-intellij/issues/134."
+                    )
+                    e.showHelp(configuration.project)
+                    throw e
+                }
             }
         }
     }
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		Remove leading space from non-RVM Ruby help message.