build_env(): respect the MSYS environment variable

dscho · lazka · commit bce9e93d1913 · 2023-02-15T10:13:49.000+01:00
With this commit, you can call

	MSYS=noemptyenvvalues my-command

and it does what is expected: to pass no empty-valued environment
variables to `my-command`.

Signed-off-by: Johannes Schindelin &lt;johannes.schindelin@gmx.de&gt;
diff --git a/winsup/cygwin/environ.cc b/winsup/cygwin/environ.cc
@@ -1203,7 +1203,11 @@ build_env (const char * const *envp, PWCHAR &envblock, int &envc,
     {
       bool calc_tl = !no_envblock;
 #ifdef __MSYS__
-      if (!keep_posix)
+      if (ascii_strncasematch(*srcp, "MSYS=", 5))
+        {
+          parse_options (*srcp + 5);
+	}
+      else if (!keep_posix)
         {
           /* Don't pass timezone environment to non-msys applications */
           if (ascii_strncasematch(*srcp, "TZ=", 3))

Original file line number	Diff line number	Diff line change
`@@ -1203,7 +1203,11 @@ build_env (const char * const *envp, PWCHAR &envblock, int &envc,`
`1203`	`1203`	`{`
`1204`	`1204`	`bool calc_tl = !no_envblock;`
`1205`	`1205`	`#ifdef __MSYS__`
`1206`		`- if (!keep_posix)`
	`1206`	`+ if (ascii_strncasematch(*srcp, "MSYS=", 5))`
	`1207`	`+ {`
	`1208`	`+ parse_options (*srcp + 5);`
	`1209`	`+ }`
	`1210`	`+ else if (!keep_posix)`
`1207`	`1211`	`{`
`1208`	`1212`	`/* Don't pass timezone environment to non-msys applications */`
`1209`	`1213`	`if (ascii_strncasematch(*srcp, "TZ=", 3))`