[Spice-devel] [PATCH v2 x11spice 3/4] Bug fix: support --config=<filename> semantics.
Jeremy White
jwhite at codeweavers.com
Thu Jul 18 21:14:04 UTC 2019
Instead of trying to parse argv ourselves, just reuse getopt_long_only.
Signed-off-by: Jeremy White <jwhite at codeweavers.com>
---
src/main.c | 8 +++++---
src/options.c | 50 ++++++++++++++++++++++++++++++++++++--------------
src/options.h | 2 +-
3 files changed, 42 insertions(+), 18 deletions(-)
diff --git a/src/main.c b/src/main.c
index f18311c9..72aba5a8 100644
--- a/src/main.c
+++ b/src/main.c
@@ -67,9 +67,11 @@ int main(int argc, char *argv[])
** Parse arguments
**----------------------------------------------------------------------*/
options_init(&session.options);
- options_handle_user_config(argc, argv, &session.options);
- options_from_config(&session.options);
- rc = options_parse_arguments(argc, argv, &session.options);
+ rc = options_handle_user_config(argc, argv, &session.options);
+ if (rc == 0) {
+ options_from_config(&session.options);
+ rc = options_parse_arguments(argc, argv, &session.options);
+ }
if (rc)
goto exit;
diff --git a/src/options.c b/src/options.c
index 303c07de..5a55f2c9 100644
--- a/src/options.c
+++ b/src/options.c
@@ -224,16 +224,6 @@ void options_handle_ssl_file_options(options_t *options,
options->ssl.ciphersuite = string_option(userkey, systemkey, "ssl", "ciphersuite");
}
-void options_handle_user_config(int argc, char *argv[], options_t *options)
-{
- int i;
- for (i = 1; i < argc - 1; i++)
- if (strcmp(argv[i], "--config") == 0 || strcmp(argv[i], "-config") == 0) {
- options->user_config_file = g_strdup(argv[i + 1]);
- i++;
- }
-}
-
int options_parse_arguments(int argc, char *argv[], options_t *options)
{
int rc;
@@ -264,6 +254,7 @@ int options_parse_arguments(int argc, char *argv[], options_t *options)
{0, 0, 0, 0}
};
+ optind = 1; /* Allow reuse of this function */
while (1) {
rc = getopt_long_only(argc, argv, "", long_options, &longindex);
if (rc == -1) {
@@ -297,7 +288,9 @@ int options_parse_arguments(int argc, char *argv[], options_t *options)
break;
case OPTION_CONFIG:
- /* This was handled previously; we can ignore */
+ if (options->user_config_file)
+ g_free(options->user_config_file);
+ options->user_config_file = g_strdup(optarg);
break;
case OPTION_SSL:
@@ -358,6 +351,21 @@ int options_parse_arguments(int argc, char *argv[], options_t *options)
return rc;
}
+/* The idea is to have command line arguments run after all other
+** configuration, with one exception - we want to grab any
+** user specified configuration file first. */
+int options_handle_user_config(int argc, char *argv[], options_t *options)
+{
+ options_t tmp_options = { 0 };
+ int rc;
+
+ rc = options_parse_arguments(argc, argv, &tmp_options);
+ options->user_config_file = tmp_options.user_config_file;
+ tmp_options.user_config_file = NULL;
+ options_free(&tmp_options);
+ return rc;
+}
+
void options_from_config(options_t *options)
{
GKeyFile *userkey = g_key_file_new();
@@ -517,11 +525,25 @@ int options_impossible_config(options_t *options)
int main(int argc, char *argv[])
{
options_t options;
+ int rc;
options_init(&options);
- options_parse_arguments(argc, argv, &options);
- options_from_config(&options);
- g_message("Options parsed");
+ rc = options_handle_user_config(argc, argv, &options);
+ if (rc == 0) {
+ options_from_config(&options);
+ rc = options_parse_arguments(argc, argv, &options);
+ }
+ if (rc == 0) {
+ g_message("Options parsed");
+ rc = options_process_io(&options);
+ if (rc == 0)
+ g_message("Options io run parsed");
+ else
+ g_message("Options io failed");
+ }
+ else {
+ g_message("Options parse failed");
+ }
options_free(&options);
}
#endif
diff --git a/src/options.h b/src/options.h
index 6155984b..162643ac 100644
--- a/src/options.h
+++ b/src/options.h
@@ -73,7 +73,7 @@ typedef struct {
** Prototypes
**--------------------------------------------------------------------------*/
void options_init(options_t *options);
-void options_handle_user_config(int argc, char *argv[], options_t *options);
+int options_handle_user_config(int argc, char *argv[], options_t *options);
int options_parse_arguments(int argc, char *argv[], options_t *options);
int options_process_io(options_t *options);
void options_free(options_t *options);
--
2.11.0
More information about the Spice-devel
mailing list