Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ensure we don't use newer toml-rb which requires a compiler #458

Merged
merged 7 commits into from
Mar 10, 2024

Conversation

ramereth
Copy link
Contributor

@ramereth ramereth commented Mar 10, 2024

Fixes #455

Description

Describe what this change achieves

Issues Resolved

List any existing issues this PR resolves

Check List

  • A summary of changes made is included in the CHANGELOG under ## Unreleased
  • New functionality includes testing.
  • New functionality has been documented in the README if applicable.

@ramereth ramereth added the Release: Minor Release to Chef Supermarket as a minor release when merged label Mar 10, 2024
@ramereth ramereth requested a review from a team as a code owner March 10, 2024 19:20
ramereth and others added 6 commits March 10, 2024 13:36
The toml-rb 3.0.0 gem adds an explicit dependency on racc ~> 1.7 which requires
a compiler. For now don't pull in newer gem as it complicates the logic of the
various libraries.

This resolves #455

Signed-off-by: Lance Albertson <[email protected]>
Signed-off-by: Lance Albertson <[email protected]>
Signed-off-by: Lance Albertson <[email protected]>
@ramereth ramereth changed the title Fix issues when newer installing toml-rb Ensure we don't use newer toml-rb which requires a compiler Mar 10, 2024
AmazonLinux 2023 isn't quite ready or seems to have an issue with the dokken
container.

Signed-off-by: Lance Albertson <[email protected]>
@ramereth
Copy link
Contributor Author

@bmhughes I couldn't figure out how to get the newer gem to work with getting the compiler installed before the libraries load. For now this seems like the best short term solution. Let me know if you have a better idea.

@bmhughes
Copy link
Contributor

I've gotten this to work for postgresql so I'd think that pattern should work.

This will work for the meantime though for sure.

@ramereth ramereth merged commit a6964b7 into main Mar 10, 2024
106 checks passed
@ramereth ramereth deleted the toml-rb-fixes branch March 10, 2024 22:06
@kitchen-porter
Copy link
Contributor

Released as: 10.7.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Release: Minor Release to Chef Supermarket as a minor release when merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

toml-rb 3.0.0 breaks gem dependency installation
3 participants