build,win: simplify new msbuild_arg option

PR-URL: #26431
Refs: #25994
Reviewed-By: João Reis <reis@janeasystems.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>

Author: refack

vcbuild.bat

@@ -63,7 +63,7 @@ set no_cctest=
 setcctest=
 setopenssl_no_asm=
 setdoc=
-setextra_msbuild_args=^
+setextra_msbuild_args=
 
 :next-arg
 if"%1"==""goto args-done
@@ -138,8 +138,7 @@ if /i "%1"=="no-cctest" set no_cctest=1&goto arg-ok
 if /i "%1"=="cctest"setcctest=1&goto arg-ok
 if /i "%1"=="openssl-no-asm"setopenssl_no_asm=1&goto arg-ok
 if /i "%1"=="doc"setdoc=1&goto arg-ok
-if /i "%1"=="binlog"setextra_msbuild_args=%extra_msbuild_args% /binaryLogger:%config%\node.binlog&goto arg-ok
-if /i "%1"=="msbuild_arg"setextra_msbuild_args=%extra_msbuild_args%%2&goto arg-ok-2
+if /i "%1"=="binlog"setextra_msbuild_args=/binaryLogger:%config%\node.binlog&goto arg-ok
 
 echo Error: invalid command line option `%1`.
 exit /b 1
@@ -258,6 +257,7 @@ set vcvars_call="%VCINSTALLDIR%\Auxiliary\Build\vcvarsall.bat" %vcvarsall_arg%
 echo calling: %vcvars_call%
 call%vcvars_call%
 iferrorlevel1goto msbuild-not-found
+ifdefined DEBUG_HELPER @ECHOON
 :found_vs2017
 echo Found MSVS version %VisualStudioVersion%
 setGYP_MSVS_VERSION=2017
@@ -323,6 +323,7 @@ if "%target%"=="Build" (
 ifdefined cctest settarget="Build"
 )
 if"%target%"=="rename_node_bin_win"ifexist"%config%\cctest.exe"del"%config%\cctest.exe"
+ifdefined msbuild_args set"extra_msbuild_args=%extra_msbuild_args%%msbuild_args%"
 msbuild node.sln %msbcpu% /t:%target% /p:Configuration=%config% /p:Platform=%msbplatform% /clp:NoSummary;NoItemAndPropertyList;Verbosity=minimal /nologo %extra_msbuild_args%
 iferrorlevel1 (
 ifnotdefined project_generated echo Building Node with reused solution failed. To regenerate project files use "vcbuild projgen"