Skip to content

Conversation

@radeksimko
Copy link
Member

@radeksimko radeksimko commented Apr 22, 2025

This is to make it possible to build on Solaris again.

https://github.com/hashicorp/terraform/actions/runs/14590452943/job/40924318795#step:4:663

Background

An old version of flock (0.8.1) was brought in as a dependency of https://github.com/oracle/oci-go-sdk which itself was introduced in #36872

Pinning transitive dependencies downstream like this is not ideal as we have no control over compatibility between the direct and transitive dependency. i.e. we cannot guarantee on the level of Terraform that the pinned version of flock is and remains compatible with current and future versions of OCI SDK.

As a result, this was raised upstream:

Target Release

1.13.x

CHANGELOG entry

  • This change is user-facing and I added a changelog entry.
  • This change is not user-facing.

@radeksimko radeksimko force-pushed the radek/pin-flock-to-v0.10 branch from 89f24ee to ed2cccf Compare April 22, 2025 09:42
@radeksimko radeksimko added the no-changelog-needed Add this to your PR if the change does not require a changelog entry label Apr 22, 2025
@radeksimko radeksimko marked this pull request as ready for review April 22, 2025 09:59
@radeksimko radeksimko requested a review from a team as a code owner April 22, 2025 09:59
@radeksimko radeksimko merged commit 474fe47 into main Apr 22, 2025
15 of 16 checks passed
@radeksimko radeksimko deleted the radek/pin-flock-to-v0.10 branch April 22, 2025 12:30
@github-actions
Copy link
Contributor

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 25, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

no-changelog-needed Add this to your PR if the change does not require a changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants