public inbox for git-commits@fedoraproject.org
help / color / mirror / Atom feed
To: git-commits@fedoraproject.org
Subject: [rpms/rubygem-activesupport] rawhide: Update to Active Support 8.1.2.
Date: Wed, 29 Jul 2026 14:37:41 GMT	[thread overview]
Message-ID: <178533586173.1.7479535949414407184.rpms-rubygem-activesupport-8bae32173114@fedoraproject.org> (raw)

            A new commit has been pushed.

            Repo   : rpms/rubygem-activesupport
            Branch : rawhide
            Commit : 8bae321731149b2d42538e4dab8650e38a1e597f
            Author : Vít Ondruch <vondruch@redhat.com>
            Date   : 2026-07-29T16:10:07+02:00
            Stats  : +60/-185 in 6 file(s)
            URL    : https://src.fedoraproject.org/rpms/rubygem-activesupport/c/8bae321731149b2d42538e4dab8650e38a1e597f?branch=rawhide

            Log:
            Update to Active Support 8.1.2.

Related: rhzb#2405582

---
diff --git a/rubygem-activesupport-8.0.3-Always-pass-default-precision-to-BigDecimal-when-parsing.patch b/rubygem-activesupport-8.0.3-Always-pass-default-precision-to-BigDecimal-when-parsing.patch
deleted file mode 100644
index 7027f9a..0000000
--- a/rubygem-activesupport-8.0.3-Always-pass-default-precision-to-BigDecimal-when-parsing.patch
+++ /dev/null
@@ -1,24 +0,0 @@
-From 283d96ea53f45eedf09a31bef739575df96e87df Mon Sep 17 00:00:00 2001
-From: zzak <zzak@hey.com>
-Date: Sun, 5 Oct 2025 10:31:09 +0900
-Subject: [PATCH] Always pass default precision to BigDecimal when parsing
- Float in XmlMini
-
-https://github.com/ruby/bigdecimal/blob/cb2458bde33bf90a8364b58d53e8948a7ba555ea/ext/bigdecimal/bigdecimal.c#L2747-L2749
----
- activesupport/lib/active_support/xml_mini.rb | 2 ++
- 1 file changed, 2 insertions(+)
-
-diff --git a/activesupport/lib/active_support/xml_mini.rb b/activesupport/lib/active_support/xml_mini.rb
-index 2c2b8185b1b80..c6d7ce5b7c251 100644
---- a/activesupport/lib/active_support/xml_mini.rb
-+++ b/activesupport/lib/active_support/xml_mini.rb
-@@ -74,6 +74,8 @@ def content_type
-         "decimal"      => Proc.new do |number|
-           if String === number
-             number.to_d
-+          elsif Float === number
-+            BigDecimal(number, 0)
-           else
-             BigDecimal(number)
-           end

diff --git a/rubygem-activesupport-pr56202-minitest6-rescue-loaderror.patch b/rubygem-activesupport-pr56202-minitest6-rescue-loaderror.patch
index d755a4e..ff8eb9b 100644
--- a/rubygem-activesupport-pr56202-minitest6-rescue-loaderror.patch
+++ b/rubygem-activesupport-pr56202-minitest6-rescue-loaderror.patch
@@ -1,9 +1,9 @@
 --- activesupport-8.0.3/lib/active_support/testing/autorun.rb.orig	2025-12-30 16:47:16.780346179 +0900
 +++ activesupport-8.0.3/lib/active_support/testing/autorun.rb	2025-12-30 16:52:01.482400639 +0900
-@@ -8,5 +8,8 @@ require "minitest"
- # used in some cases. This conditional can probably go after the bump
- # is complete? ... but could still fail for developers working w/
- # multiple versions installed.
+@@ -6,5 +6,8 @@ require "minitest"
+ # unbundled environment, which triggers MT5 usage. This conditional may
+ # be removable after the version bump, though it currently safeguards
+ # against issues in environments with multiple versions installed.
 -Minitest.load :rails if Minitest.respond_to? :load
 +begin
 +  Minitest.load :rails if Minitest.respond_to? :load

diff --git a/rubygem-activesupport-pr56202-minitest6.patch b/rubygem-activesupport-pr56202-minitest6.patch
deleted file mode 100644
index 00209d5..0000000
--- a/rubygem-activesupport-pr56202-minitest6.patch
+++ /dev/null
@@ -1,129 +0,0 @@
-From 9da4460ad0e71e5c3de32566ffbc302674b1f76e Mon Sep 17 00:00:00 2001
-From: Ryan Davis <ryand-ruby@zenspider.com>
-Date: Thu, 20 Nov 2025 12:50:33 -0800
-Subject: [PATCH 1/5] MT6: Load rails plugin
-
----
- activesupport/lib/active_support/testing/autorun.rb | 7 +++++++
- 1 file changed, 7 insertions(+)
-
-diff --git a/activesupport/lib/active_support/testing/autorun.rb b/activesupport/lib/active_support/testing/autorun.rb
-index d5d5fc7ae8e45..068aac0b487e0 100644
---- a/activesupport/lib/active_support/testing/autorun.rb
-+++ b/activesupport/lib/active_support/testing/autorun.rb
-@@ -2,4 +2,11 @@
- 
- require "minitest"
- 
-+##
-+# I shouldn't need this respond_to check but some tests are running
-+# sub-process tests in an unbundled environment, causing MT5 to be
-+# used in some cases. This conditional can probably go after the bump
-+# is complete? ... but could still fail for developers working w/
-+# multiple versions installed.
-+Minitest.load :rails if Minitest.respond_to? :load
- Minitest.autorun
-
-From 831f0f96d0f9c132b28d3fa22ab82806115747b0 Mon Sep 17 00:00:00 2001
-From: Ryan Davis <ryand-ruby@zenspider.com>
-Date: Thu, 20 Nov 2025 14:16:51 -0800
-Subject: [PATCH 3/5] MT6: implementation fixes
-
-MT6 changes the way assertion messages work. Now, if a proc is passed
-in for the message, it wins untouched. So for the rails assertions
-that want to have diffs shown while calling assert_equal with a
-message proc, the proc needs to call diff itself. This feels redundant
-to me, but not my call.
-
-And since the procs win now, they need to provide their own periods at
-the end of the text.
----
- activesupport/lib/active_support/testing/assertions.rb    | 8 +++++---
- .../lib/active_support/testing/parallelization/worker.rb  | 6 +++++-
- 2 files changed, 10 insertions(+), 4 deletions(-)
-
-diff --git a/activesupport/lib/active_support/testing/assertions.rb b/activesupport/lib/active_support/testing/assertions.rb
-index 178b5b350abba..4f2c6b105bc19 100644
---- a/activesupport/lib/active_support/testing/assertions.rb
-+++ b/activesupport/lib/active_support/testing/assertions.rb
-@@ -124,7 +124,8 @@ def assert_difference(expression, *args, &block)
-           actual = exp.call
-           rich_message = -> do
-             code_string = code.respond_to?(:call) ? _callable_to_source_string(code) : code
--            error = "`#{code_string}` didn't change by #{diff}, but by #{actual - before_value}"
-+            error = "`#{code_string}` didn't change by #{diff}, but by #{actual - before_value}."
-+            error = "#{error}\n#{diff before_value + diff, actual}" if Minitest::VERSION > "6"
-             error = "#{message}.\n#{error}" if message
-             error
-           end
-@@ -228,7 +229,7 @@ def assert_changes(expression, message = nil, from: UNTRACKED, to: UNTRACKED, &b
-         rich_message = -> do
-           code_string = expression.respond_to?(:call) ? _callable_to_source_string(expression) : expression
-           error = "`#{code_string}` didn't change"
--          error = "#{error}. It was already #{to.inspect}" if before == to
-+          error = "#{error}. It was already #{to.inspect}." if before == to
-           error = "#{message}.\n#{error}" if message
-           error
-         end
-@@ -296,8 +297,9 @@ def assert_no_changes(expression, message = nil, from: UNTRACKED, &block)
- 
-         rich_message = -> do
-           code_string = expression.respond_to?(:call) ? _callable_to_source_string(expression) : expression
--          error = "`#{code_string}` changed"
-+          error = "`#{code_string}` changed."
-           error = "#{message}.\n#{error}" if message
-+          error = "#{error}\n#{diff before, after}" if Minitest::VERSION > "6"
-           error
-         end
- 
-diff --git a/activesupport/lib/active_support/testing/parallelization/worker.rb b/activesupport/lib/active_support/testing/parallelization/worker.rb
-index d008277f8924c..daad6ce659103 100644
---- a/activesupport/lib/active_support/testing/parallelization/worker.rb
-+++ b/activesupport/lib/active_support/testing/parallelization/worker.rb
-@@ -49,7 +49,11 @@ def perform_job(job)
-           set_process_title("#{klass}##{method}")
- 
-           result = klass.with_info_handler reporter do
--            Minitest.run_one_method(klass, method)
-+            if Minitest.respond_to?(:run_one_method) then
-+              Minitest.run_one_method(klass, method)
-+            else
-+              klass.new(method).run
-+            end
-           end
- 
-           safe_record(reporter, result)
-
-From 2c1ca03402de9a5bc4e482da739e39f5bd47f0b3 Mon Sep 17 00:00:00 2001
-From: Ryan Davis <ryand-ruby@zenspider.com>
-Date: Thu, 20 Nov 2025 15:34:55 -0800
-Subject: [PATCH 4/5] MT6: test fixes
-
-Mostly minor and mostly centered around whether there are diffs.
----
- actionpack/test/dispatch/routing/route_set_test.rb | 4 ++--
- activesupport/test/test_case_test.rb               | 4 ++--
- 2 files changed, 4 insertions(+), 4 deletions(-)
-
-diff --git a/activesupport/test/test_case_test.rb b/activesupport/test/test_case_test.rb
-index 7e5084e4b041c..05ed8f2ae629d 100644
---- a/activesupport/test/test_case_test.rb
-+++ b/activesupport/test/test_case_test.rb
-@@ -243,7 +243,7 @@ def test_assert_changes_with_to_option_but_no_change_has_special_message
-       end
-     end
- 
--    assert_equal "`@object.num` didn't change. It was already 0.\nExpected 0 to not be equal to 0.", error.message
-+    assert_match "`@object.num` didn't change. It was already 0.", error.message
-   end
- 
-   def test_assert_changes_message_with_lambda
-@@ -255,7 +255,7 @@ def test_assert_changes_message_with_lambda
-       end
-     end
- 
--    assert_equal "`@object.num` didn't change. It was already 0.\nExpected 0 to not be equal to 0.", error.message
-+    assert_match "`@object.num` didn't change. It was already 0.", error.message
-   end
- 
-   def test_assert_changes_with_wrong_to_option

diff --git a/rubygem-activesupport.spec b/rubygem-activesupport.spec
index def2ce6..a4e3c3d 100644
--- a/rubygem-activesupport.spec
+++ b/rubygem-activesupport.spec
@@ -4,31 +4,24 @@
 
 Name: rubygem-%{gem_name}
 Epoch: 1
-Version: 8.0.3
-Release: 6%{?dist}
+Version: 8.1.2
+Release: 1%{?dist}
 Summary: A support libraries and Ruby core extensions extracted from the Rails framework
 License: MIT
 URL: https://rubyonrails.org
 Source0: https://rubygems.org/gems/%{gem_name}-%{version}%{?prerelease}.gem
 # git clone http://github.com/rails/rails.git && cd rails/activesupport
-# git archive -v -o activesupport-8.0.3-tests.tar.gz v8.0.3 test/
+# git archive -v -o activesupport-8.1.2-tests.tar.gz v8.1.2 test/
 Source1: %{gem_name}-%{version}%{?prerelease}-tests.tar.gz
+# `RailsStrictWarnings` is needed by test/event_reporter_test.rb
+# https://github.com/rails/rails/blob/main/tools/strict_warnings.rb
+Source2: https://raw.githubusercontent.com/rails/rails/refs/tags/v8.1.2/tools/strict_warnings.rb
 # This is needed due to `force_skip` alias.
 # https://github.com/rails/rails/blob/main/tools/test_common.rb
-Source2: https://raw.githubusercontent.com/rails/rails/e25d738430bdc6bdd04cd28be705484ea953e74e/tools/test_common.rb
-# Fix XmlMiniTest::ParsingTest#test_decimal test failure with BigDecimal 3.2.3+
-# https://github.com/rails/rails/pull/55840
-Patch1: rubygem-activesupport-8.0.3-Always-pass-default-precision-to-BigDecimal-when-parsing.patch
-# Support minitest 6
-# https://github.com/rails/rails/pull/56202/
-Patch2: rubygem-activesupport-pr56202-minitest6.patch
+Source3: https://raw.githubusercontent.com/rails/rails/e25d738430bdc6bdd04cd28be705484ea953e74e/tools/test_common.rb
 # We don't always install railties with activesupport, so rescue this
 Patch3: rubygem-activesupport-pr56202-minitest6-rescue-loaderror.patch
 
-# Ruby package has just soft dependency on rubygem(json), while
-# ActiveSupport always requires it.
-Requires: rubygem(json)
-
 # Runtime dependency, lot of build failures in other packages.
 # https://fedoraproject.org/wiki/Changes/AllowRemovalOfTzdata
 Requires: tzdata
@@ -42,7 +35,6 @@ BuildRequires: rubygem(builder)
 BuildRequires: rubygem(concurrent-ruby)
 BuildRequires: rubygem(connection_pool)
 BuildRequires: rubygem(dalli)
-BuildRequires: rubygem(drb)
 BuildRequires: rubygem(i18n) >= 0.7
 BuildRequires: rubygem(listen)
 BuildRequires: rubygem(minitest) >= 5.0.0
@@ -76,8 +68,6 @@ Documentation for %{name}.
 %prep
 %setup -q -n %{gem_name}-%{version}%{?prerelease} -a 1
 
-%patch 1 -p2
-%patch 2 -p2
 %patch 3 -p1
 
 # lib/active_support/testing/method_call_assertions.rb
@@ -97,20 +87,13 @@ cp -a .%{gem_dir}/* \
 # Move the tests into place
 cp -a test .%{gem_instdir}
 
-cd .%{gem_instdir}
+( cd .%{gem_instdir}
 
 mkdir ../tools
-ln -s %{SOURCE2} ../tools/
-touch ../tools/strict_warnings.rb
+ln -s %{SOURCE2} %{SOURCE3} ../tools/
 
 sed -i '/require .bundler./ s/^/#/' test/abstract_unit.rb
 
-# backported from:
-# https://github.com/rails/rails/commit/632b2c5128581731c2451459081176a43f474f74
-# benchmark 0.5.0 in ruby4_0 defines Benchmark.ms{}, so the following
-# test is no longer needed
-sed -i test/core_ext/benchmark_test.rb -e '\@test_is_deprecated@s@$@ ; skip ""@'
-
 # Start a testing Valkey (Redis) server instance
 %ifnarch %{ix86}
 VALKEY_DIR=$(mktemp -d)
@@ -131,6 +114,7 @@ kill -15 $mPID
 %ifnarch %{ix86}
 kill -INT $(cat $VALKEY_DIR/valkey.pid)
 %endif
+)
 
 %files
 %dir %{gem_instdir}
@@ -145,6 +129,10 @@ kill -INT $(cat $VALKEY_DIR/valkey.pid)
 %doc %{gem_instdir}/README.rdoc
 
 %changelog
+* Wed Jul 29 2026 Vít Ondruch <vondruch@redhat.com> - 1:8.1.2-1
+- Update to Active Support 8.1.2.
+  Related: rhzb#2405582
+
 * Thu Jul 16 2026 Fedora Release Engineering <releng@fedoraproject.org> - 1:8.0.3-6
 - Rebuilt for https://fedoraproject.org/wiki/Fedora_45_Mass_Rebuild
 

diff --git a/sources b/sources
index f66d7f6..858545a 100644
--- a/sources
+++ b/sources
@@ -1,2 +1,2 @@
-SHA512 (activesupport-8.0.3-tests.tar.gz) = d11560cc2246aaa16fcb7f213061cb6a355bd2e4bbc0cd3e0541db979aa90d28b738ceaf36935f49688953faf94314e2ae8da3e2f88436ac31c0a77a5804a91e
-SHA512 (activesupport-8.0.3.gem) = f46b6710c65d7b59e0c7f1eb48641aa4ef0568b2d64147866e1dfa699c0b4c068bf443cc9967190ed47c2f6ea98137668a300455792982061e280a7df605bb4f
+SHA512 (activesupport-8.1.2-tests.tar.gz) = fa4b08d2d10830b6975daf4fff101a24a3ca90e979a28d761f792c6933db7e48359173355c314d16789d349351a888ebd08d1635df82345d4cd1c59bcf9797c9
+SHA512 (activesupport-8.1.2.gem) = faf1c54009b851a87430167804b2f2750546cb69ee1579d2887e25f2173673a3e38c19335f3cdbd19837f38b55b72bea5d9d09108c7f731d49dbd47c22bd1671

diff --git a/strict_warnings.rb b/strict_warnings.rb
new file mode 100644
index 0000000..4bff9c5
--- /dev/null
+++ b/strict_warnings.rb
@@ -0,0 +1,40 @@
+# frozen_string_literal: true
+
+$VERBOSE = true
+Warning[:deprecated] = true
+
+module RailsStrictWarnings # :nodoc:
+  class WarningError < StandardError; end
+
+  PROJECT_ROOT = File.expand_path("../", __dir__)
+  ALLOWED_WARNINGS = Regexp.union(
+    /circular require considered harmful.*delayed_job/, # Bug in delayed job.
+    /circular require considered harmful.*backburner/, # Bug in backburner
+
+    # Expected non-verbose warning emitted by Rails.
+    /Ignoring .*\.yml because it has expired/,
+    /Failed to validate the schema cache because/,
+  )
+
+  SUPPRESSED_WARNINGS = Regexp.union(
+    # TODO: remove if https://github.com/mikel/mail/pull/1557 or similar fix
+    %r{/lib/mail/parsers/.*statement not reached},
+    %r{/lib/mail/parsers/.*assigned but unused variable - disp_type_s},
+    %r{/lib/mail/parsers/.*assigned but unused variable - testEof}
+  )
+
+  def warn(message, ...)
+    return if SUPPRESSED_WARNINGS.match?(message)
+
+    super
+
+    testpath = message[/test\/.*\.rb/]&.chomp || message
+    return unless message.include?(PROJECT_ROOT) || Pathname.new(testpath).exist?
+    return if ALLOWED_WARNINGS.match?(message)
+    return unless ENV["RAILS_STRICT_WARNINGS"] || ENV["BUILDKITE"]
+
+    raise WarningError.new(message)
+  end
+end
+
+Warning.singleton_class.prepend(RailsStrictWarnings)

                 reply	other threads:[~2026-07-29 14:37 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=178533586173.1.7479535949414407184.rpms-rubygem-activesupport-8bae32173114@fedoraproject.org \
    --to=git-commits@fedoraproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox