Skip to content
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: show mode if target does not have S_IWUSR #6696

Closed
wants to merge 8 commits into from

Conversation

andrewliebenow
Copy link
Contributor

If the target exists, and does not have the user write bit (S_IWUSR) set, additional information should be added to the overwrite confirmation prompt.

This should get the "i-2" test to pass. See
#6658.

If the target exists, and does not have the user write bit (S_IWUSR)
set, additional information should be added to the overwrite
confirmation prompt.

This should get the "i-2" test to pass. See
uutils#6658.
Copy link

GNU testsuite comparison:

Congrats! The gnu test tests/mv/i-2 is no longer failing!

src/uu/cp/src/cp.rs Outdated Show resolved Hide resolved
src/uu/cp/src/cp.rs Outdated Show resolved Hide resolved
Copy link

GNU testsuite comparison:

Skip an intermittent issue tests/tail/inotify-dir-recreate (fails in this run but passes in the 'main' branch)
Congrats! The gnu test tests/mv/i-2 is no longer failing!

Copy link

GNU testsuite comparison:

Congrats! The gnu test tests/mv/i-2 is no longer failing!

andrewliebenow and others added 2 commits September 16, 2024 02:03
Co-authored-by: Sylvestre Ledru <sylvestre@debian.org>
@andrewliebenow
Copy link
Contributor Author

Okay, I thought using the GitHub "update branch" option would do a rebase, sorry about that.

@andrewliebenow
Copy link
Contributor Author

Closed in favor of #6696.
Sorry about the CI spam, I didn't want to add unnecessary non-linearity to the Git history.

Copy link

GNU testsuite comparison:

Congrats! The gnu test tests/mv/i-2 is no longer failing!

sylvestre added a commit that referenced this pull request Sep 18, 2024
* cp: show mode if target does not have S_IWUSR

If the target exists, and does not have the user write bit (S_IWUSR)
set, additional information should be added to the overwrite
confirmation prompt.

This should get the "i-2" test to pass. See
#6658.

* cp: with -i, delete destination if needed

---------

Co-authored-by: Sylvestre Ledru <sylvestre@debian.org>
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.

2 participants