-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
cp: consolidate and simplify #11065
cp: consolidate and simplify #11065
Conversation
Based on comments from @rhatdan @containers/podman-maintainers PTAL |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: vrothberg The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Consolidate and simplify code in `podman cp` a bit. PR containers#11049 introduced some code duplicates that were worth tackling. [NO TESTS NEEDED] Signed-off-by: Valentin Rothberg <[email protected]>
LGTM, Big improvement. |
Thank you @vrothberg! Can you please update the manpage? I have made the changes, just add them to your commit if possible. If you want I can open a new PR to do that, but would be irrelevant I think. |
I updated the man page already in #11049. If you want more changes, please open a PR. |
Green and ready to merge |
/lgtm |
Consolidate and simplify code in
podman cp
a bit. PR #11049introduced some code duplicates that were worth tackling.
Signed-off-by: Valentin Rothberg [email protected]