Änderungen zur Unterstützung der Diskussion über TruffleRuby angefordert

bjfish · 28. April 2022 um 17:06

Ich möchte die folgenden 3 Änderungen beantragen, um die Ausführung von Discourse auf TruffleRuby zu unterstützen:

Angefragte Änderung
Aktualisieren Sie bin/rails, um Unicorn zu deaktivieren, wenn Fork nicht unterstützt wird.

Fehler

internal:core/kernel.rb:764:in `fork': fork is not available (NotImplementedError)

Dateien
bin/rails

github.com/discourse/discourse

bin/rails

b266a3696

#!/usr/bin/env ruby
# frozen_string_literal: true

if !ENV["RAILS_ENV"] && (ARGV[0] == "s" || ARGV[0] == "server")
  ENV["UNICORN_PORT"] ||= "3000"

  if ARGV[1] == "-p" && (port = ARGV[2].to_i) > 0
    ENV["UNICORN_PORT"] = port.to_s
  end

  ENV["RAILS_LOGS_STDOUT"] ||= "1"

  exec File.expand_path("unicorn", __dir__)
end

APP_PATH = File.expand_path('../config/application', __dir__)
require_relative '../config/boot'
require 'rails/commands'

Warum
TruffleRuby unterstützt das Forken nicht.

Vorschlag
Deaktivieren Sie Unicorn, wenn !Process.respond_to?(:fork)

========================================================

Angefragte Änderung
Deaktivieren Sie mini_profiler für TruffleRuby

Fehler

internal:core/kernel.rb:236:in `gem_original_require': cannot load such file -- stackprof (LoadError)

Dateien
config/initializers/006-mini_profiler.rb

github.com/discourse/discourse

config/initializers/006-mini_profiler.rb

b266a3696

# frozen_string_literal: true

# If Mini Profiler is included via gem
if Rails.configuration.respond_to?(:load_mini_profiler) && Rails.configuration.load_mini_profiler
  require 'rack-mini-profiler'
  require 'stackprof'

  begin
    require 'memory_profiler'
  rescue => e
    STDERR.put "#{e} failed to require mini profiler"
  end

  # initialization is skipped so trigger it
  Rack::MiniProfilerRails.initialize!(Rails.application)
end

if defined?(Rack::MiniProfiler) && defined?(Rack::MiniProfiler::Config)
  # note, we may want to add some extra security here that disables mini profiler in a multi hosted env unless user global admin
  #   raw_connection means results are not namespaced

This file has been truncated. show original

Warum
Stackprof wird derzeit auf TruffleRuby nicht unterstützt.

Vorschlag
Deaktivieren Sie das Laden von mini_profiler, wenn RUBY_ENGINE “TruffleRuby” ist.

========================================================

Angefragte Änderung
Verhindern Sie das Laden von rbtrace

Ähnliche Verwendungen verhindern bereits das Laden von rb_trace:

github.com/discourse/discourse

config/application.rb

b266a3696


      
                        discourse/tests/test-boot-rails.js
                        wizard/test/test_helper.js
                      }.include?(logical_path) ||
                      logical_path =~ /\/node_modules/ ||
                      logical_path =~ /\/dist/
                  end
                end
              end
          
              if ENV['RBTRACE'] == "1"
                require 'rbtrace'
              end
          
              config.generators do |g|
                g.test_framework :rspec, fixture: false
              end
            end
          end

github.com/discourse/discourse

config/environments/development.rb

b266a3696


      
            config.hosts.concat(hosts.split(","))
          end
          
          require 'middleware/turbo_dev'
          config.middleware.insert 0, Middleware::TurboDev
          require 'middleware/missing_avatars'
          config.middleware.insert 1, Middleware::MissingAvatars
          
          config.enable_anon_caching = false
          if RUBY_ENGINE == "ruby"
            require 'rbtrace'
          end
          
          if emails = GlobalSetting.developer_emails
            config.developer_emails = emails.split(",").map(&:downcase).map(&:strip)
          end
          
          if ENV["DISCOURSE_SKIP_CSS_WATCHER"] != "1" && (defined?(Rails::Server) || defined?(Puma) || defined?(Unicorn))
            require 'stylesheet/watcher'
            STDERR.puts "Starting CSS change watcher"
            @watcher = Stylesheet::Watcher.watch

Fehler
Failure/Error: require ‘rbtrace’

Vorschlag
Verwenden Sie eine der Methoden in den obigen Links, um das Laden von rbtrace zu verhindern.

Dateien
spec/rails_helper.rb

github.com/discourse/discourse

spec/rails_helper.rb

b266a3696


      
            SimpleCov.start 'rails' do
              add_group 'Libraries', /^\/lib\/(?!tasks).*$/
              add_group 'Scripts', 'script'
              add_group 'Serializers', 'app/serializers'
              add_group 'Services', 'app/services'
              add_group 'Tasks', 'lib/tasks'
            end
          end
          
          require 'rubygems'
          require 'rbtrace'
          require 'pry'
          require 'pry-byebug'
          require 'pry-rails'
          require 'fabrication'
          require 'mocha/api'
          require 'certified'
          require 'webmock/rspec'
          
          class RspecErrorTracker
            def self.last_exception=(ex)

gerhard · 29. April 2022 um 12:12

Das scheinen vernünftige Anfragen zu sein.

Übrigens: Hat sich etwas daran geändert, dass mini_racer auf TruffleRuby nicht unterstützt wird? Das ist ein großes Hindernis.

bjfish · 29. April 2022 um 18:55

Übrigens: Hat sich etwas daran geändert, dass mini_racer auf TruffleRuby nicht unterstützt wird? Das ist ein großes Hindernis.

Ich habe einen Branch mit den meisten mini_racer-Funktionen, der die Testsuite mit GraalJS für TruffleRuby besteht. Ich glaube, dass er alle für Discourse benötigten Funktionen abdeckt. Er muss nur noch überprüft werden und dann werde ich versuchen, ihn zu mergen. So habe ich ihn lokal ausgeführt: https://twitter.com/brandonfish/status/1486755128577253376

bjfish · 2. Mai 2022 um 15:44

Korrektur:
mini_profiler-Laden deaktivieren, wenn RUBY_ENGINE „truffleruby“ ist

sam · 3. Mai 2022 um 05:04

mini_profiler sollte immer noch in Ordnung sein. Ich würde empfehlen, dass wir rbtrace / stackprof usw. einfach nicht laden. Es gibt lose Abhängigkeiten.

mini_profiler kann in der Truffle-Welt immer noch viel Wert bieten, nichts Spezifisches für MRI daran.

gerhard · 4. Mai 2022 um 20:10

Hier ist ein PR mit den angeforderten Änderungen.

Ich konnte es immer noch nicht erfolgreich zum Laufen bringen, es gab einen java.lang.NullPointerException, aber das könnte nur daran liegen, dass ich mini_racer grob deaktiviert habe. Ich freue mich wirklich darauf, ein voll funktionsfähiges Discourse auf TruffleRuby laufen zu sehen.

Ich habe das nicht weiter untersucht, aber ich habe einen Fehler bemerkt, als ich nur das Laden von stackprof und memory_profiler deaktiviert habe. Daher habe ich mini_profiler vorerst komplett deaktiviert.

2022-05-04 21:48:13 +0200 Rack app ("POST /mini-profiler-resources/results" - (127.0.0.1)): #<SystemStackError: stack level too deep
        from StackOverflowCheckImpl.java:328:in `com.oracle.svm.core.graal.snippets.StackOverflowCheckImpl.newStackOverflowError0'>

Es könnte ein Nebeneffekt anderer Probleme gewesen sein. Ich schätze, wir könnten das untersuchen, wenn andere Probleme gelöst sind.

Thema		Antworten	Aufrufe
Update Gemfile fast_xor platform to :ruby Dev	5	1102	21. August 2022
Discourse on Truffleruby Dev	14	1252	16. Mai 2025
Disable page rendering stats on every page Support	3	882	27. Februar 2016
Discourse Build Error: “An error occurred while installing rbtrace (0.5.1), and Bundler cannot continue Installation	20	2565	25. Januar 2024
Install Discourse on macOS for development Developer Guides dev-install , how-to	101	112313	23. September 2025

Änderungen zur Unterstützung der Diskussion über TruffleRuby angefordert

Verwandte Themen