-
-
Couldn't load subscription status.
- Fork 1.6k
uu: updated error message for mv on overwriting non-empty directory inside destination directory #8988
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
base: main
Are you sure you want to change the base?
Conversation
|
GNU testsuite comparison: |
|
GNU testsuite comparison: |
|
I noticed that my mv command produces this error msg in the scenario mentioned in #5102. For context, my system is on GNU coreutils 9.8-2:
this error msg was proposed by Paul here: https://lists.gnu.org/archive/html/bug-coreutils/2023-07/msg00032.html and it seems to be implemented in GNU coreutils 9.8-2 This is different from the previous error msg that rust coreutils produces:
Currently, I have it set with the error message proposed by PThorpe92:
Should I change it to the one that GNU coreutils use as of 9.8-2? |
|
Moreover, I'm unsure why the Android/Test build is failing. |
|
GNU testsuite comparison: |
|
GNU testsuite comparison: |
…n-empty destination directory's content with source directory of the same name
…without T flag produces the cannot overwrite error
|
GNU testsuite comparison: |
This PR implements the solution mentioned by @tertsdiepraam and @PThorpe92 in #5102 to provide a clearer move error message on overwriting a non-empty directory in the destination directory with the same name as source directory.
I added and modified a couple of error messages within the locales directory of the mv command. I did use Google Translate to translate the error message in English to French, so I'm not sure if it is well translated.