Feature #30356: Drop Ruby 2.2 support - Redmine

Actions

Copy link

Feature #30356

closed

Drop Ruby 2.2 support

Added by Go MAEDA almost 6 years ago. Updated almost 6 years ago.

Status:

Closed

Priority:

Normal

Assignee:

Go MAEDA

Category:

Ruby support

Target version:

4.1.0

Start date:

Due date:

% Done:

Estimated time:

Resolution:

Fixed

Description

Currently, Redmine 4.0 supports Ruby 2.2. But the latest nokogiri gem 1.10.0 supports 2.3 and later, does not 2.2. So, we cannot update nogogiri gem to 1.10.0 as long as Redmine officially supports Ruby 2.2.

However, it is necessary to update nokogiri to 1.10.0 in order to support Ruby 2.6 because older nokogiri does not support Ruby 2.6 on Windows (https://github.com/sparklemotion/nokogiri/pull/1850).

We have to choose the Ruby version we support, 2.2 or 2.6. Needless to say, Redmine 4.0 should support Ruby 2.6 rather than retired 2.2.

Files

remove-code-for-ruby-two-point-two.diff (1.18 KB) remove-code-for-ruby-two-point-two.diff

Go MAEDA, 2019-02-03 06:54

Related issues

Actions

Copy link

Updated by Go MAEDA almost 6 years ago

Related to Patch #30241: Update nokogiri gem (~> 1.10.0) added

Actions

Copy link

Updated by Go MAEDA almost 6 years ago

Blocks Feature #30118: Ruby 2.6 support added

Actions

Copy link

Updated by Pavel Rosický almost 6 years ago

What about just locking nokogiri 1.9.0 for Ruby 2.2?

Actions

Copy link

Updated by Go MAEDA almost 6 years ago

Pavel Rosický wrote:

What about just locking nokogiri 1.9.0 for Ruby 2.2?

You are right. I quite forgot about it. This should work.

Index: Gemfile
===================================================================
--- Gemfile    (revision 17777)
+++ Gemfile    (working copy)
@@ -14,7 +14,7 @@
 gem "mail", "~> 2.7.1" 
 gem "csv", "~> 3.0.1" if RUBY_VERSION >= "2.3" && RUBY_VERSION < "2.6" 

-gem "nokogiri", "~> 1.9.0" 
+gem "nokogiri", (RUBY_VERSION >= "2.3" ? "~> 1.10.0" : "~> 1.9.1")
 gem "i18n", "~> 0.7.0" 

 # Windows does not include zoneinfo files, so bundle the tzinfo-data gem

Actions

Copy link

Updated by Marius BĂLTEANU almost 6 years ago

In my opinion, we should drop Ruby 2.2 support in 4.1.0.

Actions

Copy link

Updated by Go MAEDA almost 6 years ago

Assignee set to Jean-Philippe Lang
Target version set to 4.0.1

Jean-Philippe Lang is considering to drop Ruby 2.2 support in 4.0-stable (see #30161#note-3). I will leave this issue to him.

Actions

Copy link

Updated by Marius BĂLTEANU almost 6 years ago

Go MAEDA wrote:

Jean-Philippe Lang is considering to drop Ruby 2.2 support in 4.0-stable (see #30161#note-3). I will leave this issue to him.

Agree.

I proposed 4.1.0 because 4.0.1 will be a maintenance release (mostly with bug fixes and small changes) and dropping support for Ruby 2.2 could have an important impact:

Server administrators will need to update the Ruby version on installations with Ruby 2.2.
The codebase of the plugins compatible with 4.0 must be rechecked by their developers to support 4.0.1.

Discussions on the same topic: #25538

Actions

Copy link

Updated by Go MAEDA almost 6 years ago

We don't have to drop Ruby 2.2 support in a hurry for now. And I think there isn't any harm to commit the patch #30356#note-4 to the trunk and 4.0-stable.

I would like to commit #30356#note-4 as a part of #30241. What do you all think?

Actions

Copy link

Updated by Marius BĂLTEANU almost 6 years ago

Go MAEDA wrote:

We don't have to drop Ruby 2.2 support in a hurry for now. And I think there isn't any harm to commit the patch #30356#note-4 to the trunk and 4.0-stable.

I would like to commit #30356#note-4 as a part of #30241. What do you all think?

I’m totally agree.

Actions

Copy link

#10