Disable echoing CMD code in copy_directory and copy_file rules on Windows

Author: zpervan

rules/private/copy_directory_private.bzl

@@ -34,12 +34,13 @@ def _copy_cmd(ctx, src, dst):
     # https://docs.microsoft.com/en-us/windows-server/administration/windows-commands/robocopy
     # NB: robocopy return non-zero exit codes on success so we must exit 0 after calling it
     cmd_tmpl = """\
-if not exist \"{src}\\\" (
-  echo Error: \"{src}\" is not a directory
-  @exit 1
-)
-@robocopy \"{src}\" \"{dst}\" /E /MIR >NUL & @exit 0
-"""
+        @ECHO OFF
+        if not exist \"{src}\\\" (
+            echo Error: \"{src}\" is not a directory
+            @exit 1
+        )
+        @robocopy \"{src}\" \"{dst}\" /E /MIR >NUL & @exit 0
+        """
     mnemonic = "CopyDirectory"
     progress_message = "Copying directory %{input}"
 

rules/private/copy_file_private.bzl

@@ -28,11 +28,17 @@ def copy_cmd(ctx, src, dst):
     # To fix that we write the command to a .bat file so no command line
     # quoting or escaping is required.
     bat = ctx.actions.declare_file(ctx.label.name + "-cmd.bat")
+
+    cmd_tmpl = """\
+        @ECHO OFF
+        @copy /Y \"%s\" \"%s\" >NUL
+        """
+
     ctx.actions.write(
         output = bat,
         # Do not use lib/shell.bzl's shell.quote() method, because that uses
         # Bash quoting syntax, which is different from cmd.exe's syntax.
-        content = "@copy /Y \"%s\" \"%s\" >NUL" % (
+        content = cmd_tmpl.format(
             src.path.replace("/", "\\"),
             dst.path.replace("/", "\\"),
         ),