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

Accept ffi >= 1.16.3, < 1.17.0 #9330

Merged
merged 1 commit into from Apr 25, 2024
Merged

Conversation

joshcooper
Copy link
Contributor

ffi 1.16.0 - 1.16.2 had a bug when defining a layout in an FFI::Struct and it was fixed in 1.16.3.

See puppetlabs/facter#2707

ffi 1.16.0 - 1.16.2 had a bug when defining a layout in an FFI::Struct and it
was fixed in 1.16.3.

See puppetlabs/facter#2707
@mhashizume mhashizume merged commit 1d188e1 into puppetlabs:main Apr 25, 2024
9 checks passed
@mhashizume mhashizume added maintenance Maintenance chores are excluded from changelogs backport 7.x Generate a backport PR to 7.x labels Apr 25, 2024
Copy link

Backport failed for 7.x, because it was unable to cherry-pick the commit(s).

Please cherry-pick the changes locally.

git fetch origin 7.x
git worktree add -d .worktree/backport-9330-to-7.x origin/7.x
cd .worktree/backport-9330-to-7.x
git checkout -b backport-9330-to-7.x
ancref=$(git merge-base fa213f5808c271c9e30fce04779489806fc93da7 aa513403255081f315de7dbdda8aaae1016ee036)
git cherry-pick -x $ancref..aa513403255081f315de7dbdda8aaae1016ee036

@mhashizume
Copy link
Contributor

We'll need to wait for the changes in #9325 to get backported to 7.x before we can backport these FFI changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 7.x Generate a backport PR to 7.x maintenance Maintenance chores are excluded from changelogs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants