• Allen Bauer's avatar
    Update autoninja.bat to work with other shells. · ba352e00
    Allen Bauer authored
    Other shells, such as TCC, have subtle differences in how batch files are processed. This will not break cmd.exe compatibility.
    
    Without the "call" command, running a nested batch file should, according to CMD.exe "rules", cancel the current batch file and run the new one. However, the 'FOR /f "usebackq"' command will call the batch file and return. One could argue for both approaches, however, adding the "call" command doesn't seem to affect cmd.exe and fixes the other shell.
    
    There is a similar issue with the "do" (in the FOR command) block and the use of the '&' separator. The '%a' variable isn't expanded, but adding the '()' block works in both instances.
    
    Bug: 1351817
    Change-Id: Idd685a64404ea950b71e1f3cc0f5d1e3bf13b8b6
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3824199
    Commit-Queue: Allen Bauer <kylixrd@chromium.org>
    Reviewed-by: 's avatarBruce Dawson <brucedawson@chromium.org>
    ba352e00
autoninja.bat 2.11 KB