Skip to content

fix a regression in KDDI transcoding#8671

Merged
headius merged 1 commit intojruby:jruby-9.4from
ahorek:enable_encoding_tests
Mar 7, 2025
Merged

fix a regression in KDDI transcoding#8671
headius merged 1 commit intojruby:jruby-9.4from
ahorek:enable_encoding_tests

Conversation

@ahorek
Copy link
Contributor

@ahorek ahorek commented Mar 4, 2025

@headius
Copy link
Member

headius commented Mar 4, 2025

Some commits snuck into master but not jruby-9.4 so I rebased the latter. Shouldn't affect this if you rebase the PR.

Once we have a repaired jcodings I will merge that to 9.4 and you can merge or rebase onto the PR.

@ahorek ahorek mentioned this pull request Mar 5, 2025
@ahorek ahorek force-pushed the enable_encoding_tests branch from e0cce8e to ce48d84 Compare March 5, 2025 23:11
@ahorek ahorek changed the title Draft: enable encoding tests fix a regression in KDDI transcoding Mar 5, 2025
@ahorek ahorek added this to the JRuby 9.4.13.0 milestone Mar 5, 2025
@headius
Copy link
Member

headius commented Mar 7, 2025

@ahorek We're close! Looks like two failures here after the updated versions of jcodings and joni are available. I get the same failures locally without the jcodings/joni updates, so it's not a regression from anything you did. I'm going to go ahead and merge this and exclude from the jruby-9.4 branch. If you feel so inclined to keep digging into those last two, go for it!

@headius headius merged commit d17690c into jruby:jruby-9.4 Mar 7, 2025
92 of 96 checks passed
headius added a commit to headius/jruby that referenced this pull request Mar 7, 2025
These are probably not recent regressions, but tagging them for
now to get master back to green after recent jcodings updates.

Relates to jruby#8671
headius added a commit that referenced this pull request Mar 7, 2025
These are probably not recent regressions, but tagging them for
now to get master back to green after recent jcodings updates.

Relates to #8671
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.

2 participants