Title: [289752] trunk/Tools
Revision
289752
Author
[email protected]
Date
2022-02-14 12:29:39 -0800 (Mon, 14 Feb 2022)

Log Message

run-jsc-stress-tests reports "in `<module:URI>': uninitialized class variable @@schemes in URI (NameError)" with Ruby 3.1
https://bugs.webkit.org/show_bug.cgi?id=236281

Reviewed by Yusuke Suzuki.

Ruby 3.1.0 removed @@schemes and added URI.register_scheme.

* Scripts/run-jsc-stress-tests: Added `register_scheme` class
method if not exists. Use the register_scheme method for ssh.

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (289751 => 289752)


--- trunk/Tools/ChangeLog	2022-02-14 20:12:50 UTC (rev 289751)
+++ trunk/Tools/ChangeLog	2022-02-14 20:29:39 UTC (rev 289752)
@@ -1,3 +1,15 @@
+2022-02-14  Fujii Hironori  <[email protected]>
+
+        run-jsc-stress-tests reports "in `<module:URI>': uninitialized class variable @@schemes in URI (NameError)" with Ruby 3.1
+        https://bugs.webkit.org/show_bug.cgi?id=236281
+
+        Reviewed by Yusuke Suzuki.
+
+        Ruby 3.1.0 removed @@schemes and added URI.register_scheme.
+
+        * Scripts/run-jsc-stress-tests: Added `register_scheme` class
+        method if not exists. Use the register_scheme method for ssh.
+
 2022-02-14  Tim Nguyen  <[email protected]>
 
         Don't perform layout in WKBundlePagePostSynchronousMessageForTesting

Modified: trunk/Tools/Scripts/run-jsc-stress-tests (289751 => 289752)


--- trunk/Tools/Scripts/run-jsc-stress-tests	2022-02-14 20:12:50 UTC (rev 289751)
+++ trunk/Tools/Scripts/run-jsc-stress-tests	2022-02-14 20:29:39 UTC (rev 289752)
@@ -40,7 +40,12 @@
     class SSH < Generic
         DEFAULT_PORT = 22
     end
-    @@schemes['SSH'] = SSH
+    unless defined?(self.register_scheme)
+      def self.register_scheme(scheme, klass)
+        @@schemes[scheme] = klass
+      end
+    end
+    register_scheme 'SSH', SSH
 end
 
 class String
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to