Re: [PATCH v6 4/6] robocopy: Add basic output

[Zebediah Figura (she/her)]

On 9/21/21 09:54, Florian Eder wrote:
> Adds output method and prints a header and the source / destination / files to include
> back to the user
> Signed-off-by: Florian Eder
> v6: Identical to patch in v5
>
> programs/robocopy/main.c      | 63 +++++++++++++++++++++++++++++++++++
>   programs/robocopy/robocopy.h  |  6 ++++
>   programs/robocopy/robocopy.rc | 11 ++++++
>   3 files changed, 80 insertions(+)

I know Microsoft's robocopy does this, but do we need to? I mean, it 
doesn't tell me anything I can't find just from reading the command 
line. And then on top of that it's more strings to translate, and more 
code, and so on.

It does help validate 3/6, but ideally the implementation and tests 
should do that.