2007-10-15 01:35:37 +02:00
|
|
|
#include "git-compat-util.h"
|
|
|
|
#include "parse-options.h"
|
2008-06-23 22:55:11 +02:00
|
|
|
#include "cache.h"
|
2009-01-26 15:13:23 +01:00
|
|
|
#include "commit.h"
|
Add an optional argument for --color options
Make git-branch, git-show-branch, git-grep, and all the diff-based
programs accept an optional argument <when> for --color. The argument
is a colorbool: "always", "never", or "auto". If no argument is given,
"always" is used; --no-color is an alias for --color=never. This makes
the command-line interface consistent with other GNU tools, such as `ls'
and `grep', and with the git-config color options. Note that, without
an argument, --color and --no-color work exactly as before.
To implement this, two internal changes were made:
1. Allow the first argument of git_config_colorbool() to be NULL,
in which case it returns -1 if the argument isn't "always", "never",
or "auto".
2. Add OPT_COLOR_FLAG(), OPT__COLOR(), and parse_opt_color_flag_cb()
to the option parsing library. The callback uses
git_config_colorbool(), so color.h is now a dependency
of parse-options.c.
Signed-off-by: Mark Lodato <lodatom@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2010-02-17 05:55:58 +01:00
|
|
|
#include "color.h"
|
2007-10-15 01:35:37 +02:00
|
|
|
|
2010-06-12 14:57:39 +02:00
|
|
|
static int parse_options_usage(struct parse_opt_ctx_t *ctx,
|
|
|
|
const char * const *usagestr,
|
2010-05-17 17:34:41 +02:00
|
|
|
const struct option *opts, int err);
|
2010-01-12 07:28:45 +01:00
|
|
|
|
2007-10-15 01:35:37 +02:00
|
|
|
#define OPT_SHORT 1
|
|
|
|
#define OPT_UNSET 2
|
|
|
|
|
2011-08-11 11:15:37 +02:00
|
|
|
int optbug(const struct option *opt, const char *reason)
|
2010-12-02 07:01:18 +01:00
|
|
|
{
|
|
|
|
if (opt->long_name)
|
|
|
|
return error("BUG: option '%s' %s", opt->long_name, reason);
|
|
|
|
return error("BUG: switch '%c' %s", opt->short_name, reason);
|
|
|
|
}
|
|
|
|
|
2011-08-11 11:15:37 +02:00
|
|
|
int opterror(const struct option *opt, const char *reason, int flags)
|
2007-10-15 01:35:37 +02:00
|
|
|
{
|
|
|
|
if (flags & OPT_SHORT)
|
|
|
|
return error("switch `%c' %s", opt->short_name, reason);
|
|
|
|
if (flags & OPT_UNSET)
|
|
|
|
return error("option `no-%s' %s", opt->long_name, reason);
|
|
|
|
return error("option `%s' %s", opt->long_name, reason);
|
|
|
|
}
|
|
|
|
|
2008-07-08 12:34:08 +02:00
|
|
|
static int get_arg(struct parse_opt_ctx_t *p, const struct option *opt,
|
|
|
|
int flags, const char **arg)
|
|
|
|
{
|
|
|
|
if (p->opt) {
|
|
|
|
*arg = p->opt;
|
|
|
|
p->opt = NULL;
|
|
|
|
} else if (p->argc == 1 && (opt->flags & PARSE_OPT_LASTARG_DEFAULT)) {
|
|
|
|
*arg = (const char *)opt->defval;
|
2008-07-21 20:30:36 +02:00
|
|
|
} else if (p->argc > 1) {
|
2008-07-08 12:34:08 +02:00
|
|
|
p->argc--;
|
|
|
|
*arg = *++p->argv;
|
|
|
|
} else
|
|
|
|
return opterror(opt, "requires a value", flags);
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2009-05-23 20:53:13 +02:00
|
|
|
static void fix_filename(const char *prefix, const char **file)
|
|
|
|
{
|
|
|
|
if (!file || !*file || !prefix || is_absolute_path(*file)
|
|
|
|
|| !strcmp("-", *file))
|
|
|
|
return;
|
|
|
|
*file = xstrdup(prefix_filename(prefix, strlen(prefix), *file));
|
|
|
|
}
|
|
|
|
|
2008-06-23 21:59:37 +02:00
|
|
|
static int get_value(struct parse_opt_ctx_t *p,
|
2008-07-08 12:34:08 +02:00
|
|
|
const struct option *opt, int flags)
|
2007-10-15 01:35:37 +02:00
|
|
|
{
|
2007-10-15 01:45:45 +02:00
|
|
|
const char *s, *arg;
|
2007-11-07 11:20:27 +01:00
|
|
|
const int unset = flags & OPT_UNSET;
|
2009-05-23 20:53:13 +02:00
|
|
|
int err;
|
2007-10-15 01:35:37 +02:00
|
|
|
|
2007-11-07 11:20:27 +01:00
|
|
|
if (unset && p->opt)
|
2007-10-15 01:35:37 +02:00
|
|
|
return opterror(opt, "takes no value", flags);
|
2007-11-07 11:20:27 +01:00
|
|
|
if (unset && (opt->flags & PARSE_OPT_NONEG))
|
|
|
|
return opterror(opt, "isn't available", flags);
|
2010-12-02 00:30:40 +01:00
|
|
|
if (!(flags & OPT_SHORT) && p->opt && (opt->flags & PARSE_OPT_NOARG))
|
|
|
|
return opterror(opt, "takes no value", flags);
|
2007-11-07 11:20:27 +01:00
|
|
|
|
|
|
|
switch (opt->type) {
|
parse-options: allow git commands to invent new option types
parse-options provides a variety of option behaviors, including
OPTION_CALLBACK, which should take care of just about any sane
behavior. All supported behaviors obey the following constraint:
A --foo option can only accept (and base its behavior on)
one argument, which would be the following command-line
argument in the "unsticked" form.
Alas, some existing git commands have options that do not obey that
constraint. For example, update-index --cacheinfo takes three
arguments, and update-index --resolve takes all later parameters as
arguments.
Introduces an OPTION_LOWLEVEL_CALLBACK backdoor to parse-options so
such option types can be supported without tempting inventors of other
commands through mention in the public API. Commands can set the
callback field to a function accepting three arguments: the option
parsing context, the option itself, and a flag indicating whether the
the option was negated. When the option is encountered, that function
is called to take over from get_value(). The return value should be
zero for success, -1 for usage errors.
Thanks to Stephen Boyd for API guidance.
Improved-by: Stephen Boyd <bebarino@gmail.com>
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2010-12-02 00:32:16 +01:00
|
|
|
case OPTION_LOWLEVEL_CALLBACK:
|
|
|
|
return (*(parse_opt_ll_cb *)opt->callback)(p, opt, unset);
|
|
|
|
|
2007-11-07 11:20:27 +01:00
|
|
|
case OPTION_BIT:
|
|
|
|
if (unset)
|
|
|
|
*(int *)opt->value &= ~opt->defval;
|
2007-10-15 01:35:37 +02:00
|
|
|
else
|
2007-11-07 11:20:27 +01:00
|
|
|
*(int *)opt->value |= opt->defval;
|
|
|
|
return 0;
|
|
|
|
|
2009-05-07 21:44:17 +02:00
|
|
|
case OPTION_NEGBIT:
|
|
|
|
if (unset)
|
|
|
|
*(int *)opt->value |= opt->defval;
|
|
|
|
else
|
|
|
|
*(int *)opt->value &= ~opt->defval;
|
|
|
|
return 0;
|
|
|
|
|
parse-options: deprecate OPT_BOOLEAN
It is natural to expect that an option defined with OPT_BOOLEAN() could be
used in this way:
int option = -1; /* unspecified */
struct option options[] = {
OPT_BOOLEAN(0, "option", &option, "set option"),
OPT_END()
};
parse_options(ac, av, prefix, options, usage, 0);
if (option < 0)
... do the default thing ...
else if (!option)
... --no-option was given ...
else
... --option was given ...
to easily tell three cases apart:
- There is no mention of the `--option` on the command line;
- The variable is positively set with `--option`; or
- The variable is explicitly negated with `--no-option`.
Unfortunately, this is not the case. OPT_BOOLEAN() increments the variable
every time `--option` is given, and resets it to zero when `--no-option`
is given.
As a first step to remedy this, introduce a true boolean OPT_BOOL(), and
rename OPT_BOOLEAN() to OPT_COUNTUP(). To help transitioning, OPT_BOOLEAN
and OPTION_BOOLEAN are defined as deprecated synonyms to OPT_COUNTUP and
OPTION_COUNTUP respectively.
This is what db7244b (parse-options new features., 2007-11-07) from four
years ago started by marking OPTION_BOOLEAN as "INCR would have been a
better name".
Some existing users do depend on the count-up semantics; for example,
users of OPT__VERBOSE() could use it to raise the verbosity level with
repeated use of `-v` on the command line, but they probably should be
rewritten to use OPT__VERBOSITY() instead these days. I suspect that some
users of OPT__FORCE() may also use it to implement different level of
forcibleness but I didn't check.
On top of this patch, here are the remaining clean-up tasks that other
people can help:
- Look at each hit in "git grep -e OPT_BOOLEAN"; trace all uses of the
value that is set to the underlying variable, and if it can proven that
the variable is only used as a boolean, replace it with OPT_BOOL(). If
the caller does depend on the count-up semantics, replace it with
OPT_COUNTUP() instead.
- Same for OPTION_BOOLEAN; replace it with OPTION_SET_INT and arrange to
set 1 to the variable for a true boolean, and otherwise replace it with
OPTION_COUNTUP.
- Look at each hit in "git grep -e OPT__VERBOSE -e OPT__QUIET" and see if
they can be replaced with OPT__VERBOSITY().
I'll follow this message up with a separate patch as an example.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2011-09-28 01:56:49 +02:00
|
|
|
case OPTION_COUNTUP:
|
2007-11-07 11:20:27 +01:00
|
|
|
*(int *)opt->value = unset ? 0 : *(int *)opt->value + 1;
|
|
|
|
return 0;
|
|
|
|
|
|
|
|
case OPTION_SET_INT:
|
|
|
|
*(int *)opt->value = unset ? 0 : opt->defval;
|
|
|
|
return 0;
|
|
|
|
|
|
|
|
case OPTION_SET_PTR:
|
|
|
|
*(void **)opt->value = unset ? NULL : (void *)opt->defval;
|
2007-10-15 01:35:37 +02:00
|
|
|
return 0;
|
|
|
|
|
|
|
|
case OPTION_STRING:
|
2008-07-08 12:34:08 +02:00
|
|
|
if (unset)
|
2007-11-07 11:20:27 +01:00
|
|
|
*(const char **)opt->value = NULL;
|
2008-07-08 12:34:08 +02:00
|
|
|
else if (opt->flags & PARSE_OPT_OPTARG && !p->opt)
|
2007-10-15 01:45:45 +02:00
|
|
|
*(const char **)opt->value = (const char *)opt->defval;
|
2008-07-08 12:34:08 +02:00
|
|
|
else
|
|
|
|
return get_arg(p, opt, flags, (const char **)opt->value);
|
2007-10-15 01:35:37 +02:00
|
|
|
return 0;
|
|
|
|
|
2009-05-23 20:53:13 +02:00
|
|
|
case OPTION_FILENAME:
|
|
|
|
err = 0;
|
|
|
|
if (unset)
|
|
|
|
*(const char **)opt->value = NULL;
|
|
|
|
else if (opt->flags & PARSE_OPT_OPTARG && !p->opt)
|
|
|
|
*(const char **)opt->value = (const char *)opt->defval;
|
|
|
|
else
|
|
|
|
err = get_arg(p, opt, flags, (const char **)opt->value);
|
|
|
|
|
|
|
|
if (!err)
|
|
|
|
fix_filename(p->prefix, (const char **)opt->value);
|
|
|
|
return err;
|
|
|
|
|
2007-10-15 01:45:45 +02:00
|
|
|
case OPTION_CALLBACK:
|
2007-11-07 11:20:27 +01:00
|
|
|
if (unset)
|
2008-06-23 22:46:36 +02:00
|
|
|
return (*opt->callback)(opt, NULL, 1) ? (-1) : 0;
|
2007-11-07 11:20:27 +01:00
|
|
|
if (opt->flags & PARSE_OPT_NOARG)
|
2008-06-23 22:46:36 +02:00
|
|
|
return (*opt->callback)(opt, NULL, 0) ? (-1) : 0;
|
2007-12-21 11:41:41 +01:00
|
|
|
if (opt->flags & PARSE_OPT_OPTARG && !p->opt)
|
2008-06-23 22:46:36 +02:00
|
|
|
return (*opt->callback)(opt, NULL, 0) ? (-1) : 0;
|
2008-07-08 12:34:08 +02:00
|
|
|
if (get_arg(p, opt, flags, &arg))
|
|
|
|
return -1;
|
|
|
|
return (*opt->callback)(opt, arg, 0) ? (-1) : 0;
|
2007-10-15 01:45:45 +02:00
|
|
|
|
2007-10-15 01:35:37 +02:00
|
|
|
case OPTION_INTEGER:
|
2007-11-07 11:20:27 +01:00
|
|
|
if (unset) {
|
2007-10-15 01:35:37 +02:00
|
|
|
*(int *)opt->value = 0;
|
|
|
|
return 0;
|
|
|
|
}
|
2007-12-21 11:41:41 +01:00
|
|
|
if (opt->flags & PARSE_OPT_OPTARG && !p->opt) {
|
2007-10-15 01:45:45 +02:00
|
|
|
*(int *)opt->value = opt->defval;
|
|
|
|
return 0;
|
|
|
|
}
|
2008-07-08 12:34:08 +02:00
|
|
|
if (get_arg(p, opt, flags, &arg))
|
|
|
|
return -1;
|
|
|
|
*(int *)opt->value = strtol(arg, (char **)&s, 10);
|
2007-10-15 01:35:37 +02:00
|
|
|
if (*s)
|
|
|
|
return opterror(opt, "expects a numerical value", flags);
|
|
|
|
return 0;
|
|
|
|
|
|
|
|
default:
|
|
|
|
die("should not happen, someone must be hit on the forehead");
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2008-06-23 21:59:37 +02:00
|
|
|
static int parse_short_opt(struct parse_opt_ctx_t *p, const struct option *options)
|
2007-10-15 01:35:37 +02:00
|
|
|
{
|
2009-05-07 21:45:08 +02:00
|
|
|
const struct option *numopt = NULL;
|
|
|
|
|
2007-10-15 01:35:37 +02:00
|
|
|
for (; options->type != OPTION_END; options++) {
|
|
|
|
if (options->short_name == *p->opt) {
|
|
|
|
p->opt = p->opt[1] ? p->opt + 1 : NULL;
|
|
|
|
return get_value(p, options, OPT_SHORT);
|
|
|
|
}
|
2009-05-07 21:45:08 +02:00
|
|
|
|
|
|
|
/*
|
|
|
|
* Handle the numerical option later, explicit one-digit
|
|
|
|
* options take precedence over it.
|
|
|
|
*/
|
|
|
|
if (options->type == OPTION_NUMBER)
|
|
|
|
numopt = options;
|
|
|
|
}
|
|
|
|
if (numopt && isdigit(*p->opt)) {
|
|
|
|
size_t len = 1;
|
|
|
|
char *arg;
|
|
|
|
int rc;
|
|
|
|
|
|
|
|
while (isdigit(p->opt[len]))
|
|
|
|
len++;
|
|
|
|
arg = xmemdupz(p->opt, len);
|
|
|
|
p->opt = p->opt[len] ? p->opt + len : NULL;
|
|
|
|
rc = (*numopt->callback)(numopt, arg, 0) ? (-1) : 0;
|
|
|
|
free(arg);
|
|
|
|
return rc;
|
2007-10-15 01:35:37 +02:00
|
|
|
}
|
2008-06-23 22:46:36 +02:00
|
|
|
return -2;
|
2007-10-15 01:35:37 +02:00
|
|
|
}
|
|
|
|
|
2008-06-23 21:59:37 +02:00
|
|
|
static int parse_long_opt(struct parse_opt_ctx_t *p, const char *arg,
|
2007-10-15 01:35:37 +02:00
|
|
|
const struct option *options)
|
|
|
|
{
|
2007-10-14 18:54:06 +02:00
|
|
|
const char *arg_end = strchr(arg, '=');
|
2007-11-05 14:15:21 +01:00
|
|
|
const struct option *abbrev_option = NULL, *ambiguous_option = NULL;
|
|
|
|
int abbrev_flags = 0, ambiguous_flags = 0;
|
2007-10-14 18:54:06 +02:00
|
|
|
|
|
|
|
if (!arg_end)
|
|
|
|
arg_end = arg + strlen(arg);
|
|
|
|
|
2007-10-15 01:35:37 +02:00
|
|
|
for (; options->type != OPTION_END; options++) {
|
2012-02-25 20:14:54 +01:00
|
|
|
const char *rest, *long_name = options->long_name;
|
|
|
|
int flags = 0, opt_flags = 0;
|
2007-10-15 01:35:37 +02:00
|
|
|
|
2012-02-25 20:14:54 +01:00
|
|
|
if (!long_name)
|
2007-10-15 01:35:37 +02:00
|
|
|
continue;
|
|
|
|
|
2012-02-25 20:14:54 +01:00
|
|
|
again:
|
|
|
|
rest = skip_prefix(arg, long_name);
|
2008-03-02 11:35:56 +01:00
|
|
|
if (options->type == OPTION_ARGUMENT) {
|
|
|
|
if (!rest)
|
|
|
|
continue;
|
|
|
|
if (*rest == '=')
|
|
|
|
return opterror(options, "takes no value", flags);
|
|
|
|
if (*rest)
|
|
|
|
continue;
|
|
|
|
p->out[p->cpidx++] = arg - 2;
|
|
|
|
return 0;
|
|
|
|
}
|
2007-10-15 01:35:37 +02:00
|
|
|
if (!rest) {
|
2007-10-14 18:54:06 +02:00
|
|
|
/* abbreviated? */
|
2012-02-25 20:14:54 +01:00
|
|
|
if (!strncmp(long_name, arg, arg_end - arg)) {
|
2007-10-14 18:54:06 +02:00
|
|
|
is_abbreviated:
|
2007-11-05 14:15:21 +01:00
|
|
|
if (abbrev_option) {
|
|
|
|
/*
|
|
|
|
* If this is abbreviated, it is
|
|
|
|
* ambiguous. So when there is no
|
|
|
|
* exact match later, we need to
|
|
|
|
* error out.
|
|
|
|
*/
|
|
|
|
ambiguous_option = abbrev_option;
|
|
|
|
ambiguous_flags = abbrev_flags;
|
|
|
|
}
|
2007-10-14 18:54:06 +02:00
|
|
|
if (!(flags & OPT_UNSET) && *arg_end)
|
|
|
|
p->opt = arg_end + 1;
|
|
|
|
abbrev_option = options;
|
2012-02-25 20:14:54 +01:00
|
|
|
abbrev_flags = flags ^ opt_flags;
|
2007-10-14 18:54:06 +02:00
|
|
|
continue;
|
|
|
|
}
|
2009-09-25 20:44:44 +02:00
|
|
|
/* negation allowed? */
|
|
|
|
if (options->flags & PARSE_OPT_NONEG)
|
|
|
|
continue;
|
2007-10-14 18:54:06 +02:00
|
|
|
/* negated and abbreviated very much? */
|
|
|
|
if (!prefixcmp("no-", arg)) {
|
|
|
|
flags |= OPT_UNSET;
|
|
|
|
goto is_abbreviated;
|
|
|
|
}
|
|
|
|
/* negated? */
|
2012-02-25 20:14:54 +01:00
|
|
|
if (prefixcmp(arg, "no-")) {
|
|
|
|
if (!prefixcmp(long_name, "no-")) {
|
|
|
|
long_name += 3;
|
|
|
|
opt_flags |= OPT_UNSET;
|
|
|
|
goto again;
|
|
|
|
}
|
2007-10-15 01:35:37 +02:00
|
|
|
continue;
|
2012-02-25 20:14:54 +01:00
|
|
|
}
|
2007-10-15 01:35:37 +02:00
|
|
|
flags |= OPT_UNSET;
|
2012-02-25 20:14:54 +01:00
|
|
|
rest = skip_prefix(arg + 3, long_name);
|
2007-10-14 18:54:06 +02:00
|
|
|
/* abbreviated and negated? */
|
2012-02-25 20:14:54 +01:00
|
|
|
if (!rest && !prefixcmp(long_name, arg + 3))
|
2007-10-14 18:54:06 +02:00
|
|
|
goto is_abbreviated;
|
2007-10-15 01:35:37 +02:00
|
|
|
if (!rest)
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
if (*rest) {
|
|
|
|
if (*rest != '=')
|
|
|
|
continue;
|
|
|
|
p->opt = rest + 1;
|
|
|
|
}
|
2012-02-25 20:14:54 +01:00
|
|
|
return get_value(p, options, flags ^ opt_flags);
|
2007-10-15 01:35:37 +02:00
|
|
|
}
|
2007-11-05 14:15:21 +01:00
|
|
|
|
|
|
|
if (ambiguous_option)
|
|
|
|
return error("Ambiguous option: %s "
|
|
|
|
"(could be --%s%s or --%s%s)",
|
|
|
|
arg,
|
|
|
|
(ambiguous_flags & OPT_UNSET) ? "no-" : "",
|
|
|
|
ambiguous_option->long_name,
|
|
|
|
(abbrev_flags & OPT_UNSET) ? "no-" : "",
|
|
|
|
abbrev_option->long_name);
|
2007-10-14 18:54:06 +02:00
|
|
|
if (abbrev_option)
|
|
|
|
return get_value(p, abbrev_option, abbrev_flags);
|
2008-06-23 22:46:36 +02:00
|
|
|
return -2;
|
2007-10-15 01:35:37 +02:00
|
|
|
}
|
|
|
|
|
2009-05-07 21:45:42 +02:00
|
|
|
static int parse_nodash_opt(struct parse_opt_ctx_t *p, const char *arg,
|
|
|
|
const struct option *options)
|
|
|
|
{
|
|
|
|
for (; options->type != OPTION_END; options++) {
|
|
|
|
if (!(options->flags & PARSE_OPT_NODASH))
|
|
|
|
continue;
|
|
|
|
if (options->short_name == arg[0] && arg[1] == '\0')
|
|
|
|
return get_value(p, options, OPT_SHORT);
|
|
|
|
}
|
|
|
|
return -2;
|
|
|
|
}
|
|
|
|
|
2008-07-16 12:42:18 +02:00
|
|
|
static void check_typos(const char *arg, const struct option *options)
|
2008-01-26 12:26:57 +01:00
|
|
|
{
|
|
|
|
if (strlen(arg) < 3)
|
|
|
|
return;
|
|
|
|
|
|
|
|
if (!prefixcmp(arg, "no-")) {
|
|
|
|
error ("did you mean `--%s` (with two dashes ?)", arg);
|
|
|
|
exit(129);
|
|
|
|
}
|
|
|
|
|
|
|
|
for (; options->type != OPTION_END; options++) {
|
|
|
|
if (!options->long_name)
|
|
|
|
continue;
|
|
|
|
if (!prefixcmp(options->long_name, arg)) {
|
|
|
|
error ("did you mean `--%s` (with two dashes ?)", arg);
|
|
|
|
exit(129);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2009-06-09 10:23:44 +02:00
|
|
|
static void parse_options_check(const struct option *opts)
|
|
|
|
{
|
|
|
|
int err = 0;
|
|
|
|
|
|
|
|
for (; opts->type != OPTION_END; opts++) {
|
|
|
|
if ((opts->flags & PARSE_OPT_LASTARG_DEFAULT) &&
|
2010-12-02 07:01:18 +01:00
|
|
|
(opts->flags & PARSE_OPT_OPTARG))
|
|
|
|
err |= optbug(opts, "uses incompatible flags "
|
|
|
|
"LASTARG_DEFAULT and OPTARG");
|
2010-12-02 07:05:05 +01:00
|
|
|
if (opts->flags & PARSE_OPT_NODASH &&
|
|
|
|
((opts->flags & PARSE_OPT_OPTARG) ||
|
|
|
|
!(opts->flags & PARSE_OPT_NOARG) ||
|
|
|
|
!(opts->flags & PARSE_OPT_NONEG) ||
|
|
|
|
opts->long_name))
|
|
|
|
err |= optbug(opts, "uses feature "
|
|
|
|
"not supported for dashless options");
|
2010-12-02 07:08:57 +01:00
|
|
|
switch (opts->type) {
|
parse-options: deprecate OPT_BOOLEAN
It is natural to expect that an option defined with OPT_BOOLEAN() could be
used in this way:
int option = -1; /* unspecified */
struct option options[] = {
OPT_BOOLEAN(0, "option", &option, "set option"),
OPT_END()
};
parse_options(ac, av, prefix, options, usage, 0);
if (option < 0)
... do the default thing ...
else if (!option)
... --no-option was given ...
else
... --option was given ...
to easily tell three cases apart:
- There is no mention of the `--option` on the command line;
- The variable is positively set with `--option`; or
- The variable is explicitly negated with `--no-option`.
Unfortunately, this is not the case. OPT_BOOLEAN() increments the variable
every time `--option` is given, and resets it to zero when `--no-option`
is given.
As a first step to remedy this, introduce a true boolean OPT_BOOL(), and
rename OPT_BOOLEAN() to OPT_COUNTUP(). To help transitioning, OPT_BOOLEAN
and OPTION_BOOLEAN are defined as deprecated synonyms to OPT_COUNTUP and
OPTION_COUNTUP respectively.
This is what db7244b (parse-options new features., 2007-11-07) from four
years ago started by marking OPTION_BOOLEAN as "INCR would have been a
better name".
Some existing users do depend on the count-up semantics; for example,
users of OPT__VERBOSE() could use it to raise the verbosity level with
repeated use of `-v` on the command line, but they probably should be
rewritten to use OPT__VERBOSITY() instead these days. I suspect that some
users of OPT__FORCE() may also use it to implement different level of
forcibleness but I didn't check.
On top of this patch, here are the remaining clean-up tasks that other
people can help:
- Look at each hit in "git grep -e OPT_BOOLEAN"; trace all uses of the
value that is set to the underlying variable, and if it can proven that
the variable is only used as a boolean, replace it with OPT_BOOL(). If
the caller does depend on the count-up semantics, replace it with
OPT_COUNTUP() instead.
- Same for OPTION_BOOLEAN; replace it with OPTION_SET_INT and arrange to
set 1 to the variable for a true boolean, and otherwise replace it with
OPTION_COUNTUP.
- Look at each hit in "git grep -e OPT__VERBOSE -e OPT__QUIET" and see if
they can be replaced with OPT__VERBOSITY().
I'll follow this message up with a separate patch as an example.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2011-09-28 01:56:49 +02:00
|
|
|
case OPTION_COUNTUP:
|
2010-12-02 07:08:57 +01:00
|
|
|
case OPTION_BIT:
|
|
|
|
case OPTION_NEGBIT:
|
|
|
|
case OPTION_SET_INT:
|
|
|
|
case OPTION_SET_PTR:
|
|
|
|
case OPTION_NUMBER:
|
|
|
|
if ((opts->flags & PARSE_OPT_OPTARG) ||
|
|
|
|
!(opts->flags & PARSE_OPT_NOARG))
|
|
|
|
err |= optbug(opts, "should not accept an argument");
|
|
|
|
default:
|
|
|
|
; /* ok. (usually accepts an argument) */
|
|
|
|
}
|
2009-06-09 10:23:44 +02:00
|
|
|
}
|
|
|
|
if (err)
|
2010-12-02 07:01:18 +01:00
|
|
|
exit(128);
|
2009-06-09 10:23:44 +02:00
|
|
|
}
|
|
|
|
|
2008-06-23 21:59:37 +02:00
|
|
|
void parse_options_start(struct parse_opt_ctx_t *ctx,
|
2009-05-23 20:53:12 +02:00
|
|
|
int argc, const char **argv, const char *prefix,
|
2010-12-06 08:57:42 +01:00
|
|
|
const struct option *options, int flags)
|
2008-06-23 21:59:37 +02:00
|
|
|
{
|
|
|
|
memset(ctx, 0, sizeof(*ctx));
|
|
|
|
ctx->argc = argc - 1;
|
|
|
|
ctx->argv = argv + 1;
|
|
|
|
ctx->out = argv;
|
2009-05-23 20:53:12 +02:00
|
|
|
ctx->prefix = prefix;
|
2008-06-24 00:31:31 +02:00
|
|
|
ctx->cpidx = ((flags & PARSE_OPT_KEEP_ARGV0) != 0);
|
2008-06-23 21:59:37 +02:00
|
|
|
ctx->flags = flags;
|
2009-03-09 21:57:38 +01:00
|
|
|
if ((flags & PARSE_OPT_KEEP_UNKNOWN) &&
|
|
|
|
(flags & PARSE_OPT_STOP_AT_NON_OPTION))
|
|
|
|
die("STOP_AT_NON_OPTION and KEEP_UNKNOWN don't go together");
|
2010-12-06 08:57:42 +01:00
|
|
|
parse_options_check(options);
|
2008-06-23 21:59:37 +02:00
|
|
|
}
|
|
|
|
|
2010-06-12 14:57:39 +02:00
|
|
|
static int usage_with_options_internal(struct parse_opt_ctx_t *,
|
|
|
|
const char * const *,
|
2010-05-17 17:34:41 +02:00
|
|
|
const struct option *, int, int);
|
2007-11-19 10:21:44 +01:00
|
|
|
|
2008-06-23 22:38:58 +02:00
|
|
|
int parse_options_step(struct parse_opt_ctx_t *ctx,
|
|
|
|
const struct option *options,
|
|
|
|
const char * const usagestr[])
|
2007-10-15 01:35:37 +02:00
|
|
|
{
|
2009-03-08 19:15:08 +01:00
|
|
|
int internal_help = !(ctx->flags & PARSE_OPT_NO_INTERNAL_HELP);
|
|
|
|
|
2008-06-23 22:55:11 +02:00
|
|
|
/* we must reset ->opt, unknown short option leave it dangling */
|
|
|
|
ctx->opt = NULL;
|
|
|
|
|
2008-06-23 22:38:58 +02:00
|
|
|
for (; ctx->argc; ctx->argc--, ctx->argv++) {
|
|
|
|
const char *arg = ctx->argv[0];
|
2007-10-15 01:35:37 +02:00
|
|
|
|
|
|
|
if (*arg != '-' || !arg[1]) {
|
2009-05-07 21:45:42 +02:00
|
|
|
if (parse_nodash_opt(ctx, arg, options) == 0)
|
|
|
|
continue;
|
2008-06-23 22:38:58 +02:00
|
|
|
if (ctx->flags & PARSE_OPT_STOP_AT_NON_OPTION)
|
2010-12-02 00:32:55 +01:00
|
|
|
return PARSE_OPT_NON_OPTION;
|
2008-06-23 22:38:58 +02:00
|
|
|
ctx->out[ctx->cpidx++] = ctx->argv[0];
|
2007-10-15 01:35:37 +02:00
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (arg[1] != '-') {
|
2008-06-23 22:38:58 +02:00
|
|
|
ctx->opt = arg + 1;
|
2009-03-08 19:15:08 +01:00
|
|
|
if (internal_help && *ctx->opt == 'h')
|
2010-06-12 14:57:39 +02:00
|
|
|
return parse_options_usage(ctx, usagestr, options, 0);
|
2008-06-23 22:46:36 +02:00
|
|
|
switch (parse_short_opt(ctx, options)) {
|
|
|
|
case -1:
|
2010-06-12 14:57:39 +02:00
|
|
|
return parse_options_usage(ctx, usagestr, options, 1);
|
2008-06-23 22:46:36 +02:00
|
|
|
case -2:
|
2012-03-03 12:00:29 +01:00
|
|
|
if (ctx->opt)
|
|
|
|
check_typos(arg + 1, options);
|
2009-03-08 19:12:47 +01:00
|
|
|
goto unknown;
|
2008-06-23 22:46:36 +02:00
|
|
|
}
|
2008-06-23 22:38:58 +02:00
|
|
|
if (ctx->opt)
|
2008-01-26 12:26:57 +01:00
|
|
|
check_typos(arg + 1, options);
|
2008-06-23 22:38:58 +02:00
|
|
|
while (ctx->opt) {
|
2009-03-08 19:15:08 +01:00
|
|
|
if (internal_help && *ctx->opt == 'h')
|
2010-06-12 14:57:39 +02:00
|
|
|
return parse_options_usage(ctx, usagestr, options, 0);
|
2008-06-23 22:46:36 +02:00
|
|
|
switch (parse_short_opt(ctx, options)) {
|
|
|
|
case -1:
|
2010-06-12 14:57:39 +02:00
|
|
|
return parse_options_usage(ctx, usagestr, options, 1);
|
2008-06-23 22:46:36 +02:00
|
|
|
case -2:
|
2008-06-23 22:55:11 +02:00
|
|
|
/* fake a short option thing to hide the fact that we may have
|
|
|
|
* started to parse aggregated stuff
|
|
|
|
*
|
|
|
|
* This is leaky, too bad.
|
|
|
|
*/
|
|
|
|
ctx->argv[0] = xstrdup(ctx->opt - 1);
|
|
|
|
*(char *)ctx->argv[0] = '-';
|
2009-03-08 19:12:47 +01:00
|
|
|
goto unknown;
|
2008-06-23 22:46:36 +02:00
|
|
|
}
|
2008-01-26 12:26:57 +01:00
|
|
|
}
|
2007-10-15 01:35:37 +02:00
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (!arg[2]) { /* "--" */
|
2008-06-23 22:38:58 +02:00
|
|
|
if (!(ctx->flags & PARSE_OPT_KEEP_DASHDASH)) {
|
|
|
|
ctx->argc--;
|
|
|
|
ctx->argv++;
|
2007-10-15 01:35:37 +02:00
|
|
|
}
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
|
2009-03-08 19:15:08 +01:00
|
|
|
if (internal_help && !strcmp(arg + 2, "help-all"))
|
2010-06-12 14:57:39 +02:00
|
|
|
return usage_with_options_internal(ctx, usagestr, options, 1, 0);
|
2009-03-08 19:15:08 +01:00
|
|
|
if (internal_help && !strcmp(arg + 2, "help"))
|
2010-06-12 14:57:39 +02:00
|
|
|
return parse_options_usage(ctx, usagestr, options, 0);
|
2008-06-23 22:46:36 +02:00
|
|
|
switch (parse_long_opt(ctx, arg + 2, options)) {
|
|
|
|
case -1:
|
2010-06-12 14:57:39 +02:00
|
|
|
return parse_options_usage(ctx, usagestr, options, 1);
|
2008-06-23 22:46:36 +02:00
|
|
|
case -2:
|
2009-03-08 19:12:47 +01:00
|
|
|
goto unknown;
|
2008-06-23 22:46:36 +02:00
|
|
|
}
|
2009-03-08 19:12:47 +01:00
|
|
|
continue;
|
|
|
|
unknown:
|
|
|
|
if (!(ctx->flags & PARSE_OPT_KEEP_UNKNOWN))
|
|
|
|
return PARSE_OPT_UNKNOWN;
|
|
|
|
ctx->out[ctx->cpidx++] = ctx->argv[0];
|
|
|
|
ctx->opt = NULL;
|
2008-06-23 22:38:58 +02:00
|
|
|
}
|
|
|
|
return PARSE_OPT_DONE;
|
|
|
|
}
|
|
|
|
|
|
|
|
int parse_options_end(struct parse_opt_ctx_t *ctx)
|
|
|
|
{
|
|
|
|
memmove(ctx->out + ctx->cpidx, ctx->argv, ctx->argc * sizeof(*ctx->out));
|
|
|
|
ctx->out[ctx->cpidx + ctx->argc] = NULL;
|
|
|
|
return ctx->cpidx + ctx->argc;
|
|
|
|
}
|
|
|
|
|
2009-05-23 20:53:12 +02:00
|
|
|
int parse_options(int argc, const char **argv, const char *prefix,
|
|
|
|
const struct option *options, const char * const usagestr[],
|
|
|
|
int flags)
|
2008-06-23 22:38:58 +02:00
|
|
|
{
|
|
|
|
struct parse_opt_ctx_t ctx;
|
|
|
|
|
2010-12-06 08:57:42 +01:00
|
|
|
parse_options_start(&ctx, argc, argv, prefix, options, flags);
|
2008-06-23 22:38:58 +02:00
|
|
|
switch (parse_options_step(&ctx, options, usagestr)) {
|
|
|
|
case PARSE_OPT_HELP:
|
|
|
|
exit(129);
|
2010-12-02 00:32:55 +01:00
|
|
|
case PARSE_OPT_NON_OPTION:
|
2008-06-23 22:38:58 +02:00
|
|
|
case PARSE_OPT_DONE:
|
|
|
|
break;
|
|
|
|
default: /* PARSE_OPT_UNKNOWN */
|
2008-06-23 22:46:36 +02:00
|
|
|
if (ctx.argv[0][1] == '-') {
|
|
|
|
error("unknown option `%s'", ctx.argv[0] + 2);
|
|
|
|
} else {
|
|
|
|
error("unknown switch `%c'", *ctx.opt);
|
|
|
|
}
|
|
|
|
usage_with_options(usagestr, options);
|
2007-10-15 01:35:37 +02:00
|
|
|
}
|
|
|
|
|
2008-06-23 21:59:37 +02:00
|
|
|
return parse_options_end(&ctx);
|
2007-10-15 01:35:37 +02:00
|
|
|
}
|
2007-10-15 01:38:30 +02:00
|
|
|
|
2010-05-17 17:34:41 +02:00
|
|
|
static int usage_argh(const struct option *opts, FILE *outfile)
|
2009-05-21 09:33:17 +02:00
|
|
|
{
|
|
|
|
const char *s;
|
2009-06-05 01:43:57 +02:00
|
|
|
int literal = (opts->flags & PARSE_OPT_LITERAL_ARGHELP) || !opts->argh;
|
2009-05-21 09:33:17 +02:00
|
|
|
if (opts->flags & PARSE_OPT_OPTARG)
|
|
|
|
if (opts->long_name)
|
|
|
|
s = literal ? "[=%s]" : "[=<%s>]";
|
|
|
|
else
|
|
|
|
s = literal ? "[%s]" : "[<%s>]";
|
|
|
|
else
|
|
|
|
s = literal ? " %s" : " <%s>";
|
2012-05-06 16:23:51 +02:00
|
|
|
return fprintf(outfile, s, opts->argh ? _(opts->argh) : _("..."));
|
2009-05-21 09:33:17 +02:00
|
|
|
}
|
|
|
|
|
2007-10-15 01:38:30 +02:00
|
|
|
#define USAGE_OPTS_WIDTH 24
|
|
|
|
#define USAGE_GAP 2
|
|
|
|
|
2010-06-12 14:57:39 +02:00
|
|
|
static int usage_with_options_internal(struct parse_opt_ctx_t *ctx,
|
|
|
|
const char * const *usagestr,
|
|
|
|
const struct option *opts, int full, int err)
|
2007-10-15 01:38:30 +02:00
|
|
|
{
|
2010-05-17 17:34:41 +02:00
|
|
|
FILE *outfile = err ? stderr : stdout;
|
|
|
|
|
2009-03-08 19:16:58 +01:00
|
|
|
if (!usagestr)
|
|
|
|
return PARSE_OPT_HELP;
|
|
|
|
|
2010-06-12 14:57:39 +02:00
|
|
|
if (!err && ctx && ctx->flags & PARSE_OPT_SHELL_EVAL)
|
|
|
|
fprintf(outfile, "cat <<\\EOF\n");
|
|
|
|
|
2012-05-06 16:23:51 +02:00
|
|
|
fprintf_ln(outfile, _("usage: %s"), _(*usagestr++));
|
2007-10-14 00:10:51 +02:00
|
|
|
while (*usagestr && **usagestr)
|
2012-05-06 16:23:51 +02:00
|
|
|
/* TRANSLATORS: the colon here should align with the
|
|
|
|
one in "usage: %s" translation */
|
|
|
|
fprintf_ln(outfile, _(" or: %s"), _(*usagestr++));
|
2008-06-14 09:27:21 +02:00
|
|
|
while (*usagestr) {
|
2012-05-06 16:23:51 +02:00
|
|
|
if (**usagestr)
|
|
|
|
fprintf_ln(outfile, _(" %s"), _(*usagestr));
|
|
|
|
else
|
|
|
|
putchar('\n');
|
2008-06-14 09:27:21 +02:00
|
|
|
usagestr++;
|
|
|
|
}
|
2007-10-15 01:38:30 +02:00
|
|
|
|
|
|
|
if (opts->type != OPTION_GROUP)
|
2010-05-17 17:34:41 +02:00
|
|
|
fputc('\n', outfile);
|
2007-10-15 01:38:30 +02:00
|
|
|
|
|
|
|
for (; opts->type != OPTION_END; opts++) {
|
|
|
|
size_t pos;
|
|
|
|
int pad;
|
|
|
|
|
|
|
|
if (opts->type == OPTION_GROUP) {
|
2010-05-17 17:34:41 +02:00
|
|
|
fputc('\n', outfile);
|
2007-10-15 01:38:30 +02:00
|
|
|
if (*opts->help)
|
2012-05-06 16:23:51 +02:00
|
|
|
fprintf(outfile, "%s\n", _(opts->help));
|
2007-10-15 01:38:30 +02:00
|
|
|
continue;
|
|
|
|
}
|
2007-11-19 10:21:44 +01:00
|
|
|
if (!full && (opts->flags & PARSE_OPT_HIDDEN))
|
|
|
|
continue;
|
2007-10-15 01:38:30 +02:00
|
|
|
|
2010-05-17 17:34:41 +02:00
|
|
|
pos = fprintf(outfile, " ");
|
2012-02-28 20:06:09 +01:00
|
|
|
if (opts->short_name) {
|
2009-05-07 21:45:42 +02:00
|
|
|
if (opts->flags & PARSE_OPT_NODASH)
|
2010-05-17 17:34:41 +02:00
|
|
|
pos += fprintf(outfile, "%c", opts->short_name);
|
2009-05-07 21:45:42 +02:00
|
|
|
else
|
2010-05-17 17:34:41 +02:00
|
|
|
pos += fprintf(outfile, "-%c", opts->short_name);
|
2009-05-07 21:45:42 +02:00
|
|
|
}
|
2007-10-15 01:38:30 +02:00
|
|
|
if (opts->long_name && opts->short_name)
|
2010-05-17 17:34:41 +02:00
|
|
|
pos += fprintf(outfile, ", ");
|
2007-10-15 01:38:30 +02:00
|
|
|
if (opts->long_name)
|
2012-02-28 20:06:09 +01:00
|
|
|
pos += fprintf(outfile, "--%s", opts->long_name);
|
2009-05-07 21:45:08 +02:00
|
|
|
if (opts->type == OPTION_NUMBER)
|
2010-05-17 17:34:41 +02:00
|
|
|
pos += fprintf(outfile, "-NUM");
|
2007-10-15 01:38:30 +02:00
|
|
|
|
2010-12-02 00:31:36 +01:00
|
|
|
if ((opts->flags & PARSE_OPT_LITERAL_ARGHELP) ||
|
|
|
|
!(opts->flags & PARSE_OPT_NOARG))
|
2010-05-17 17:34:41 +02:00
|
|
|
pos += usage_argh(opts, outfile);
|
2007-10-15 01:38:30 +02:00
|
|
|
|
2007-10-14 00:10:51 +02:00
|
|
|
if (pos <= USAGE_OPTS_WIDTH)
|
|
|
|
pad = USAGE_OPTS_WIDTH - pos;
|
2007-10-15 01:38:30 +02:00
|
|
|
else {
|
2010-05-17 17:34:41 +02:00
|
|
|
fputc('\n', outfile);
|
2007-10-15 01:38:30 +02:00
|
|
|
pad = USAGE_OPTS_WIDTH;
|
|
|
|
}
|
2012-05-06 16:23:51 +02:00
|
|
|
fprintf(outfile, "%*s%s\n", pad + USAGE_GAP, "", _(opts->help));
|
2007-10-15 01:38:30 +02:00
|
|
|
}
|
2010-05-17 17:34:41 +02:00
|
|
|
fputc('\n', outfile);
|
2007-10-14 00:10:51 +02:00
|
|
|
|
2010-06-12 14:57:39 +02:00
|
|
|
if (!err && ctx && ctx->flags & PARSE_OPT_SHELL_EVAL)
|
|
|
|
fputs("EOF\n", outfile);
|
|
|
|
|
2008-06-23 22:28:04 +02:00
|
|
|
return PARSE_OPT_HELP;
|
2007-10-15 01:38:30 +02:00
|
|
|
}
|
2007-10-14 11:05:12 +02:00
|
|
|
|
Fix sparse warnings
Fix warnings from 'make check'.
- These files don't include 'builtin.h' causing sparse to complain that
cmd_* isn't declared:
builtin/clone.c:364, builtin/fetch-pack.c:797,
builtin/fmt-merge-msg.c:34, builtin/hash-object.c:78,
builtin/merge-index.c:69, builtin/merge-recursive.c:22
builtin/merge-tree.c:341, builtin/mktag.c:156, builtin/notes.c:426
builtin/notes.c:822, builtin/pack-redundant.c:596,
builtin/pack-refs.c:10, builtin/patch-id.c:60, builtin/patch-id.c:149,
builtin/remote.c:1512, builtin/remote-ext.c:240,
builtin/remote-fd.c:53, builtin/reset.c:236, builtin/send-pack.c:384,
builtin/unpack-file.c:25, builtin/var.c:75
- These files have symbols which should be marked static since they're
only file scope:
submodule.c:12, diff.c:631, replace_object.c:92, submodule.c:13,
submodule.c:14, trace.c:78, transport.c:195, transport-helper.c:79,
unpack-trees.c:19, url.c:3, url.c:18, url.c:104, url.c:117, url.c:123,
url.c:129, url.c:136, thread-utils.c:21, thread-utils.c:48
- These files redeclare symbols to be different types:
builtin/index-pack.c:210, parse-options.c:564, parse-options.c:571,
usage.c:49, usage.c:58, usage.c:63, usage.c:72
- These files use a literal integer 0 when they really should use a NULL
pointer:
daemon.c:663, fast-import.c:2942, imap-send.c:1072, notes-merge.c:362
While we're in the area, clean up some unused #includes in builtin files
(mostly exec_cmd.h).
Signed-off-by: Stephen Boyd <bebarino@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2011-03-22 08:51:05 +01:00
|
|
|
void NORETURN usage_with_options(const char * const *usagestr,
|
2008-06-23 22:38:58 +02:00
|
|
|
const struct option *opts)
|
2007-11-19 10:21:44 +01:00
|
|
|
{
|
2010-06-12 14:57:39 +02:00
|
|
|
usage_with_options_internal(NULL, usagestr, opts, 0, 1);
|
2008-06-23 22:38:58 +02:00
|
|
|
exit(129);
|
2007-11-19 10:21:44 +01:00
|
|
|
}
|
|
|
|
|
Fix sparse warnings
Fix warnings from 'make check'.
- These files don't include 'builtin.h' causing sparse to complain that
cmd_* isn't declared:
builtin/clone.c:364, builtin/fetch-pack.c:797,
builtin/fmt-merge-msg.c:34, builtin/hash-object.c:78,
builtin/merge-index.c:69, builtin/merge-recursive.c:22
builtin/merge-tree.c:341, builtin/mktag.c:156, builtin/notes.c:426
builtin/notes.c:822, builtin/pack-redundant.c:596,
builtin/pack-refs.c:10, builtin/patch-id.c:60, builtin/patch-id.c:149,
builtin/remote.c:1512, builtin/remote-ext.c:240,
builtin/remote-fd.c:53, builtin/reset.c:236, builtin/send-pack.c:384,
builtin/unpack-file.c:25, builtin/var.c:75
- These files have symbols which should be marked static since they're
only file scope:
submodule.c:12, diff.c:631, replace_object.c:92, submodule.c:13,
submodule.c:14, trace.c:78, transport.c:195, transport-helper.c:79,
unpack-trees.c:19, url.c:3, url.c:18, url.c:104, url.c:117, url.c:123,
url.c:129, url.c:136, thread-utils.c:21, thread-utils.c:48
- These files redeclare symbols to be different types:
builtin/index-pack.c:210, parse-options.c:564, parse-options.c:571,
usage.c:49, usage.c:58, usage.c:63, usage.c:72
- These files use a literal integer 0 when they really should use a NULL
pointer:
daemon.c:663, fast-import.c:2942, imap-send.c:1072, notes-merge.c:362
While we're in the area, clean up some unused #includes in builtin files
(mostly exec_cmd.h).
Signed-off-by: Stephen Boyd <bebarino@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2011-03-22 08:51:05 +01:00
|
|
|
void NORETURN usage_msg_opt(const char *msg,
|
2009-02-02 06:12:58 +01:00
|
|
|
const char * const *usagestr,
|
|
|
|
const struct option *options)
|
|
|
|
{
|
|
|
|
fprintf(stderr, "%s\n\n", msg);
|
|
|
|
usage_with_options(usagestr, options);
|
|
|
|
}
|
|
|
|
|
2010-06-12 14:57:39 +02:00
|
|
|
static int parse_options_usage(struct parse_opt_ctx_t *ctx,
|
|
|
|
const char * const *usagestr,
|
2010-05-17 17:34:41 +02:00
|
|
|
const struct option *opts, int err)
|
2008-06-23 22:28:04 +02:00
|
|
|
{
|
2010-06-12 14:57:39 +02:00
|
|
|
return usage_with_options_internal(ctx, usagestr, opts, 0, err);
|
2008-06-23 22:28:04 +02:00
|
|
|
}
|
|
|
|
|