Re: [PATCH 2/4] Windows: Add 'move' to sh_cmds_dos

[Ray Donnelly]

Thanks Eli,

On systems where move *is* a built-in, gnumake fails as CreateProcess(NULL, "move" ...) doesn't work.

On systems where move *is not* a built-in, my proposed fix is fine as "cmd.exe /c move" still works.

Cheers,

Ray.

On Tue, Apr 16, 2013 at 2:59 PM, Eli Zaretskii <address@hidden> wrote:

> Date: Tue, 16 Apr 2013 13:39:58 +0100
> From: Ray Donnelly <address@hidden>

>
> 'move' is not listed as a cmd.exe builtin when it needs to be.
>
>
> Not sure how this hasn't been spotted and fixed before now!

It's not a bug, it is done on purpose: 'move' is a built-in on some
versions of Windows, and a .exe program on others.

Thanks.