diff options
| author | Matthias Clasen <mclasen@redhat.com> | 2018-11-09 08:49:23 -0500 |
|---|---|---|
| committer | Matthias Clasen <mclasen@redhat.com> | 2018-11-09 08:49:23 -0500 |
| commit | 945432210071128e0ce583fc2a7adf29933afcb8 (patch) | |
| tree | 309f61d18f5d6f032cfc8f395f5872635a485362 | |
| parent | 610bc8a53223bc09cc721bf327c096c4e161e4ba (diff) | |
| download | flatpak-945432210071128e0ce583fc2a7adf29933afcb8.tar.gz | |
Fix misleading error messagemisleading-error
The error messages here make it appear that values can
be prepended with a '!', but a few lines down we throw
an error if thats the case.
| -rw-r--r-- | common/flatpak-context.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/common/flatpak-context.c b/common/flatpak-context.c index 7289368d..986e5833 100644 --- a/common/flatpak-context.c +++ b/common/flatpak-context.c @@ -1114,7 +1114,7 @@ option_add_generic_policy_cb (const gchar *option_name, if (t == NULL) { g_set_error (error, G_OPTION_ERROR, G_OPTION_ERROR_BAD_VALUE, - _("--policy arguments must be in the form SUBSYSTEM.KEY=[!]VALUE")); + _("--add-policy arguments must be in the form SUBSYSTEM.KEY=VALUE")); return FALSE; } policy_value = t + 1; @@ -1122,14 +1122,14 @@ option_add_generic_policy_cb (const gchar *option_name, if (strchr (key, '.') == NULL) { g_set_error (error, G_OPTION_ERROR, G_OPTION_ERROR_BAD_VALUE, - _("--policy arguments must be in the form SUBSYSTEM.KEY=[!]VALUE")); + _("--add-policy arguments must be in the form SUBSYSTEM.KEY=VALUE")); return FALSE; } if (policy_value[0] == '!') { g_set_error (error, G_OPTION_ERROR, G_OPTION_ERROR_BAD_VALUE, - _("--policy values can't start with \"!\"")); + _("--add-policy values can't start with \"!\"")); return FALSE; } @@ -1154,7 +1154,7 @@ option_remove_generic_policy_cb (const gchar *option_name, if (t == NULL) { g_set_error (error, G_OPTION_ERROR, G_OPTION_ERROR_BAD_VALUE, - _("--policy arguments must be in the form SUBSYSTEM.KEY=[!]VALUE")); + _("--remove-policy arguments must be in the form SUBSYSTEM.KEY=VALUE")); return FALSE; } policy_value = t + 1; @@ -1162,14 +1162,14 @@ option_remove_generic_policy_cb (const gchar *option_name, if (strchr (key, '.') == NULL) { g_set_error (error, G_OPTION_ERROR, G_OPTION_ERROR_BAD_VALUE, - _("--policy arguments must be in the form SUBSYSTEM.KEY=[!]VALUE")); + _("--remove-policy arguments must be in the form SUBSYSTEM.KEY=VALUE")); return FALSE; } if (policy_value[0] == '!') { g_set_error (error, G_OPTION_ERROR, G_OPTION_ERROR_BAD_VALUE, - _("--policy values can't start with \"!\"")); + _("--remove-policy values can't start with \"!\"")); return FALSE; } |