Skip to content

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

Merged
cakebaker merged 1 commit intouutils:mainfrom
sylvestre:cp-multiple
Jul 3, 2023
Merged

cp: allow multiple usages of -R - Closes: #5027#5028
cakebaker merged 1 commit intouutils:mainfrom
sylvestre:cp-multiple

Conversation

@sylvestre
Copy link
Contributor

No description provided.

@tertsdiepraam
Copy link
Collaborator

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

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.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants