Skip to content

Remove deprecated taint followup#277

Merged
tenderlove merged 3 commits intosparklemotion:masterfrom
casperisfine:remove_deprecated_taint-followup
Dec 18, 2019
Merged

Remove deprecated taint followup#277
tenderlove merged 3 commits intosparklemotion:masterfrom
casperisfine:remove_deprecated_taint-followup

Conversation

@casperisfine
Copy link
Copy Markdown

This is a followup on @kamipo's #273

It also avoid calling rb_check_safe_obj on MRI 2.7. That warning is totally spamming Active Record's test suite.

cc @tenderlove since apparently you're maintainer.

cc @rafaelfranca @Edouard-chin.

NB: Based on @kamipo's PR I expect half of the CI jobs to fail, I'll try to make them pass by pushing another commit.

kamipo and others added 2 commits November 21, 2019 09:06
The Ruby core team decided to deprecate the taint mechanism in Ruby 2.7
and will remove that in Ruby 3.

https://bugs.ruby-lang.org/issues/16131
ruby/ruby#2476

In Ruby 2.7, `Object#{taint,untaint,trust,untrust}` and related
functions in the C-API no longer have an effect (all objects are always
considered untainted), and are now warned deprecation message.

https://buildkite.com/rails/rails/builds/65054#5aa2db21-569d-4202-99cd-a8323cab583e/6-8
@casperisfine casperisfine force-pushed the remove_deprecated_taint-followup branch from 2b40d08 to 0b5e04e Compare December 18, 2019 15:42
@casperisfine casperisfine force-pushed the remove_deprecated_taint-followup branch from 0b5e04e to 19d0486 Compare December 18, 2019 15:49
@tenderlove
Copy link
Copy Markdown
Member

Thank you!

@tenderlove tenderlove merged commit 3a9d52f into sparklemotion:master Dec 18, 2019
@kbrock
Copy link
Copy Markdown

kbrock commented Dec 19, 2019

Thank you

composerinteralia added a commit to thoughtbot/factory_bot that referenced this pull request Jul 15, 2020
Ruby 3 is not compatible with Rails 5.0. Rails 5.0 locked to
sqlite3 "~> 1.3.6", but only sqlite3 "<= 1.4.2" is compatible with Ruby
3. [Older versions of sqlite3 referenced `rb_check_safe_obj`][sqlite3],
which raised a warning in Ruby 2.7 and will fail in Ruby 3.0.

Since this combination of tests will always fail, this commit updates
Travis to stop running it.

[sqlite3]: sparklemotion/sqlite3-ruby#277
composerinteralia added a commit to thoughtbot/factory_bot that referenced this pull request Jul 15, 2020
Ruby 3 is not compatible with Rails 5.0. Rails 5.0 locked to
sqlite3 "~> 1.3.6", but only sqlite3 "<= 1.4.2" is compatible with Ruby
3. [Older versions of sqlite3 referenced `rb_check_safe_obj`][sqlite3],
which raised a warning in Ruby 2.7 and will fail in Ruby 3.0.

Since this combination of tests will always fail, this commit updates
Travis to stop running it.

[sqlite3]: sparklemotion/sqlite3-ruby#277
connorchris831 pushed a commit to connorchris831/factory_bot that referenced this pull request Dec 13, 2022
Ruby 3 is not compatible with Rails 5.0. Rails 5.0 locked to
sqlite3 "~> 1.3.6", but only sqlite3 "<= 1.4.2" is compatible with Ruby
3. [Older versions of sqlite3 referenced `rb_check_safe_obj`][sqlite3],
which raised a warning in Ruby 2.7 and will fail in Ruby 3.0.

Since this combination of tests will always fail, this commit updates
Travis to stop running it.

[sqlite3]: sparklemotion/sqlite3-ruby#277
SriHarshaxi pushed a commit to SriHarshaxi/factory_bot that referenced this pull request Mar 17, 2023
Ruby 3 is not compatible with Rails 5.0. Rails 5.0 locked to
sqlite3 "~> 1.3.6", but only sqlite3 "<= 1.4.2" is compatible with Ruby
3. [Older versions of sqlite3 referenced `rb_check_safe_obj`][sqlite3],
which raised a warning in Ruby 2.7 and will fail in Ruby 3.0.

Since this combination of tests will always fail, this commit updates
Travis to stop running it.

[sqlite3]: sparklemotion/sqlite3-ruby#277
gold-crow-997 added a commit to gold-crow-997/factory-bot that referenced this pull request Jun 8, 2025
Ruby 3 is not compatible with Rails 5.0. Rails 5.0 locked to
sqlite3 "~> 1.3.6", but only sqlite3 "<= 1.4.2" is compatible with Ruby
3. [Older versions of sqlite3 referenced `rb_check_safe_obj`][sqlite3],
which raised a warning in Ruby 2.7 and will fail in Ruby 3.0.

Since this combination of tests will always fail, this commit updates
Travis to stop running it.

[sqlite3]: sparklemotion/sqlite3-ruby#277
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants