Skip to content

Don't suppress the pstore LoadError#70

Open
ekohl wants to merge 1 commit intoruby:masterfrom
ekohl:dont-suppress-loaderror
Open

Don't suppress the pstore LoadError#70
ekohl wants to merge 1 commit intoruby:masterfrom
ekohl:dont-suppress-loaderror

Conversation

@ekohl
Copy link
Copy Markdown

@ekohl ekohl commented Dec 28, 2025

Since Ruby 4.0 pstore is a bundled gem instead of a default gem. YAML::Store inherits from PStore so it's a hard requirement. Supressing the LoadError makes it hard to understand why it's not working.

Link: https://www.ruby-lang.org/en/news/2025/12/25/ruby-4-0-0-released/#stdlib-updates
Link: https://stdgems.org/pstore/
Fixes: #69

Since Ruby 4.0 pstore is a bundled gem instead of a default gem.
YAML::Store inherits from PStore so it's a hard requirement. Supressing
the LoadError makes it hard to understand why it's not working.

Link: https://www.ruby-lang.org/en/news/2025/12/25/ruby-4-0-0-released/#stdlib-updates
Link: https://stdgems.org/pstore/
@hsbt
Copy link
Copy Markdown
Member

hsbt commented Jan 16, 2026

This broke ruby/ruby test suite. It's better to add guide message instead of this change.

@ekohl
Copy link
Copy Markdown
Author

ekohl commented Jan 16, 2026

Do you have any link to that breakage?

It's better to add guide message instead of this change.

As a developer I never like those messages. A real exception is IMHO a better experience because I can catch that.

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.

YAML::Store dependency for pstore is required and fails on Ruby 4.0

2 participants