[PATCH 3/3] [cmd] Space does not delimit environment variable names
Ann and Jason Edmeades
jason at edmeades.me.uk
Wed Sep 5 18:31:38 CDT 2012
Just tidying up a 'todo' in the tests, but set fred =1 sets
a variable 'fred ' to the value one (which works), but you
cannot subsequently echo it back.
[No bug, just some todos]
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.winehq.org/pipermail/wine-patches/attachments/20120906/19c25e85/attachment.html>
-------------- next part --------------
From 16e9d0e3adfdae554f8fad99c7f467492702dc1f Mon Sep 17 00:00:00 2001
From: Jason Edmeades <jason at edmeades.me.uk>
Date: Thu, 6 Sep 2012 00:09:23 +0100
Subject: [PATCH 3/3] [cmd] Space does not delimit environment variable names
Just tidying up a 'todo' in the tests, but set fred =1 sets
a variable 'fred ' to the value one (which works), but you
cannot subsequently echo it back.
[No bug, just some todos]
---
programs/cmd/builtins.c | 2 ++
programs/cmd/tests/test_builtins.cmd.exp | 4 ++--
programs/cmd/wcmdmain.c | 2 +-
3 files changed, 5 insertions(+), 3 deletions(-)
diff --git a/programs/cmd/builtins.c b/programs/cmd/builtins.c
index 270692c..64a2c17 100755
--- a/programs/cmd/builtins.c
+++ b/programs/cmd/builtins.c
@@ -2366,6 +2366,8 @@ void WCMD_setshow_env (WCHAR *s) {
*p++ = '\0';
if (strlenW(p) == 0) p = NULL;
+ WINE_TRACE("set: Setting var '%s' to '%s'\n", wine_dbgstr_w(s),
+ wine_dbgstr_w(p));
status = SetEnvironmentVariableW(s, p);
gle = GetLastError();
if ((!status) & (gle == ERROR_ENVVAR_NOT_FOUND)) {
diff --git a/programs/cmd/tests/test_builtins.cmd.exp b/programs/cmd/tests/test_builtins.cmd.exp
index 9578c75..be5c4e0 100644
--- a/programs/cmd/tests/test_builtins.cmd.exp
+++ b/programs/cmd/tests/test_builtins.cmd.exp
@@ -203,10 +203,10 @@ baz at space@
1
0
FOOBAR not defined
- at todo_wine@ baz
+ baz
FOOBAR = baz
0
- at todo_wine@ baz2
+ baz2
0
bar
@todo_wine at FOOBAR= bar
diff --git a/programs/cmd/wcmdmain.c b/programs/cmd/wcmdmain.c
index 93a2bb8..f762b70 100644
--- a/programs/cmd/wcmdmain.c
+++ b/programs/cmd/wcmdmain.c
@@ -557,7 +557,7 @@ static WCHAR *WCMD_expand_envvar(WCHAR *start,
static const WCHAR Time[] = {'T','I','M','E','\0'};
static const WCHAR Cd[] = {'C','D','\0'};
static const WCHAR Random[] = {'R','A','N','D','O','M','\0'};
- static const WCHAR Delims[] = {'%',' ',':','\0'};
+ static const WCHAR Delims[] = {'%',':','\0'};
WINE_TRACE("Expanding: %s (%s,%s)\n", wine_dbgstr_w(start),
wine_dbgstr_w(forVal), wine_dbgstr_w(forVar));
--
1.7.5.4
More information about the wine-patches
mailing list