Merge pull request #5303 from ChrisCatCP/vs_unicode

fix msvc output when toolset less than 8.0

Author: waruqi

xmake/modules/private/tools/vstool.lua

@@ -24,14 +24,20 @@ function runv(program, argv, opt)
     -- init options
     opt = opt or {}
 
+    -- if has VS_BINARY_OUTPUT dont enable unicode output
+    local envs = opt.envs or {}
+    if envs.VS_BINARY_OUTPUT then
+        return os.runv(program, argv, opt)
+    end
+
     -- make temporary output and error file
     local outpath = os.tmpfile()
     local errpath = os.tmpfile()
     local outfile = io.open(outpath, 'w')
 
     -- enable unicode output for vs toolchains, e.g. cl.exe, link.exe and etc.
     -- @see https://github.com/xmake-io/xmake/issues/528
-    opt.envs = table.join(opt.envs or {}, {VS_UNICODE_OUTPUT = outfile:rawfd()})
+    opt.envs = table.join(envs, {VS_UNICODE_OUTPUT = outfile:rawfd()})
 
     -- execute it
     local ok, syserrors = os.execv(program, argv, table.join(opt, {try = true, stdout = outfile, stderr = errpath}))
@@ -85,6 +91,12 @@ function iorunv(program, argv, opt)
 
     -- init options
     opt = opt or {}
+    
+    -- if has VS_BINARY_OUTPUT dont enable unicode output
+    local envs = opt.envs or {}
+    if envs.VS_BINARY_OUTPUT then
+        return os.runv(program, argv, opt)
+    end
 
     -- make temporary output and error file
     local outpath = os.tmpfile()
@@ -93,7 +105,7 @@ function iorunv(program, argv, opt)
 
     -- enable unicode output for vs toolchains, e.g. cl.exe, link.exe and etc.
     -- @see https://github.com/xmake-io/xmake/issues/528
-    opt.envs = table.join(opt.envs or {}, {VS_UNICODE_OUTPUT = outfile:rawfd()})
+    opt.envs = table.join(envs, {VS_UNICODE_OUTPUT = outfile:rawfd()})
 
     -- run command
     local ok, syserrors = os.execv(program, argv, table.join(opt, {try = true, stdout = outfile, stderr = errpath}))

xmake/toolchains/msvc/load.lua

@@ -20,6 +20,7 @@
 
 -- imports
 import("core.base.option")
+import("core.base.semver")
 import("core.project.config")
 import("detect.sdks.find_vstudio")
 
@@ -78,5 +79,11 @@ function main(toolchain)
             _add_vsenv(toolchain, name, curenvs)
         end
     end
+
+    -- check and add vs_binary_output env
+    local vs = toolchain:config("vs")
+    if vs and semver.is_valid(vs) and semver.compare(vs, "2005") < 0 then
+        toolchain:add("runenvs", "VS_BINARY_OUTPUT", "1")
+    end
 end