8000 cp: allow multiple usages of -R - Closes: #5027 by sylvestre · Pull Request #5028 · uutils/coreutils · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

cp: allow multiple usages of -R - Closes: #5027 #5028

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
Merged
merged 1 commit into from
Jul 3, 2023
Merged

cp: allow multiple usages of -R - Closes: #5027 #5028

merged 1 commit into from
Jul 3, 2023

Conversation

sylvestre
Copy link
Contributor

No description provided.

@tertsdiepraam
8000 Copy link
Member

Can we use Command::args_override_self to fix all the arguments at once or does that lead to other problems?

cakebaker
cakebaker approved these changes Jul 3, 2023
View reviewed changes
Copy link
Contributor
@cakebaker cakebaker left a comment

Choose a reason for hiding this comment

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

Looks fine. I didn't merge it because of @tertsdiepraam's question.

@sylvestre
cp: allow multiple usages of -R/-r Closes: #5027
e491bbd 
works for all other rm args
@sylvestre
Copy link
Contributor Author

Can we use Command::args_override_self to fix all the arguments at once or does that lead to other problems?

Let's see :)
I just pushed it
cargo test passed, we will see with GNU

@cakebaker cakebaker merged commit 11b098f into uutils:main Jul 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cakebaker cakebaker cakebaker approved these changes

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

Successfully merging this pull request may close these issues.
3 participants
@sylvestre @tertsdiepraam @cakebaker
0