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.

Sign up for GitHub

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

Jump to bottom

[v0.10 backport] vendor tonistiigi/fsutil v0.0.0-20220510150904-0dbf3a8a7d58 #3144

Closed
wants to merge 2 commits into from
Closed

[v0.10 backport] vendor tonistiigi/fsutil v0.0.0-20220510150904-0dbf3a8a7d58 #3144

wants to merge 2 commits into from

Conversation

thaJeztah
Copy link
Member

@thaJeztah thaJeztah mentioned this pull request Oct 3, 2022
Closed
@thaJeztah thaJeztah marked this pull request as ready for review October 3, 2022 19:27
@thaJeztah
Copy link
Member Author

@tonistiigi PTAL; this makes sure BuildKit is compatible with the breaking change in fsutil; without this, we'd be stuck to older versions.

@thaJeztah thaJeztah mentioned this pull request Oct 4, 2022
Closed
@thaJeztah
Copy link
Member Author

@crazy-max @tonistiigi @AkihiroSuda ptal 🙏

This was referenced Oct 13, 2022
Closed
Closed
Closed
tonistiigi and others added 2 commits November 14, 2022 10:16
@tonistiigi @thaJeztah
vendor: update fsutil to 3c5c7e8
05bd51b 
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com>
(cherry picked from commit 60addc4)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@nicks @thaJeztah
vendor: upgrade github.com/tonistiigi/fsutil
77c20e8 
fixes moby#2837

Signed-off-by: Nick Santos <nick@tilt.dev>
(cherry picked from commit 844618f)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah force-pushed the 0.10_backport_fsutil_compat branch from 9a75f58 to 77c20e8 Compare November 14, 2022 09:17
@thaJeztah thaJeztah mentioned this pull request Nov 24, 2022
Merged
@tonistiigi tonistiigi closed this Feb 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AkihiroSuda AkihiroSuda AkihiroSuda approved these changes

@tonistiigi tonistiigi Awaiting requested review from tonistiigi

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@thaJeztah @AkihiroSuda @tonistiigi @nicks