uu: updated error message for mv on overwriting non-empty directory inside destination directory#8988
uu: updated error message for mv on overwriting non-empty directory inside destination directory#8988asder8215 wants to merge 7 commits into
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: |
please fix them |
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.