Patch #29441


Remove code related to JRuby and unsupported Ruby versions

Added by Marius BALTEANU almost 5 years ago. Updated almost 4 years ago.

Code cleanup/refactoring
Target version:
Start date:
Due date:
% Done:


Estimated time:


code_cleanup.patch (4.44 KB) code_cleanup.patch Marius BALTEANU, 2018-08-26 14:27
29411-gemfile.diff (737 Bytes) 29411-gemfile.diff Go MAEDA, 2018-11-27 12:17

Related issues

Related to Redmine - Defect #31387: Don't rescue Exception classClosedGo MAEDA

Related to Redmine - Patch #32937: test_revisions_latin_1_identifier should be skipped on WindowsClosedGo MAEDA

Actions #1

Updated by Go MAEDA almost 5 years ago

  • Target version changed from Candidate for next major release to 4.1.0
Actions #2

Updated by Go MAEDA over 4 years ago

  • Assignee set to Jean-Philippe Lang
Actions #3

Updated by Jeremy Bailey over 4 years ago

It might be interesting to mention hear that the TruffleRuby team is making rapid progress, and I’ve reported a few issues they are actively working on that will eventually allow Redmine (and other production Rails apps) to run, considerably faster than MRI. TruffleRuby is based on JRuby but adds a lot to it. I hope Redmine will officially support it when compatibility is achieved.

Actions #4

Updated by Holger Just over 4 years ago

TruffleRuby is way less mature that JRuby. Also, it doesn't solve the issue that many external gems used by Redmine do not fully support JRuby (and even less likely TruffleRuby). Thus, while Truffle is interesting, full support for it likely requires a long way to go.

Actions #5

Updated by Pavel Rosický over 4 years ago

lower your expectations, TruffleRuby will definitely excel in numeric operations (micro-benchmarks), partial evaluation is also an interesting technique, but a webserver is usually limited by IO (handling requests, database access etc.) so I don't expect it'll be MUCH faster.

Actions #6

Updated by Go MAEDA over 4 years ago

Gemfile also has JRuby related code that can be removed.

Actions #7

Updated by Go MAEDA about 4 years ago

Actions #8

Updated by Jean-Philippe Lang almost 4 years ago

  • Status changed from New to Closed

Committed, thanks.

Actions #9

Updated by Go MAEDA over 3 years ago

  • Related to Patch #32937: test_revisions_latin_1_identifier should be skipped on Windows added

Also available in: Atom PDF