Skip to content

[28.x backport] cli/connhelper: remove dependency on pkg/process#6262

Merged
thaJeztah merged 1 commit intodocker:28.xfrom
thaJeztah:28.x_backport_no_pkg_process
Aug 14, 2025
Merged

[28.x backport] cli/connhelper: remove dependency on pkg/process#6262
thaJeztah merged 1 commit intodocker:28.xfrom
thaJeztah:28.x_backport_no_pkg_process

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

This package will not be included in the api or client modules, and we're currently only using a single function of it, and only the unix implementation, so let's fork it for now (although the package may be moved to moby/sys).

This removes the last dependency on github.com/docker/docker.

(cherry picked from commit 2abcbf8)

- What I did

- How I did it

- How to verify it

- Human readable description for the release notes

- A picture of a cute animal (not mandatory but encouraged)

@thaJeztah thaJeztah added this to the 28.4.0 milestone Aug 14, 2025
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@thaJeztah thaJeztah marked this pull request as draft August 14, 2025 15:59
This package will not be included in the api or client modules, and
we're currently only using a single function of it, and only the
unix implementation, so let's fork it for now (although the package
may be moved to moby/sys).

This removes the last dependency on github.com/docker/docker.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
(cherry picked from commit 2abcbf8)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah force-pushed the 28.x_backport_no_pkg_process branch from c9e9edf to 7399781 Compare August 14, 2025 16:45
@thaJeztah thaJeztah marked this pull request as ready for review August 14, 2025 16:59
@thaJeztah thaJeztah merged commit 3220783 into docker:28.x Aug 14, 2025
100 of 101 checks passed
@thaJeztah thaJeztah deleted the 28.x_backport_no_pkg_process branch August 14, 2025 19:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants