[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v2] linux-user: Restore original behavior of the -E
From: |
Thomas Schwinge |
Subject: |
[Qemu-devel] [PATCH v2] linux-user: Restore original behavior of the -E and -U command-line options. |
Date: |
Thu, 25 Apr 2013 18:37:41 +0200 |
Revert the change in behavior that had been introducecd in commit
fc9c54124d134dbd76338a92a91804dab2df8166 for the -E and -U command-line
options, but keep the comma-splitting for the QEMU_SET_ENV and QEMU_UNSET_ENV
environment variables.
Signed-off-by: Thomas Schwinge <address@hidden>
---
linux-user/main.c | 51 +++++++++++++++++++++++++++++++++++----------------
1 file changed, 35 insertions(+), 16 deletions(-)
diff --git linux-user/main.c linux-user/main.c
index 50bbadf..c539125 100644
--- linux-user/main.c
+++ linux-user/main.c
@@ -3209,15 +3209,37 @@ static void handle_arg_log_filename(arg_origin whence,
const char *arg)
static void handle_arg_set_env(arg_origin whence, const char *arg)
{
- if (envlist_parse_set(envlist, arg) != 0) {
- usage();
+ switch (whence) {
+ case ARG_ORIGIN_ENV:
+ if (envlist_parse_set(envlist, arg) != 0) {
+ usage();
+ }
+ break;
+ case ARG_ORIGIN_CMDLINE:
+ if (envlist_setenv(envlist, arg) != 0) {
+ usage();
+ }
+ break;
+ default:
+ abort();
}
}
static void handle_arg_unset_env(arg_origin whence, const char *arg)
{
- if (envlist_parse_unset(envlist, arg) != 0) {
- usage();
+ switch (whence) {
+ case ARG_ORIGIN_ENV:
+ if (envlist_parse_unset(envlist, arg) != 0) {
+ usage();
+ }
+ break;
+ case ARG_ORIGIN_CMDLINE:
+ if (envlist_unsetenv(envlist, arg) != 0) {
+ usage();
+ }
+ break;
+ default:
+ abort();
}
}
@@ -3443,18 +3465,15 @@ static void usage(void)
guest_stack_size);
printf("\n"
- "You can use -E and -U options or the QEMU_SET_ENV and\n"
- "QEMU_UNSET_ENV environment variables to set and unset\n"
- "environment variables for the target process.\n"
- "It is possible to provide several variables by separating them\n"
- "by commas in getsubopt(3) style. Additionally it is possible to\n"
- "provide the -E and -U options multiple times.\n"
- "The following lines are equivalent:\n"
- " -E var1=val2 -E var2=val2 -U LD_PRELOAD -U LD_DEBUG\n"
- " -E var1=val2,var2=val2 -U LD_PRELOAD,LD_DEBUG\n"
- " QEMU_SET_ENV=var1=val2,var2=val2
QEMU_UNSET_ENV=LD_PRELOAD,LD_DEBUG\n"
- "Note that if you provide several changes to a single variable\n"
- "the last change will stay in effect.\n");
+"The -E and -U command-line options can be provided multiple times to set\n"
+"and unset environment variables for the target process, and -E can be used\n"
+"to specify values containing commas. When using the QEMU_SET_ENV and\n"
+"QEMU_UNSET_ENV environment variables, a comma is used in getsubopt(3) style\n"
+"to set or unset several variables. The following lines are equivalent:\n"
+" -E var1=val2 -E var2=val2 -U LD_PRELOAD -U LD_DEBUG\n"
+" QEMU_SET_ENV=var1=val2,var2=val2 QEMU_UNSET_ENV=LD_PRELOAD,LD_DEBUG\n"
+"Note that if you provide several changes to a single variable, the last\n"
+"change will stay in effect.\n");
exit(1);
}
--
1.7.10.4
- Re: [Qemu-devel] Environment variables for user-mode QEMU, (continued)
- Re: [Qemu-devel] Environment variables for user-mode QEMU, Thomas Schwinge, 2013/04/24
- [Qemu-devel] [PATCH 1/4] util/envlist: Properly forward a callback's error in envlist_parse., Thomas Schwinge, 2013/04/25
- [Qemu-devel] [PATCH 2/4] linux-user: Use existing envlist_parse_set/envlist_parse_unset interface., Thomas Schwinge, 2013/04/25
- [Qemu-devel] [PATCH 4/4] linux-user: Restore original behavior of the -E and -U command-line options., Thomas Schwinge, 2013/04/25
- Re: [Qemu-devel] [PATCH 4/4] linux-user: Restore original behavior of the -E and -U command-line options., Peter Maydell, 2013/04/25
- Re: [Qemu-devel] [PATCH 4/4] linux-user: Restore original behavior of the -E and -U command-line options., Thomas Schwinge, 2013/04/25
- Re: [Qemu-devel] [PATCH 4/4] linux-user: Restore original behavior of the -E and -U command-line options., Peter Maydell, 2013/04/25
- [Qemu-devel] [PATCH] qemu-doc: Option -ignore-environment removed., Thomas Schwinge, 2013/04/26
- Re: [Qemu-devel] [PATCH] qemu-doc: Option -ignore-environment removed., Peter Maydell, 2013/04/25
- Re: [Qemu-devel] [PATCH] qemu-doc: Option -ignore-environment removed., Stefan Hajnoczi, 2013/04/26
- [Qemu-devel] [PATCH v2] linux-user: Restore original behavior of the -E and -U command-line options.,
Thomas Schwinge <=
- [Qemu-devel] [PATCH 3/4] linux-user: Tell handler_arg_* which context they're invoked from., Thomas Schwinge, 2013/04/25
Re: [Qemu-devel] Environment variables for user-mode QEMU, Riku Voipio, 2013/04/25