Dont save/restore $! in the teardown method used by embedding API

* This seems to be messing with the embedding API based testing wherein excluded failures are being reported (in Travis testing but reproducible locally as well).
jruby · Feb 14, 2015 · 5f3ff7c · 5f3ff7c
1 parent 1b5b3d7
commit 5f3ff7c
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/core/src/main/java/org/jruby/Ruby.java b/core/src/main/java/org/jruby/Ruby.java
@@ -3228,7 +3228,7 @@ public void tearDown(boolean systemExit) {
 
         while (!atExitBlocks.empty()) {
             RubyProc proc = atExitBlocks.pop();
-            IRubyObject oldExc = context.runtime.getGlobalVariables().get("$!"); // Save $!
+            // IRubyObject oldExc = context.runtime.getGlobalVariables().get("$!"); // Save $!
             try {
                 proc.call(getCurrentContext(), IRubyObject.NULL_ARRAY);
             } catch (RaiseException rj) {
@@ -3244,7 +3244,7 @@ public void tearDown(boolean systemExit) {
                     }
                 }
                 // Reset $! now that rj has been handled
-                context.runtime.getGlobalVariables().set("$!", oldExc);
+                // context.runtime.getGlobalVariables().set("$!", oldExc);
             }
         }