Skip to content

Conversation

@thaJeztah
Copy link
Member

No description provided.

@thaJeztah thaJeztah force-pushed the remove_imageutil_dead_code branch from b2c9482 to b9e25e8 Compare July 17, 2023 10:35
@thaJeztah
Copy link
Member Author

oh, nice; that also removes ttrpc as dependency 😄

@thaJeztah
Copy link
Member Author

@jedevc good to go?

@jedevc jedevc merged commit 7c3b77f into docker:master Jul 17, 2023
@thaJeztah thaJeztah deleted the remove_imageutil_dead_code branch July 17, 2023 11:29
github.com/cenkalti/backoff/v4 v4.2.0 // indirect
github.com/cespare/xxhash/v2 v2.2.0 // indirect
github.com/cloudflare/cfssl v0.0.0-20181213083726-b94e044bb51e // indirect
github.com/containerd/ttrpc v1.2.2 // indirect
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now sure why it does remove ttrpc 🤔

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh github.com/moby/buildkit/util/imageutil has a depedency on it nevermind.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yup! Turtles all the way down

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.

3 participants