-
Notifications
You must be signed in to change notification settings - Fork 202
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
Consider splitting up pkg/util into semantically-relevant subpackages #1654
Comments
1 task
I agree lets break this into a series of subpackages. |
I am also supportive. Some files in Podman import 3 or more "util" or "utils" packages. |
rhatdan
added a commit
to rhatdan/common
that referenced
this issue
Sep 18, 2023
Fixes: containers#1654 Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
rhatdan
added a commit
to rhatdan/common
that referenced
this issue
Sep 18, 2023
Fixes: containers#1654 Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
rhatdan
added a commit
to rhatdan/common
that referenced
this issue
Sep 19, 2023
Fixes: containers#1654 Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
rhatdan
added a commit
to rhatdan/common
that referenced
this issue
Sep 19, 2023
Fixes: containers#1654 Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
Thanks @rhatdan . |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
See https://github.com/containers/skopeo/pull/2112/files : adding a call to
util.ReadPassword
triggers addition of several other unnecessary subpackages. (I am starting to think that autil
subpackage is always a mistake because it will become a dependency magnet like this, caller of one small thing accrues several other unwanted dependencies.)This is, of course, quite low-priority.
The text was updated successfully, but these errors were encountered: