2006-12-19 22:34:12 +00:00
|
|
|
#include "builtin.h"
|
2017-06-14 18:07:36 +00:00
|
|
|
#include "config.h"
|
2023-03-21 06:26:03 +00:00
|
|
|
#include "environment.h"
|
2018-04-10 21:26:18 +00:00
|
|
|
#include "exec-cmd.h"
|
2023-03-21 06:25:54 +00:00
|
|
|
#include "gettext.h"
|
2010-09-01 02:29:08 +00:00
|
|
|
#include "help.h"
|
2024-02-08 23:17:31 +00:00
|
|
|
#include "object-file.h"
|
2023-04-11 07:41:59 +00:00
|
|
|
#include "pager.h"
|
2023-05-16 06:33:56 +00:00
|
|
|
#include "read-cache-ll.h"
|
2009-01-28 07:38:14 +00:00
|
|
|
#include "run-command.h"
|
2018-05-20 18:40:06 +00:00
|
|
|
#include "alias.h"
|
2023-02-24 00:09:33 +00:00
|
|
|
#include "replace-object.h"
|
2023-03-21 06:26:05 +00:00
|
|
|
#include "setup.h"
|
2023-05-06 04:15:29 +00:00
|
|
|
#include "attr.h"
|
2020-04-30 19:48:50 +00:00
|
|
|
#include "shallow.h"
|
2023-04-11 03:00:38 +00:00
|
|
|
#include "trace.h"
|
|
|
|
#include "trace2.h"
|
2005-11-15 23:31:25 +00:00
|
|
|
|
2018-03-24 20:35:18 +00:00
|
|
|
#define RUN_SETUP (1<<0)
|
|
|
|
#define RUN_SETUP_GENTLY (1<<1)
|
|
|
|
#define USE_PAGER (1<<2)
|
|
|
|
/*
|
|
|
|
* require working tree to be present -- anything uses this needs
|
|
|
|
* RUN_SETUP for reading from the configuration file.
|
|
|
|
*/
|
|
|
|
#define NEED_WORK_TREE (1<<3)
|
read-tree: add "--super-prefix" option, eliminate global
The "--super-prefix" option to "git" was initially added in [1] for
use with "ls-files"[2], and shortly thereafter "submodule--helper"[3]
and "grep"[4]. It wasn't until [5] that "read-tree" made use of it.
At the time [5] made sense, but since then we've made "ls-files"
recurse in-process in [6], "grep" in [7], and finally
"submodule--helper" in the preceding commits.
Let's also remove it from "read-tree", which allows us to remove the
option to "git" itself.
We can do this because the only remaining user of it is the submodule
API, which will now invoke "read-tree" with its new "--super-prefix"
option. It will only do so when the "submodule_move_head()" function
is called.
That "submodule_move_head()" function was then only invoked by
"read-tree" itself, but now rather than setting an environment
variable to pass "--super-prefix" between cmd_read_tree() we:
- Set a new "super_prefix" in "struct unpack_trees_options". The
"super_prefixed()" function in "unpack-trees.c" added in [5] will now
use this, rather than get_super_prefix() looking up the environment
variable we set earlier in the same process.
- Add the same field to the "struct checkout", which is only needed to
ferry the "super_prefix" in the "struct unpack_trees_options" all the
way down to the "entry.c" callers of "submodule_move_head()".
Those calls which used the super prefix all originated in
"cmd_read_tree()". The only other caller is the "unlink_entry()"
caller in "builtin/checkout.c", which now passes a "NULL".
1. 74866d75793 (git: make super-prefix option, 2016-10-07)
2. e77aa336f11 (ls-files: optionally recurse into submodules, 2016-10-07)
3. 89c86265576 (submodule helper: support super prefix, 2016-12-08)
4. 0281e487fd9 (grep: optionally recurse into submodules, 2016-12-16)
5. 3d415425c7b (unpack-trees: support super-prefix option, 2017-01-17)
6. 188dce131fa (ls-files: use repository object, 2017-06-22)
7. f9ee2fcdfa0 (grep: recurse in-process using 'struct repository', 2017-08-02)
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2022-12-20 12:39:56 +00:00
|
|
|
#define DELAY_PAGER_CONFIG (1<<4)
|
|
|
|
#define NO_PARSEOPT (1<<5) /* parse-options is not used */
|
2018-03-24 20:35:18 +00:00
|
|
|
|
|
|
|
struct cmd_struct {
|
|
|
|
const char *cmd;
|
|
|
|
int (*fn)(int, const char **, const char *);
|
2018-03-24 20:35:19 +00:00
|
|
|
unsigned int option;
|
2018-03-24 20:35:18 +00:00
|
|
|
};
|
|
|
|
|
2006-07-30 21:42:25 +00:00
|
|
|
const char git_usage_string[] =
|
2022-03-31 21:27:09 +00:00
|
|
|
N_("git [-v | --version] [-h | --help] [-C <path>] [-c <name>=<value>]\n"
|
2018-02-13 13:19:15 +00:00
|
|
|
" [--exec-path[=<path>]] [--html-path] [--man-path] [--info-path]\n"
|
2024-05-03 07:17:04 +00:00
|
|
|
" [-p | --paginate | -P | --no-pager] [--no-replace-objects] [--no-lazy-fetch]\n"
|
2024-05-03 07:17:06 +00:00
|
|
|
" [--no-optional-locks] [--no-advice] [--bare] [--git-dir=<path>]\n"
|
|
|
|
" [--work-tree=<path>] [--namespace=<name>] [--config-env=<name>=<envvar>]\n"
|
|
|
|
" <command> [<args>]");
|
2006-07-30 21:42:25 +00:00
|
|
|
|
2008-06-05 21:15:36 +00:00
|
|
|
const char git_more_info_string[] =
|
2014-08-30 19:56:01 +00:00
|
|
|
N_("'git help -a' and 'git help -g' list available subcommands and some\n"
|
2013-04-02 22:39:48 +00:00
|
|
|
"concept guides. See 'git help <command>' or 'git help <concept>'\n"
|
2019-05-15 22:47:08 +00:00
|
|
|
"to read about a specific subcommand or concept.\n"
|
|
|
|
"See 'git help git' for an overview of the system.");
|
2008-06-05 21:15:36 +00:00
|
|
|
|
Allow per-command pager config
There is great debate over whether some commands should set
up a pager automatically. This patch allows individuals to
set their own pager preferences for each command, overriding
the default. For example, to disable the pager for git
status:
git config pager.status false
If "--pager" or "--no-pager" is specified on the command
line, it takes precedence over the config option.
There are two caveats:
- you can turn on the pager for plumbing commands.
Combined with "core.pager = always", this will probably
break a lot of things. Don't do it.
- This only works for builtin commands. The reason is
somewhat complex:
Calling git_config before we do setup_git_directory
has bad side effects, because it wants to know where
the git_dir is to find ".git/config". Unfortunately,
we cannot call setup_git_directory indiscriminately,
because some builtins (like "init") break if we do.
For builtins, this is OK, since we can just wait until
after we call setup_git_directory. But for aliases, we
don't know until we expand (recursively) which command
we're doing. This should not be a huge problem for
aliases, which can simply use "--pager" or "--no-pager"
in the alias as appropriate.
For external commands, however, we don't know we even
have an external command until we exec it, and by then
it is too late to check the config.
An alternative approach would be to have a config mode
where we don't bother looking at .git/config, but only
at the user and system config files. This would make the
behavior consistent across builtins, aliases, and
external commands, at the cost of not allowing per-repo
pager config for at all.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2008-07-03 11:46:57 +00:00
|
|
|
static int use_pager = -1;
|
2014-06-08 09:37:10 +00:00
|
|
|
|
2018-05-20 18:39:58 +00:00
|
|
|
static void list_builtins(struct string_list *list, unsigned int exclude_option);
|
2017-05-30 05:18:43 +00:00
|
|
|
|
2018-05-20 18:40:07 +00:00
|
|
|
static void exclude_helpers_from_list(struct string_list *list)
|
|
|
|
{
|
|
|
|
int i = 0;
|
|
|
|
|
|
|
|
while (i < list->nr) {
|
|
|
|
if (strstr(list->items[i].string, "--"))
|
|
|
|
unsorted_string_list_delete_item(list, i, 0);
|
|
|
|
else
|
|
|
|
i++;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2018-05-20 18:39:57 +00:00
|
|
|
static int match_token(const char *spec, int len, const char *token)
|
|
|
|
{
|
|
|
|
int token_len = strlen(token);
|
|
|
|
|
|
|
|
return len == token_len && !strncmp(spec, token, token_len);
|
|
|
|
}
|
|
|
|
|
|
|
|
static int list_cmds(const char *spec)
|
|
|
|
{
|
2018-05-20 18:39:58 +00:00
|
|
|
struct string_list list = STRING_LIST_INIT_DUP;
|
|
|
|
int i;
|
2019-03-20 18:03:26 +00:00
|
|
|
int nongit;
|
|
|
|
|
|
|
|
/*
|
|
|
|
* Set up the repository so we can pick up any repo-level config (like
|
|
|
|
* completion.commands).
|
|
|
|
*/
|
|
|
|
setup_git_directory_gently(&nongit);
|
2018-05-20 18:39:58 +00:00
|
|
|
|
2018-05-20 18:39:57 +00:00
|
|
|
while (*spec) {
|
|
|
|
const char *sep = strchrnul(spec, ',');
|
|
|
|
int len = sep - spec;
|
|
|
|
|
|
|
|
if (match_token(spec, len, "builtins"))
|
2018-05-20 18:39:58 +00:00
|
|
|
list_builtins(&list, 0);
|
2018-05-20 18:39:59 +00:00
|
|
|
else if (match_token(spec, len, "main"))
|
|
|
|
list_all_main_cmds(&list);
|
|
|
|
else if (match_token(spec, len, "others"))
|
|
|
|
list_all_other_cmds(&list);
|
2018-05-20 18:40:07 +00:00
|
|
|
else if (match_token(spec, len, "nohelpers"))
|
|
|
|
exclude_helpers_from_list(&list);
|
2018-05-20 18:40:08 +00:00
|
|
|
else if (match_token(spec, len, "alias"))
|
|
|
|
list_aliases(&list);
|
2018-05-20 18:40:09 +00:00
|
|
|
else if (match_token(spec, len, "config"))
|
|
|
|
list_cmds_by_config(&list);
|
2018-05-20 18:40:00 +00:00
|
|
|
else if (len > 5 && !strncmp(spec, "list-", 5)) {
|
|
|
|
struct strbuf sb = STRBUF_INIT;
|
|
|
|
|
|
|
|
strbuf_add(&sb, spec + 5, len - 5);
|
|
|
|
list_cmds_by_category(&list, sb.buf);
|
|
|
|
strbuf_release(&sb);
|
|
|
|
}
|
2018-05-20 18:39:57 +00:00
|
|
|
else
|
|
|
|
die(_("unsupported command listing type '%s'"), spec);
|
|
|
|
spec += len;
|
|
|
|
if (*spec == ',')
|
|
|
|
spec++;
|
|
|
|
}
|
2018-05-20 18:39:58 +00:00
|
|
|
for (i = 0; i < list.nr; i++)
|
|
|
|
puts(list.items[i].string);
|
|
|
|
string_list_clear(&list, 0);
|
2018-05-20 18:39:57 +00:00
|
|
|
return 0;
|
|
|
|
}
|
2017-05-30 05:18:43 +00:00
|
|
|
|
2018-12-09 10:25:21 +00:00
|
|
|
static void commit_pager_choice(void)
|
|
|
|
{
|
Allow per-command pager config
There is great debate over whether some commands should set
up a pager automatically. This patch allows individuals to
set their own pager preferences for each command, overriding
the default. For example, to disable the pager for git
status:
git config pager.status false
If "--pager" or "--no-pager" is specified on the command
line, it takes precedence over the config option.
There are two caveats:
- you can turn on the pager for plumbing commands.
Combined with "core.pager = always", this will probably
break a lot of things. Don't do it.
- This only works for builtin commands. The reason is
somewhat complex:
Calling git_config before we do setup_git_directory
has bad side effects, because it wants to know where
the git_dir is to find ".git/config". Unfortunately,
we cannot call setup_git_directory indiscriminately,
because some builtins (like "init") break if we do.
For builtins, this is OK, since we can just wait until
after we call setup_git_directory. But for aliases, we
don't know until we expand (recursively) which command
we're doing. This should not be a huge problem for
aliases, which can simply use "--pager" or "--no-pager"
in the alias as appropriate.
For external commands, however, we don't know we even
have an external command until we exec it, and by then
it is too late to check the config.
An alternative approach would be to have a config mode
where we don't bother looking at .git/config, but only
at the user and system config files. This would make the
behavior consistent across builtins, aliases, and
external commands, at the cost of not allowing per-repo
pager config for at all.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2008-07-03 11:46:57 +00:00
|
|
|
switch (use_pager) {
|
|
|
|
case 0:
|
|
|
|
setenv("GIT_PAGER", "cat", 1);
|
|
|
|
break;
|
|
|
|
case 1:
|
|
|
|
setup_pager();
|
|
|
|
break;
|
|
|
|
default:
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2017-08-02 19:40:51 +00:00
|
|
|
void setup_auto_pager(const char *cmd, int def)
|
|
|
|
{
|
|
|
|
if (use_pager != -1 || pager_in_use())
|
|
|
|
return;
|
|
|
|
use_pager = check_pager_config(cmd);
|
|
|
|
if (use_pager == -1)
|
|
|
|
use_pager = def;
|
|
|
|
commit_pager_choice();
|
|
|
|
}
|
|
|
|
|
2009-05-01 09:06:36 +00:00
|
|
|
static int handle_options(const char ***argv, int *argc, int *envchanged)
|
2006-07-24 12:10:45 +00:00
|
|
|
{
|
2011-05-24 22:50:35 +00:00
|
|
|
const char **orig_argv = *argv;
|
2006-07-24 12:10:45 +00:00
|
|
|
|
|
|
|
while (*argc > 0) {
|
|
|
|
const char *cmd = (*argv)[0];
|
|
|
|
if (cmd[0] != '-')
|
|
|
|
break;
|
|
|
|
|
2006-07-25 18:24:22 +00:00
|
|
|
/*
|
|
|
|
* For legacy reasons, the "version" and "help"
|
|
|
|
* commands can be written with "--" prepended
|
|
|
|
* to make them look like flags.
|
|
|
|
*/
|
2022-03-31 21:27:09 +00:00
|
|
|
if (!strcmp(cmd, "--help") || !strcmp(cmd, "-h") ||
|
|
|
|
!strcmp(cmd, "--version") || !strcmp(cmd, "-v"))
|
2006-07-25 18:24:22 +00:00
|
|
|
break;
|
|
|
|
|
|
|
|
/*
|
|
|
|
* Check remaining flags.
|
|
|
|
*/
|
use skip_prefix to avoid magic numbers
It's a common idiom to match a prefix and then skip past it
with a magic number, like:
if (starts_with(foo, "bar"))
foo += 3;
This is easy to get wrong, since you have to count the
prefix string yourself, and there's no compiler check if the
string changes. We can use skip_prefix to avoid the magic
numbers here.
Note that some of these conversions could be much shorter.
For example:
if (starts_with(arg, "--foo=")) {
bar = arg + 6;
continue;
}
could become:
if (skip_prefix(arg, "--foo=", &bar))
continue;
However, I have left it as:
if (skip_prefix(arg, "--foo=", &v)) {
bar = v;
continue;
}
to visually match nearby cases which need to actually
process the string. Like:
if (skip_prefix(arg, "--foo=", &v)) {
bar = atoi(v);
continue;
}
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2014-06-18 19:47:50 +00:00
|
|
|
if (skip_prefix(cmd, "--exec-path", &cmd)) {
|
2006-07-25 18:24:22 +00:00
|
|
|
if (*cmd == '=')
|
2018-04-10 15:05:44 +00:00
|
|
|
git_set_exec_path(cmd + 1);
|
2006-07-25 18:24:22 +00:00
|
|
|
else {
|
|
|
|
puts(git_exec_path());
|
2019-02-22 22:25:01 +00:00
|
|
|
trace2_cmd_name("_query_");
|
2006-07-25 18:24:22 +00:00
|
|
|
exit(0);
|
|
|
|
}
|
2009-04-05 02:15:16 +00:00
|
|
|
} else if (!strcmp(cmd, "--html-path")) {
|
|
|
|
puts(system_path(GIT_HTML_PATH));
|
2019-02-22 22:25:01 +00:00
|
|
|
trace2_cmd_name("_query_");
|
2009-04-05 02:15:16 +00:00
|
|
|
exit(0);
|
2011-05-01 08:16:25 +00:00
|
|
|
} else if (!strcmp(cmd, "--man-path")) {
|
|
|
|
puts(system_path(GIT_MAN_PATH));
|
2019-02-22 22:25:01 +00:00
|
|
|
trace2_cmd_name("_query_");
|
2011-05-01 08:16:25 +00:00
|
|
|
exit(0);
|
|
|
|
} else if (!strcmp(cmd, "--info-path")) {
|
|
|
|
puts(system_path(GIT_INFO_PATH));
|
2019-02-22 22:25:01 +00:00
|
|
|
trace2_cmd_name("_query_");
|
2011-05-01 08:16:25 +00:00
|
|
|
exit(0);
|
2006-07-25 18:24:22 +00:00
|
|
|
} else if (!strcmp(cmd, "-p") || !strcmp(cmd, "--paginate")) {
|
Allow per-command pager config
There is great debate over whether some commands should set
up a pager automatically. This patch allows individuals to
set their own pager preferences for each command, overriding
the default. For example, to disable the pager for git
status:
git config pager.status false
If "--pager" or "--no-pager" is specified on the command
line, it takes precedence over the config option.
There are two caveats:
- you can turn on the pager for plumbing commands.
Combined with "core.pager = always", this will probably
break a lot of things. Don't do it.
- This only works for builtin commands. The reason is
somewhat complex:
Calling git_config before we do setup_git_directory
has bad side effects, because it wants to know where
the git_dir is to find ".git/config". Unfortunately,
we cannot call setup_git_directory indiscriminately,
because some builtins (like "init") break if we do.
For builtins, this is OK, since we can just wait until
after we call setup_git_directory. But for aliases, we
don't know until we expand (recursively) which command
we're doing. This should not be a huge problem for
aliases, which can simply use "--pager" or "--no-pager"
in the alias as appropriate.
For external commands, however, we don't know we even
have an external command until we exec it, and by then
it is too late to check the config.
An alternative approach would be to have a config mode
where we don't bother looking at .git/config, but only
at the user and system config files. This would make the
behavior consistent across builtins, aliases, and
external commands, at the cost of not allowing per-repo
pager config for at all.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2008-07-03 11:46:57 +00:00
|
|
|
use_pager = 1;
|
git: add -P as a short option for --no-pager
It is possible to configure 'less', the pager, to use an alternate
screen to show the content, for example, by setting LESS=RS in the
environment. When it is closed in this configuration, it switches
back to the original screen, and all content is gone.
It is not uncommon to request that the output remains visible in
the terminal. For this, the option --no-pager can be used. But
it is a bit cumbersome to type, even when command completion is
available. Provide a short option, -P, to make the option more
easily accessible.
Signed-off-by: Johannes Sixt <j6t@kdbg.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2018-05-03 17:15:08 +00:00
|
|
|
} else if (!strcmp(cmd, "-P") || !strcmp(cmd, "--no-pager")) {
|
Allow per-command pager config
There is great debate over whether some commands should set
up a pager automatically. This patch allows individuals to
set their own pager preferences for each command, overriding
the default. For example, to disable the pager for git
status:
git config pager.status false
If "--pager" or "--no-pager" is specified on the command
line, it takes precedence over the config option.
There are two caveats:
- you can turn on the pager for plumbing commands.
Combined with "core.pager = always", this will probably
break a lot of things. Don't do it.
- This only works for builtin commands. The reason is
somewhat complex:
Calling git_config before we do setup_git_directory
has bad side effects, because it wants to know where
the git_dir is to find ".git/config". Unfortunately,
we cannot call setup_git_directory indiscriminately,
because some builtins (like "init") break if we do.
For builtins, this is OK, since we can just wait until
after we call setup_git_directory. But for aliases, we
don't know until we expand (recursively) which command
we're doing. This should not be a huge problem for
aliases, which can simply use "--pager" or "--no-pager"
in the alias as appropriate.
For external commands, however, we don't know we even
have an external command until we exec it, and by then
it is too late to check the config.
An alternative approach would be to have a config mode
where we don't bother looking at .git/config, but only
at the user and system config files. This would make the
behavior consistent across builtins, aliases, and
external commands, at the cost of not allowing per-repo
pager config for at all.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2008-07-03 11:46:57 +00:00
|
|
|
use_pager = 0;
|
2007-08-19 17:24:36 +00:00
|
|
|
if (envchanged)
|
|
|
|
*envchanged = 1;
|
2024-02-08 23:17:31 +00:00
|
|
|
} else if (!strcmp(cmd, "--no-lazy-fetch")) {
|
|
|
|
fetch_if_missing = 0;
|
2024-02-27 16:48:29 +00:00
|
|
|
setenv(NO_LAZY_FETCH_ENVIRONMENT, "1", 1);
|
|
|
|
if (envchanged)
|
|
|
|
*envchanged = 1;
|
2009-10-12 20:30:32 +00:00
|
|
|
} else if (!strcmp(cmd, "--no-replace-objects")) {
|
2023-06-06 13:24:35 +00:00
|
|
|
disable_replace_refs();
|
2009-11-18 06:50:58 +00:00
|
|
|
setenv(NO_REPLACE_OBJECTS_ENVIRONMENT, "1", 1);
|
|
|
|
if (envchanged)
|
|
|
|
*envchanged = 1;
|
2006-07-25 18:24:22 +00:00
|
|
|
} else if (!strcmp(cmd, "--git-dir")) {
|
2006-12-22 13:56:25 +00:00
|
|
|
if (*argc < 2) {
|
2022-01-05 20:02:20 +00:00
|
|
|
fprintf(stderr, _("no directory given for '%s' option\n" ), "--git-dir");
|
2006-12-22 13:56:25 +00:00
|
|
|
usage(git_usage_string);
|
|
|
|
}
|
2006-12-31 04:29:11 +00:00
|
|
|
setenv(GIT_DIR_ENVIRONMENT, (*argv)[1], 1);
|
2007-06-08 20:57:55 +00:00
|
|
|
if (envchanged)
|
|
|
|
*envchanged = 1;
|
2006-07-25 18:24:22 +00:00
|
|
|
(*argv)++;
|
|
|
|
(*argc)--;
|
use skip_prefix to avoid magic numbers
It's a common idiom to match a prefix and then skip past it
with a magic number, like:
if (starts_with(foo, "bar"))
foo += 3;
This is easy to get wrong, since you have to count the
prefix string yourself, and there's no compiler check if the
string changes. We can use skip_prefix to avoid the magic
numbers here.
Note that some of these conversions could be much shorter.
For example:
if (starts_with(arg, "--foo=")) {
bar = arg + 6;
continue;
}
could become:
if (skip_prefix(arg, "--foo=", &bar))
continue;
However, I have left it as:
if (skip_prefix(arg, "--foo=", &v)) {
bar = v;
continue;
}
to visually match nearby cases which need to actually
process the string. Like:
if (skip_prefix(arg, "--foo=", &v)) {
bar = atoi(v);
continue;
}
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2014-06-18 19:47:50 +00:00
|
|
|
} else if (skip_prefix(cmd, "--git-dir=", &cmd)) {
|
|
|
|
setenv(GIT_DIR_ENVIRONMENT, cmd, 1);
|
2007-06-08 20:57:55 +00:00
|
|
|
if (envchanged)
|
|
|
|
*envchanged = 1;
|
ref namespaces: infrastructure
Add support for dividing the refs of a single repository into multiple
namespaces, each of which can have its own branches, tags, and HEAD.
Git can expose each namespace as an independent repository to pull from
and push to, while sharing the object store, and exposing all the refs
to operations such as git-gc.
Storing multiple repositories as namespaces of a single repository
avoids storing duplicate copies of the same objects, such as when
storing multiple branches of the same source. The alternates mechanism
provides similar support for avoiding duplicates, but alternates do not
prevent duplication between new objects added to the repositories
without ongoing maintenance, while namespaces do.
To specify a namespace, set the GIT_NAMESPACE environment variable to
the namespace. For each ref namespace, git stores the corresponding
refs in a directory under refs/namespaces/. For example,
GIT_NAMESPACE=foo will store refs under refs/namespaces/foo/. You can
also specify namespaces via the --namespace option to git.
Note that namespaces which include a / will expand to a hierarchy of
namespaces; for example, GIT_NAMESPACE=foo/bar will store refs under
refs/namespaces/foo/refs/namespaces/bar/. This makes paths in
GIT_NAMESPACE behave hierarchically, so that cloning with
GIT_NAMESPACE=foo/bar produces the same result as cloning with
GIT_NAMESPACE=foo and cloning from that repo with GIT_NAMESPACE=bar. It
also avoids ambiguity with strange namespace paths such as
foo/refs/heads/, which could otherwise generate directory/file conflicts
within the refs directory.
Add the infrastructure for ref namespaces: handle the GIT_NAMESPACE
environment variable and --namespace option, and support iterating over
refs in a namespace.
Signed-off-by: Josh Triplett <josh@joshtriplett.org>
Signed-off-by: Jamey Sharp <jamey@minilop.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2011-07-05 17:54:44 +00:00
|
|
|
} else if (!strcmp(cmd, "--namespace")) {
|
|
|
|
if (*argc < 2) {
|
2018-02-13 13:19:15 +00:00
|
|
|
fprintf(stderr, _("no namespace given for --namespace\n" ));
|
ref namespaces: infrastructure
Add support for dividing the refs of a single repository into multiple
namespaces, each of which can have its own branches, tags, and HEAD.
Git can expose each namespace as an independent repository to pull from
and push to, while sharing the object store, and exposing all the refs
to operations such as git-gc.
Storing multiple repositories as namespaces of a single repository
avoids storing duplicate copies of the same objects, such as when
storing multiple branches of the same source. The alternates mechanism
provides similar support for avoiding duplicates, but alternates do not
prevent duplication between new objects added to the repositories
without ongoing maintenance, while namespaces do.
To specify a namespace, set the GIT_NAMESPACE environment variable to
the namespace. For each ref namespace, git stores the corresponding
refs in a directory under refs/namespaces/. For example,
GIT_NAMESPACE=foo will store refs under refs/namespaces/foo/. You can
also specify namespaces via the --namespace option to git.
Note that namespaces which include a / will expand to a hierarchy of
namespaces; for example, GIT_NAMESPACE=foo/bar will store refs under
refs/namespaces/foo/refs/namespaces/bar/. This makes paths in
GIT_NAMESPACE behave hierarchically, so that cloning with
GIT_NAMESPACE=foo/bar produces the same result as cloning with
GIT_NAMESPACE=foo and cloning from that repo with GIT_NAMESPACE=bar. It
also avoids ambiguity with strange namespace paths such as
foo/refs/heads/, which could otherwise generate directory/file conflicts
within the refs directory.
Add the infrastructure for ref namespaces: handle the GIT_NAMESPACE
environment variable and --namespace option, and support iterating over
refs in a namespace.
Signed-off-by: Josh Triplett <josh@joshtriplett.org>
Signed-off-by: Jamey Sharp <jamey@minilop.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2011-07-05 17:54:44 +00:00
|
|
|
usage(git_usage_string);
|
|
|
|
}
|
|
|
|
setenv(GIT_NAMESPACE_ENVIRONMENT, (*argv)[1], 1);
|
|
|
|
if (envchanged)
|
|
|
|
*envchanged = 1;
|
|
|
|
(*argv)++;
|
|
|
|
(*argc)--;
|
use skip_prefix to avoid magic numbers
It's a common idiom to match a prefix and then skip past it
with a magic number, like:
if (starts_with(foo, "bar"))
foo += 3;
This is easy to get wrong, since you have to count the
prefix string yourself, and there's no compiler check if the
string changes. We can use skip_prefix to avoid the magic
numbers here.
Note that some of these conversions could be much shorter.
For example:
if (starts_with(arg, "--foo=")) {
bar = arg + 6;
continue;
}
could become:
if (skip_prefix(arg, "--foo=", &bar))
continue;
However, I have left it as:
if (skip_prefix(arg, "--foo=", &v)) {
bar = v;
continue;
}
to visually match nearby cases which need to actually
process the string. Like:
if (skip_prefix(arg, "--foo=", &v)) {
bar = atoi(v);
continue;
}
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2014-06-18 19:47:50 +00:00
|
|
|
} else if (skip_prefix(cmd, "--namespace=", &cmd)) {
|
|
|
|
setenv(GIT_NAMESPACE_ENVIRONMENT, cmd, 1);
|
ref namespaces: infrastructure
Add support for dividing the refs of a single repository into multiple
namespaces, each of which can have its own branches, tags, and HEAD.
Git can expose each namespace as an independent repository to pull from
and push to, while sharing the object store, and exposing all the refs
to operations such as git-gc.
Storing multiple repositories as namespaces of a single repository
avoids storing duplicate copies of the same objects, such as when
storing multiple branches of the same source. The alternates mechanism
provides similar support for avoiding duplicates, but alternates do not
prevent duplication between new objects added to the repositories
without ongoing maintenance, while namespaces do.
To specify a namespace, set the GIT_NAMESPACE environment variable to
the namespace. For each ref namespace, git stores the corresponding
refs in a directory under refs/namespaces/. For example,
GIT_NAMESPACE=foo will store refs under refs/namespaces/foo/. You can
also specify namespaces via the --namespace option to git.
Note that namespaces which include a / will expand to a hierarchy of
namespaces; for example, GIT_NAMESPACE=foo/bar will store refs under
refs/namespaces/foo/refs/namespaces/bar/. This makes paths in
GIT_NAMESPACE behave hierarchically, so that cloning with
GIT_NAMESPACE=foo/bar produces the same result as cloning with
GIT_NAMESPACE=foo and cloning from that repo with GIT_NAMESPACE=bar. It
also avoids ambiguity with strange namespace paths such as
foo/refs/heads/, which could otherwise generate directory/file conflicts
within the refs directory.
Add the infrastructure for ref namespaces: handle the GIT_NAMESPACE
environment variable and --namespace option, and support iterating over
refs in a namespace.
Signed-off-by: Josh Triplett <josh@joshtriplett.org>
Signed-off-by: Jamey Sharp <jamey@minilop.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2011-07-05 17:54:44 +00:00
|
|
|
if (envchanged)
|
|
|
|
*envchanged = 1;
|
2007-06-06 07:10:42 +00:00
|
|
|
} else if (!strcmp(cmd, "--work-tree")) {
|
|
|
|
if (*argc < 2) {
|
2022-01-05 20:02:20 +00:00
|
|
|
fprintf(stderr, _("no directory given for '%s' option\n" ), "--work-tree");
|
2007-06-06 07:10:42 +00:00
|
|
|
usage(git_usage_string);
|
|
|
|
}
|
|
|
|
setenv(GIT_WORK_TREE_ENVIRONMENT, (*argv)[1], 1);
|
2007-06-08 20:57:55 +00:00
|
|
|
if (envchanged)
|
|
|
|
*envchanged = 1;
|
2007-06-06 07:10:42 +00:00
|
|
|
(*argv)++;
|
|
|
|
(*argc)--;
|
use skip_prefix to avoid magic numbers
It's a common idiom to match a prefix and then skip past it
with a magic number, like:
if (starts_with(foo, "bar"))
foo += 3;
This is easy to get wrong, since you have to count the
prefix string yourself, and there's no compiler check if the
string changes. We can use skip_prefix to avoid the magic
numbers here.
Note that some of these conversions could be much shorter.
For example:
if (starts_with(arg, "--foo=")) {
bar = arg + 6;
continue;
}
could become:
if (skip_prefix(arg, "--foo=", &bar))
continue;
However, I have left it as:
if (skip_prefix(arg, "--foo=", &v)) {
bar = v;
continue;
}
to visually match nearby cases which need to actually
process the string. Like:
if (skip_prefix(arg, "--foo=", &v)) {
bar = atoi(v);
continue;
}
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2014-06-18 19:47:50 +00:00
|
|
|
} else if (skip_prefix(cmd, "--work-tree=", &cmd)) {
|
|
|
|
setenv(GIT_WORK_TREE_ENVIRONMENT, cmd, 1);
|
2007-06-08 20:57:55 +00:00
|
|
|
if (envchanged)
|
|
|
|
*envchanged = 1;
|
2006-07-25 18:24:22 +00:00
|
|
|
} else if (!strcmp(cmd, "--bare")) {
|
2014-07-28 18:31:57 +00:00
|
|
|
char *cwd = xgetcwd();
|
2007-08-27 07:58:06 +00:00
|
|
|
is_bare_repository_cfg = 1;
|
2014-07-28 18:31:57 +00:00
|
|
|
setenv(GIT_DIR_ENVIRONMENT, cwd, 0);
|
|
|
|
free(cwd);
|
setup: suppress implicit "." work-tree for bare repos
If an explicit GIT_DIR is given without a working tree, we
implicitly assume that the current working directory should
be used as the working tree. E.g.,:
GIT_DIR=/some/repo.git git status
would compare against the cwd.
Unfortunately, we fool this rule for sub-invocations of git
by setting GIT_DIR internally ourselves. For example:
git init foo
cd foo/.git
git status ;# fails, as we expect
git config alias.st status
git status ;# does not fail, but should
What happens is that we run setup_git_directory when doing
alias lookup (since we need to see the config), set GIT_DIR
as a result, and then leave GIT_WORK_TREE blank (because we
do not have one). Then when we actually run the status
command, we do setup_git_directory again, which sees our
explicit GIT_DIR and uses the cwd as an implicit worktree.
It's tempting to argue that we should be suppressing that
second invocation of setup_git_directory, as it could use
the values we already found in memory. However, the problem
still exists for sub-processes (e.g., if "git status" were
an external command).
You can see another example with the "--bare" option, which
sets GIT_DIR explicitly. For example:
git init foo
cd foo/.git
git status ;# fails
git --bare status ;# does NOT fail
We need some way of telling sub-processes "even though
GIT_DIR is set, do not use cwd as an implicit working tree".
We could do it by putting a special token into
GIT_WORK_TREE, but the obvious choice (an empty string) has
some portability problems.
Instead, we add a new boolean variable, GIT_IMPLICIT_WORK_TREE,
which suppresses the use of cwd as a working tree when
GIT_DIR is set. We trigger the new variable when we know we
are in a bare setting.
The variable is left intentionally undocumented, as this is
an internal detail (for now, anyway). If somebody comes up
with a good alternate use for it, and once we are confident
we have shaken any bugs out of it, we can consider promoting
it further.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2013-03-08 09:32:22 +00:00
|
|
|
setenv(GIT_IMPLICIT_WORK_TREE_ENVIRONMENT, "0", 1);
|
2007-06-08 20:57:55 +00:00
|
|
|
if (envchanged)
|
|
|
|
*envchanged = 1;
|
2010-03-26 22:53:57 +00:00
|
|
|
} else if (!strcmp(cmd, "-c")) {
|
|
|
|
if (*argc < 2) {
|
2018-02-13 13:19:15 +00:00
|
|
|
fprintf(stderr, _("-c expects a configuration string\n" ));
|
2010-03-26 22:53:57 +00:00
|
|
|
usage(git_usage_string);
|
|
|
|
}
|
2010-08-23 19:16:00 +00:00
|
|
|
git_config_push_parameter((*argv)[1]);
|
2010-03-26 22:53:57 +00:00
|
|
|
(*argv)++;
|
|
|
|
(*argc)--;
|
2021-04-29 12:55:34 +00:00
|
|
|
} else if (!strcmp(cmd, "--config-env")) {
|
|
|
|
if (*argc < 2) {
|
|
|
|
fprintf(stderr, _("no config key given for --config-env\n" ));
|
|
|
|
usage(git_usage_string);
|
|
|
|
}
|
|
|
|
git_config_push_env((*argv)[1]);
|
|
|
|
(*argv)++;
|
|
|
|
(*argc)--;
|
2021-01-12 12:26:45 +00:00
|
|
|
} else if (skip_prefix(cmd, "--config-env=", &cmd)) {
|
|
|
|
git_config_push_env(cmd);
|
add global --literal-pathspecs option
Git takes pathspec arguments in many places to limit the
scope of an operation. These pathspecs are treated not as
literal paths, but as glob patterns that can be fed to
fnmatch. When a user is giving a specific pattern, this is a
nice feature.
However, when programatically providing pathspecs, it can be
a nuisance. For example, to find the latest revision which
modified "$foo", one can use "git rev-list -- $foo". But if
"$foo" contains glob characters (e.g., "f*"), it will
erroneously match more entries than desired. The caller
needs to quote the characters in $foo, and even then, the
results may not be exactly the same as with a literal
pathspec. For instance, the depth checks in
match_pathspec_depth do not kick in if we match via fnmatch.
This patch introduces a global command-line option (i.e.,
one for "git" itself, not for specific commands) to turn
this behavior off. It also has a matching environment
variable, which can make it easier if you are a script or
porcelain interface that is going to issue many such
commands.
This option cannot turn off globbing for particular
pathspecs. That could eventually be done with a ":(noglob)"
magic pathspec prefix. However, that level of granularity is
more cumbersome to use for many cases, and doing ":(noglob)"
right would mean converting the whole codebase to use
"struct pathspec", as the usual "const char **pathspec"
cannot represent extra per-item flags.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2012-12-19 22:37:30 +00:00
|
|
|
} else if (!strcmp(cmd, "--literal-pathspecs")) {
|
|
|
|
setenv(GIT_LITERAL_PATHSPECS_ENVIRONMENT, "1", 1);
|
|
|
|
if (envchanged)
|
|
|
|
*envchanged = 1;
|
|
|
|
} else if (!strcmp(cmd, "--no-literal-pathspecs")) {
|
|
|
|
setenv(GIT_LITERAL_PATHSPECS_ENVIRONMENT, "0", 1);
|
|
|
|
if (envchanged)
|
|
|
|
*envchanged = 1;
|
2013-07-14 08:36:08 +00:00
|
|
|
} else if (!strcmp(cmd, "--glob-pathspecs")) {
|
|
|
|
setenv(GIT_GLOB_PATHSPECS_ENVIRONMENT, "1", 1);
|
|
|
|
if (envchanged)
|
|
|
|
*envchanged = 1;
|
|
|
|
} else if (!strcmp(cmd, "--noglob-pathspecs")) {
|
|
|
|
setenv(GIT_NOGLOB_PATHSPECS_ENVIRONMENT, "1", 1);
|
|
|
|
if (envchanged)
|
|
|
|
*envchanged = 1;
|
2013-07-14 08:36:09 +00:00
|
|
|
} else if (!strcmp(cmd, "--icase-pathspecs")) {
|
|
|
|
setenv(GIT_ICASE_PATHSPECS_ENVIRONMENT, "1", 1);
|
|
|
|
if (envchanged)
|
|
|
|
*envchanged = 1;
|
git: add --no-optional-locks option
Some tools like IDEs or fancy editors may periodically run
commands like "git status" in the background to keep track
of the state of the repository. Some of these commands may
refresh the index and write out the result in an
opportunistic way: if they can get the index lock, then they
update the on-disk index with any updates they find. And if
not, then their in-core refresh is lost and just has to be
recomputed by the next caller.
But taking the index lock may conflict with other operations
in the repository. Especially ones that the user is doing
themselves, which _aren't_ opportunistic. In other words,
"git status" knows how to back off when somebody else is
holding the lock, but other commands don't know that status
would be happy to drop the lock if somebody else wanted it.
There are a couple possible solutions:
1. Have some kind of "pseudo-lock" that allows other
commands to tell status that they want the lock.
This is likely to be complicated and error-prone to
implement (and maybe even impossible with just
dotlocks to work from, as it requires some
inter-process communication).
2. Avoid background runs of commands like "git status"
that want to do opportunistic updates, preferring
instead plumbing like diff-files, etc.
This is awkward for a couple of reasons. One is that
"status --porcelain" reports a lot more about the
repository state than is available from individual
plumbing commands. And two is that we actually _do_
want to see the refreshed index. We just don't want to
take a lock or write out the result. Whereas commands
like diff-files expect us to refresh the index
separately and write it to disk so that they can depend
on the result. But that write is exactly what we're
trying to avoid.
3. Ask "status" not to lock or write the index.
This is easy to implement. The big downside is that any
work done in refreshing the index for such a call is
lost when the process exits. So a background process
may end up re-hashing a changed file multiple times
until the user runs a command that does an index
refresh themselves.
This patch implements the option 3. The idea (and the test)
is largely stolen from a Git for Windows patch by Johannes
Schindelin, 67e5ce7f63 (status: offer *not* to lock the
index and update it, 2016-08-12). The twist here is that
instead of making this an option to "git status", it becomes
a "git" option and matching environment variable.
The reason there is two-fold:
1. An environment variable is carried through to
sub-processes. And whether an invocation is a
background process or not should apply to the whole
process tree. So you could do "git --no-optional-locks
foo", and if "foo" is a script or alias that calls
"status", you'll still get the effect.
2. There may be other programs that want the same
treatment.
I've punted here on finding more callers to convert,
since "status" is the obvious one to call as a repeated
background job. But "git diff"'s opportunistic refresh
of the index may be a good candidate.
The test is taken from 67e5ce7f63, and it's worth repeating
Johannes's explanation:
Note that the regression test added in this commit does
not *really* verify that no index.lock file was written;
that test is not possible in a portable way. Instead, we
verify that .git/index is rewritten *only* when `git
status` is run without `--no-optional-locks`.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2017-09-27 06:54:30 +00:00
|
|
|
} else if (!strcmp(cmd, "--no-optional-locks")) {
|
|
|
|
setenv(GIT_OPTIONAL_LOCKS_ENVIRONMENT, "0", 1);
|
|
|
|
if (envchanged)
|
|
|
|
*envchanged = 1;
|
2013-05-26 01:16:15 +00:00
|
|
|
} else if (!strcmp(cmd, "--shallow-file")) {
|
|
|
|
(*argv)++;
|
|
|
|
(*argc)--;
|
2018-05-17 22:51:43 +00:00
|
|
|
set_alternate_shallow_file(the_repository, (*argv)[0], 1);
|
2013-05-26 01:16:15 +00:00
|
|
|
if (envchanged)
|
|
|
|
*envchanged = 1;
|
2013-09-09 13:47:43 +00:00
|
|
|
} else if (!strcmp(cmd, "-C")) {
|
|
|
|
if (*argc < 2) {
|
2022-01-05 20:02:20 +00:00
|
|
|
fprintf(stderr, _("no directory given for '%s' option\n" ), "-C");
|
2013-09-09 13:47:43 +00:00
|
|
|
usage(git_usage_string);
|
|
|
|
}
|
2015-03-06 11:18:08 +00:00
|
|
|
if ((*argv)[1][0]) {
|
|
|
|
if (chdir((*argv)[1]))
|
2018-02-13 13:19:15 +00:00
|
|
|
die_errno("cannot change to '%s'", (*argv)[1]);
|
2015-03-06 11:18:08 +00:00
|
|
|
if (envchanged)
|
|
|
|
*envchanged = 1;
|
|
|
|
}
|
2013-09-09 13:47:43 +00:00
|
|
|
(*argv)++;
|
|
|
|
(*argc)--;
|
2018-05-20 18:39:57 +00:00
|
|
|
} else if (skip_prefix(cmd, "--list-cmds=", &cmd)) {
|
2019-02-22 22:25:01 +00:00
|
|
|
trace2_cmd_name("_query_");
|
2018-05-20 18:39:57 +00:00
|
|
|
if (!strcmp(cmd, "parseopt")) {
|
2018-05-20 18:39:58 +00:00
|
|
|
struct string_list list = STRING_LIST_INIT_DUP;
|
|
|
|
int i;
|
|
|
|
|
|
|
|
list_builtins(&list, NO_PARSEOPT);
|
|
|
|
for (i = 0; i < list.nr; i++)
|
|
|
|
printf("%s ", list.items[i].string);
|
|
|
|
string_list_clear(&list, 0);
|
2018-05-20 18:39:57 +00:00
|
|
|
exit(0);
|
|
|
|
} else {
|
|
|
|
exit(list_cmds(cmd));
|
|
|
|
}
|
2023-05-06 04:15:29 +00:00
|
|
|
} else if (!strcmp(cmd, "--attr-source")) {
|
|
|
|
if (*argc < 2) {
|
|
|
|
fprintf(stderr, _("no attribute source given for --attr-source\n" ));
|
|
|
|
usage(git_usage_string);
|
|
|
|
}
|
|
|
|
setenv(GIT_ATTR_SOURCE_ENVIRONMENT, (*argv)[1], 1);
|
|
|
|
if (envchanged)
|
|
|
|
*envchanged = 1;
|
|
|
|
(*argv)++;
|
|
|
|
(*argc)--;
|
|
|
|
} else if (skip_prefix(cmd, "--attr-source=", &cmd)) {
|
|
|
|
set_git_attr_source(cmd);
|
|
|
|
setenv(GIT_ATTR_SOURCE_ENVIRONMENT, cmd, 1);
|
|
|
|
if (envchanged)
|
|
|
|
*envchanged = 1;
|
2024-05-03 07:17:06 +00:00
|
|
|
} else if (!strcmp(cmd, "--no-advice")) {
|
|
|
|
setenv(GIT_ADVICE_ENVIRONMENT, "0", 1);
|
|
|
|
if (envchanged)
|
|
|
|
*envchanged = 1;
|
2006-07-25 18:24:22 +00:00
|
|
|
} else {
|
2018-02-13 13:19:15 +00:00
|
|
|
fprintf(stderr, _("unknown option: %s\n"), cmd);
|
2006-07-30 21:42:25 +00:00
|
|
|
usage(git_usage_string);
|
2006-07-25 18:24:22 +00:00
|
|
|
}
|
2006-07-24 12:10:45 +00:00
|
|
|
|
|
|
|
(*argv)++;
|
|
|
|
(*argc)--;
|
|
|
|
}
|
2011-05-24 22:50:35 +00:00
|
|
|
return (*argv) - orig_argv;
|
2006-07-24 12:10:45 +00:00
|
|
|
}
|
|
|
|
|
2006-06-05 17:43:52 +00:00
|
|
|
static int handle_alias(int *argcp, const char ***argv)
|
|
|
|
{
|
2008-03-25 21:06:26 +00:00
|
|
|
int envchanged = 0, ret = 0, saved_errno = errno;
|
2006-07-30 01:30:29 +00:00
|
|
|
int count, option_count;
|
2009-05-01 09:06:36 +00:00
|
|
|
const char **new_argv;
|
2008-02-24 22:17:14 +00:00
|
|
|
const char *alias_command;
|
|
|
|
char *alias_string;
|
2006-07-30 01:30:29 +00:00
|
|
|
|
|
|
|
alias_command = (*argv)[0];
|
2008-02-24 22:17:14 +00:00
|
|
|
alias_string = alias_lookup(alias_command);
|
2006-07-30 01:30:29 +00:00
|
|
|
if (alias_string) {
|
git.c: handle_alias: prepend alias info when first argument is -h
Most git commands respond to -h anywhere in the command line, or at
least as a first and lone argument, by printing the usage
information. For aliases, we can provide a little more information that
might be useful in interpreting/understanding the following output by
prepending a line telling that the command is an alias, and for what.
When one invokes a simple alias, such as "cp = cherry-pick"
with -h, this results in
$ git cp -h
'cp' is aliased to 'cherry-pick'
usage: git cherry-pick [<options>] <commit-ish>...
...
When the alias consists of more than one word, this provides the
additional benefit of informing the user which options are implicit in
using the alias, e.g. with "cp = cherry-pick -n":
$ git cp -h
'cp' is aliased to 'cherry-pick -n'
usage: git cherry-pick [<options>] <commit-ish>...
...
For shell commands, we cannot know how it responds to -h, but printing
this line to stderr should not hurt, and can help in figuring out what
is happening in a case like
$ git sc -h
'sc' is aliased to '!somecommand'
somecommand: invalid option '-h'
Suggested-by: Jeff King <peff@peff.net>
Signed-off-by: Rasmus Villemoes <rv@rasmusvillemoes.dk>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2018-10-09 11:59:08 +00:00
|
|
|
if (*argcp > 1 && !strcmp((*argv)[1], "-h"))
|
|
|
|
fprintf_ln(stderr, _("'%s' is aliased to '%s'"),
|
|
|
|
alias_command, alias_string);
|
2007-02-11 00:33:58 +00:00
|
|
|
if (alias_string[0] == '!') {
|
2016-02-22 22:44:21 +00:00
|
|
|
struct child_process child = CHILD_PROCESS_INIT;
|
2017-06-14 11:36:00 +00:00
|
|
|
int nongit_ok;
|
|
|
|
|
|
|
|
/* Aliases expect GIT_PREFIX, GIT_DIR etc to be set */
|
|
|
|
setup_git_directory_gently(&nongit_ok);
|
2011-01-06 23:00:38 +00:00
|
|
|
|
2010-07-14 22:55:12 +00:00
|
|
|
commit_pager_choice();
|
2011-01-06 23:00:38 +00:00
|
|
|
|
2016-02-22 22:44:21 +00:00
|
|
|
child.use_shell = 1;
|
Wait for child on signal death for aliases to externals
When we are running an alias to an external command, we want to wait for
that process to exit even after receiving ^C which normally kills the
git process. This is useful when the process is ignoring SIGINT (which
e.g. pagers often do), and then we don't want it to be killed.
Having an alias which invokes a pager is probably not common, but it can
be useful e.g. if you have an alias to a git command which uses a
subshell as one of the arguments (in which case you have to use an
external command, not an alias to a builtin).
This patch is similar to the previous commit, but the previous commit
fixed this only for aliases to builtins, while this commit does the same
for aliases to external commands. In addition to waiting after clean
like the previous commit, this also enables cleaning the child (that was
already enabled for aliases to builtins before the previous commit),
because wait_after_clean relies on it. Lastly, while the previous commit
fixed a regression, I don't think this has ever worked properly.
Signed-off-by: Trygve Aaberge <trygveaa@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2020-07-07 12:17:15 +00:00
|
|
|
child.clean_on_exit = 1;
|
|
|
|
child.wait_after_clean = 1;
|
2019-02-22 22:25:01 +00:00
|
|
|
child.trace2_child_class = "shell_alias";
|
2020-07-28 20:24:53 +00:00
|
|
|
strvec_push(&child.args, alias_string + 1);
|
|
|
|
strvec_pushv(&child.args, (*argv) + 1);
|
2011-01-06 23:00:38 +00:00
|
|
|
|
2020-07-29 00:37:20 +00:00
|
|
|
trace2_cmd_alias(alias_command, child.args.v);
|
2019-02-22 22:25:01 +00:00
|
|
|
trace2_cmd_name("_run_shell_alias_");
|
|
|
|
|
2016-02-22 22:44:21 +00:00
|
|
|
ret = run_command(&child);
|
2011-01-06 23:00:38 +00:00
|
|
|
if (ret >= 0) /* normal exit */
|
|
|
|
exit(ret);
|
|
|
|
|
2018-11-10 05:16:00 +00:00
|
|
|
die_errno(_("while expanding alias '%s': '%s'"),
|
|
|
|
alias_command, alias_string + 1);
|
2007-02-11 00:33:58 +00:00
|
|
|
}
|
2006-07-30 01:30:29 +00:00
|
|
|
count = split_cmdline(alias_string, &new_argv);
|
2008-09-22 15:06:41 +00:00
|
|
|
if (count < 0)
|
2018-11-10 05:16:00 +00:00
|
|
|
die(_("bad alias.%s string: %s"), alias_command,
|
2018-11-10 05:16:01 +00:00
|
|
|
_(split_cmdline_strerror(count)));
|
2007-06-08 20:57:55 +00:00
|
|
|
option_count = handle_options(&new_argv, &count, &envchanged);
|
|
|
|
if (envchanged)
|
2018-11-10 05:16:00 +00:00
|
|
|
die(_("alias '%s' changes environment variables.\n"
|
|
|
|
"You can use '!git' in the alias to do this"),
|
|
|
|
alias_command);
|
2019-09-19 20:48:30 +00:00
|
|
|
MOVE_ARRAY(new_argv - option_count, new_argv, count);
|
2006-07-30 01:30:29 +00:00
|
|
|
new_argv -= option_count;
|
|
|
|
|
|
|
|
if (count < 1)
|
2018-11-10 05:16:00 +00:00
|
|
|
die(_("empty alias for %s"), alias_command);
|
2006-07-30 01:30:29 +00:00
|
|
|
|
|
|
|
if (!strcmp(alias_command, new_argv[0]))
|
2018-11-10 05:16:00 +00:00
|
|
|
die(_("recursive alias: %s"), alias_command);
|
2006-07-30 01:30:29 +00:00
|
|
|
|
2007-12-03 04:51:50 +00:00
|
|
|
trace_argv_printf(new_argv,
|
2006-09-02 16:23:48 +00:00
|
|
|
"trace: alias expansion: %s =>",
|
|
|
|
alias_command);
|
2006-06-25 13:56:18 +00:00
|
|
|
|
2014-09-16 18:56:57 +00:00
|
|
|
REALLOC_ARRAY(new_argv, count + *argcp);
|
2006-07-30 01:30:29 +00:00
|
|
|
/* insert after command name */
|
2019-09-19 20:48:30 +00:00
|
|
|
COPY_ARRAY(new_argv + count, *argv + 1, *argcp);
|
2006-06-05 17:43:52 +00:00
|
|
|
|
2019-02-22 22:25:01 +00:00
|
|
|
trace2_cmd_alias(alias_command, new_argv);
|
|
|
|
|
2006-07-30 01:30:29 +00:00
|
|
|
*argv = new_argv;
|
|
|
|
*argcp += count - 1;
|
2006-06-05 17:43:52 +00:00
|
|
|
|
2006-07-30 01:30:29 +00:00
|
|
|
ret = 1;
|
2006-06-05 17:43:52 +00:00
|
|
|
}
|
|
|
|
|
2006-06-28 10:45:27 +00:00
|
|
|
errno = saved_errno;
|
|
|
|
|
2006-06-05 17:43:52 +00:00
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
2009-01-28 07:33:53 +00:00
|
|
|
static int run_builtin(struct cmd_struct *p, int argc, const char **argv)
|
2007-06-24 17:10:40 +00:00
|
|
|
{
|
2009-11-09 15:05:01 +00:00
|
|
|
int status, help;
|
2007-06-24 17:29:33 +00:00
|
|
|
struct stat st;
|
2007-06-24 17:10:40 +00:00
|
|
|
const char *prefix;
|
2021-12-06 15:55:55 +00:00
|
|
|
int run_setup = (p->option & (RUN_SETUP | RUN_SETUP_GENTLY));
|
2007-06-24 17:10:40 +00:00
|
|
|
|
2009-11-09 15:05:01 +00:00
|
|
|
help = argc == 2 && !strcmp(argv[1], "-h");
|
2021-12-06 15:55:55 +00:00
|
|
|
if (help && (run_setup & RUN_SETUP))
|
|
|
|
/* demote to GENTLY to allow 'git cmd -h' outside repo */
|
|
|
|
run_setup = RUN_SETUP_GENTLY;
|
|
|
|
|
|
|
|
if (run_setup & RUN_SETUP) {
|
|
|
|
prefix = setup_git_directory();
|
|
|
|
} else if (run_setup & RUN_SETUP_GENTLY) {
|
|
|
|
int nongit_ok;
|
|
|
|
prefix = setup_git_directory_gently(&nongit_ok);
|
|
|
|
} else {
|
|
|
|
prefix = NULL;
|
2009-11-09 15:05:01 +00:00
|
|
|
}
|
built-ins: trust the "prefix" from run_builtin()
Change code in "builtin/grep.c" and "builtin/ls-tree.c" to trust the
"prefix" passed from "run_builtin()". The "prefix" we get from setup.c
is either going to be NULL or a string of length >0, never "".
So we can drop the "prefix && *prefix" checks added for
"builtin/grep.c" in 0d042fecf2f (git-grep: show pathnames relative to
the current directory, 2006-08-11), and for "builtin/ls-tree.c" in
a69dd585fca (ls-tree: chomp leading directories when run from a
subdirectory, 2005-12-23).
As seen in code in revision.c that was added in cd676a51367 (diff
--relative: output paths as relative to the current subdirectory,
2008-02-12) we already have existing code that does away with this
assertion.
This makes it easier to reason about a subsequent change to the
"prefix_length" code in grep.c in a subsequent commit, and since we're
going to the trouble of doing that let's leave behind an assert() to
promise this to any future callers.
For "builtin/grep.c" it would be painful to pass the "prefix" down the
callchain of:
cmd_grep -> grep_tree -> grep_submodule -> grep_cache -> grep_oid ->
grep_source_name
So for the code that needs it in grep_source_name() let's add a
"grep_prefix" variable similar to the existing "ls_tree_prefix".
While at it let's move the code in cmd_ls_tree() around so that we
assign to the "ls_tree_prefix" right after declaring the variables,
and stop assigning to "prefix". We only subsequently used that
variable later in the function after clobbering it. Let's just use our
own "grep_prefix" instead.
Let's also add an assert() in git.c, so that we'll make this promise
about the "prefix" to any current and future callers, as well as to
any readers of the code.
Code history:
* The strlen() in "grep.c" hasn't been used since 493b7a08d80 (grep:
accept relative paths outside current working directory, 2009-09-05).
When that code was added in 0d042fecf2f (git-grep: show pathnames
relative to the current directory, 2006-08-11) we used the length.
But since 493b7a08d80 we haven't used it for anything except a
boolean check that we could have done on the "prefix" member
itself.
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2022-02-16 00:00:34 +00:00
|
|
|
assert(!prefix || *prefix);
|
2021-12-06 15:55:55 +00:00
|
|
|
precompose_argv_prefix(argc, argv, NULL);
|
|
|
|
if (use_pager == -1 && run_setup &&
|
|
|
|
!(p->option & DELAY_PAGER_CONFIG))
|
|
|
|
use_pager = check_pager_config(p->cmd);
|
|
|
|
if (use_pager == -1 && p->option & USE_PAGER)
|
|
|
|
use_pager = 1;
|
|
|
|
if (run_setup && startup_info->have_repository)
|
|
|
|
/* get_git_dir() may set up repo, avoid that */
|
2023-02-19 00:25:27 +00:00
|
|
|
trace_repo_setup();
|
Allow per-command pager config
There is great debate over whether some commands should set
up a pager automatically. This patch allows individuals to
set their own pager preferences for each command, overriding
the default. For example, to disable the pager for git
status:
git config pager.status false
If "--pager" or "--no-pager" is specified on the command
line, it takes precedence over the config option.
There are two caveats:
- you can turn on the pager for plumbing commands.
Combined with "core.pager = always", this will probably
break a lot of things. Don't do it.
- This only works for builtin commands. The reason is
somewhat complex:
Calling git_config before we do setup_git_directory
has bad side effects, because it wants to know where
the git_dir is to find ".git/config". Unfortunately,
we cannot call setup_git_directory indiscriminately,
because some builtins (like "init") break if we do.
For builtins, this is OK, since we can just wait until
after we call setup_git_directory. But for aliases, we
don't know until we expand (recursively) which command
we're doing. This should not be a huge problem for
aliases, which can simply use "--pager" or "--no-pager"
in the alias as appropriate.
For external commands, however, we don't know we even
have an external command until we exec it, and by then
it is too late to check the config.
An alternative approach would be to have a config mode
where we don't bother looking at .git/config, but only
at the user and system config files. This would make the
behavior consistent across builtins, aliases, and
external commands, at the cost of not allowing per-repo
pager config for at all.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2008-07-03 11:46:57 +00:00
|
|
|
commit_pager_choice();
|
|
|
|
|
2009-11-09 15:05:01 +00:00
|
|
|
if (!help && p->option & NEED_WORK_TREE)
|
2007-11-03 11:23:11 +00:00
|
|
|
setup_work_tree();
|
|
|
|
|
2007-12-03 04:51:50 +00:00
|
|
|
trace_argv_printf(argv, "trace: built-in: git");
|
2019-02-22 22:25:01 +00:00
|
|
|
trace2_cmd_name(p->cmd);
|
2007-06-24 17:10:40 +00:00
|
|
|
|
2019-01-24 08:29:12 +00:00
|
|
|
validate_cache_entries(the_repository->index);
|
2007-06-24 17:29:33 +00:00
|
|
|
status = p->fn(argc, argv, prefix);
|
2019-01-24 08:29:12 +00:00
|
|
|
validate_cache_entries(the_repository->index);
|
2018-07-02 19:49:39 +00:00
|
|
|
|
2007-06-24 17:29:33 +00:00
|
|
|
if (status)
|
2009-07-05 18:57:46 +00:00
|
|
|
return status;
|
2007-06-24 17:29:33 +00:00
|
|
|
|
|
|
|
/* Somebody closed stdout? */
|
|
|
|
if (fstat(fileno(stdout), &st))
|
|
|
|
return 0;
|
|
|
|
/* Ignore write errors for pipes and sockets.. */
|
|
|
|
if (S_ISFIFO(st.st_mode) || S_ISSOCK(st.st_mode))
|
|
|
|
return 0;
|
|
|
|
|
|
|
|
/* Check for ENOSPC and EIO errors.. */
|
git: Try a bit harder not to lose errno in stdio
This switches the checks around upon the exit codepath of the
git wrapper, so that we may recover at least non-transient errors.
It's still not perfect. As I've been harping on, stdio simply isn't very
good for error reporting. For example, if an IO error happened, you'd want
to see EIO, wouldn't you? And yes, that's what the kernel would return.
However, with buffered stdio (and flushing outside of our control), what
would likely happen is that some intermediate error return _does_ return
EIO, but then the kernel might decide to re-mount the filesystem read-only
due to the error, and the actual *report* for us might be
"write failure on standard output: read-only filesystem"
which lost the EIO.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2007-06-30 18:44:20 +00:00
|
|
|
if (fflush(stdout))
|
2018-11-10 05:16:00 +00:00
|
|
|
die_errno(_("write failure on standard output"));
|
git: Try a bit harder not to lose errno in stdio
This switches the checks around upon the exit codepath of the
git wrapper, so that we may recover at least non-transient errors.
It's still not perfect. As I've been harping on, stdio simply isn't very
good for error reporting. For example, if an IO error happened, you'd want
to see EIO, wouldn't you? And yes, that's what the kernel would return.
However, with buffered stdio (and flushing outside of our control), what
would likely happen is that some intermediate error return _does_ return
EIO, but then the kernel might decide to re-mount the filesystem read-only
due to the error, and the actual *report* for us might be
"write failure on standard output: read-only filesystem"
which lost the EIO.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2007-06-30 18:44:20 +00:00
|
|
|
if (ferror(stdout))
|
2018-11-10 05:16:00 +00:00
|
|
|
die(_("unknown write failure on standard output"));
|
git: Try a bit harder not to lose errno in stdio
This switches the checks around upon the exit codepath of the
git wrapper, so that we may recover at least non-transient errors.
It's still not perfect. As I've been harping on, stdio simply isn't very
good for error reporting. For example, if an IO error happened, you'd want
to see EIO, wouldn't you? And yes, that's what the kernel would return.
However, with buffered stdio (and flushing outside of our control), what
would likely happen is that some intermediate error return _does_ return
EIO, but then the kernel might decide to re-mount the filesystem read-only
due to the error, and the actual *report* for us might be
"write failure on standard output: read-only filesystem"
which lost the EIO.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2007-06-30 18:44:20 +00:00
|
|
|
if (fclose(stdout))
|
2018-11-10 05:16:00 +00:00
|
|
|
die_errno(_("close failed on standard output"));
|
2007-06-24 17:29:33 +00:00
|
|
|
return 0;
|
2007-06-24 17:10:40 +00:00
|
|
|
}
|
|
|
|
|
2014-01-02 16:17:11 +00:00
|
|
|
static struct cmd_struct commands[] = {
|
|
|
|
{ "add", cmd_add, RUN_SETUP | NEED_WORK_TREE },
|
2015-08-04 13:52:06 +00:00
|
|
|
{ "am", cmd_am, RUN_SETUP | NEED_WORK_TREE },
|
2022-08-19 16:03:52 +00:00
|
|
|
{ "annotate", cmd_annotate, RUN_SETUP },
|
2014-01-02 16:17:11 +00:00
|
|
|
{ "apply", cmd_apply, RUN_SETUP_GENTLY },
|
2016-11-22 21:37:04 +00:00
|
|
|
{ "archive", cmd_archive, RUN_SETUP_GENTLY },
|
2022-11-10 16:36:46 +00:00
|
|
|
{ "bisect", cmd_bisect, RUN_SETUP },
|
2014-01-02 16:17:11 +00:00
|
|
|
{ "blame", cmd_blame, RUN_SETUP },
|
2017-11-19 15:03:49 +00:00
|
|
|
{ "branch", cmd_branch, RUN_SETUP | DELAY_PAGER_CONFIG },
|
2020-08-13 14:59:36 +00:00
|
|
|
{ "bugreport", cmd_bugreport, RUN_SETUP_GENTLY },
|
2022-08-19 16:03:52 +00:00
|
|
|
{ "bundle", cmd_bundle, RUN_SETUP_GENTLY },
|
2014-01-02 16:17:11 +00:00
|
|
|
{ "cat-file", cmd_cat_file, RUN_SETUP },
|
|
|
|
{ "check-attr", cmd_check_attr, RUN_SETUP },
|
|
|
|
{ "check-ignore", cmd_check_ignore, RUN_SETUP | NEED_WORK_TREE },
|
|
|
|
{ "check-mailmap", cmd_check_mailmap, RUN_SETUP },
|
2018-03-24 20:35:19 +00:00
|
|
|
{ "check-ref-format", cmd_check_ref_format, NO_PARSEOPT },
|
2015-07-06 17:30:56 +00:00
|
|
|
{ "checkout", cmd_checkout, RUN_SETUP | NEED_WORK_TREE },
|
parallel-checkout: make it truly parallel
Use multiple worker processes to distribute the queued entries and call
write_pc_item() in parallel for them. The items are distributed
uniformly in contiguous chunks. This minimizes the chances of two
workers writing to the same directory simultaneously, which could affect
performance due to lock contention in the kernel. Work stealing (or any
other format of re-distribution) is not implemented yet.
The protocol between the main process and the workers is quite simple.
They exchange binary messages packed in pkt-line format, and use
PKT-FLUSH to mark the end of input (from both sides). The main process
starts the communication by sending N pkt-lines, each corresponding to
an item that needs to be written. These packets contain all the
necessary information to load, smudge, and write the blob associated
with each item. Then it waits for the worker to send back N pkt-lines
containing the results for each item. The resulting packet must contain:
the identification number of the item that it refers to, the status of
the operation, and the lstat() data gathered after writing the file (iff
the operation was successful).
For now, checkout always uses a hardcoded value of 2 workers, only to
demonstrate that the parallel checkout framework correctly divides and
writes the queued entries. The next patch will add user configurations
and define a more reasonable default, based on tests with the said
settings.
Co-authored-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com>
Co-authored-by: Jeff Hostetler <jeffhost@microsoft.com>
Signed-off-by: Matheus Tavares <matheus.bernardino@usp.br>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2021-04-19 00:14:54 +00:00
|
|
|
{ "checkout--worker", cmd_checkout__worker,
|
read-tree: add "--super-prefix" option, eliminate global
The "--super-prefix" option to "git" was initially added in [1] for
use with "ls-files"[2], and shortly thereafter "submodule--helper"[3]
and "grep"[4]. It wasn't until [5] that "read-tree" made use of it.
At the time [5] made sense, but since then we've made "ls-files"
recurse in-process in [6], "grep" in [7], and finally
"submodule--helper" in the preceding commits.
Let's also remove it from "read-tree", which allows us to remove the
option to "git" itself.
We can do this because the only remaining user of it is the submodule
API, which will now invoke "read-tree" with its new "--super-prefix"
option. It will only do so when the "submodule_move_head()" function
is called.
That "submodule_move_head()" function was then only invoked by
"read-tree" itself, but now rather than setting an environment
variable to pass "--super-prefix" between cmd_read_tree() we:
- Set a new "super_prefix" in "struct unpack_trees_options". The
"super_prefixed()" function in "unpack-trees.c" added in [5] will now
use this, rather than get_super_prefix() looking up the environment
variable we set earlier in the same process.
- Add the same field to the "struct checkout", which is only needed to
ferry the "super_prefix" in the "struct unpack_trees_options" all the
way down to the "entry.c" callers of "submodule_move_head()".
Those calls which used the super prefix all originated in
"cmd_read_tree()". The only other caller is the "unlink_entry()"
caller in "builtin/checkout.c", which now passes a "NULL".
1. 74866d75793 (git: make super-prefix option, 2016-10-07)
2. e77aa336f11 (ls-files: optionally recurse into submodules, 2016-10-07)
3. 89c86265576 (submodule helper: support super prefix, 2016-12-08)
4. 0281e487fd9 (grep: optionally recurse into submodules, 2016-12-16)
5. 3d415425c7b (unpack-trees: support super-prefix option, 2017-01-17)
6. 188dce131fa (ls-files: use repository object, 2017-06-22)
7. f9ee2fcdfa0 (grep: recurse in-process using 'struct repository', 2017-08-02)
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2022-12-20 12:39:56 +00:00
|
|
|
RUN_SETUP | NEED_WORK_TREE },
|
2014-01-02 16:17:11 +00:00
|
|
|
{ "checkout-index", cmd_checkout_index,
|
|
|
|
RUN_SETUP | NEED_WORK_TREE},
|
|
|
|
{ "cherry", cmd_cherry, RUN_SETUP },
|
|
|
|
{ "cherry-pick", cmd_cherry_pick, RUN_SETUP | NEED_WORK_TREE },
|
|
|
|
{ "clean", cmd_clean, RUN_SETUP | NEED_WORK_TREE },
|
setup.c: re-fix d95138e (setup: set env $GIT_WORK_TREE when ..
Commit d95138e [1] attempted to fix a .git file problem by
setting GIT_WORK_TREE whenever GIT_DIR is set. It sounded harmless
because we handle GIT_DIR and GIT_WORK_TREE side by side for most
commands, with two exceptions: git-init and git-clone.
"git clone" is not happy with d95138e. This command ignores GIT_DIR
but respects GIT_WORK_TREE [2] [3] which means it used to run fine
from a hook, where GIT_DIR was set but GIT_WORK_TREE was not (*).
With d95138e, GIT_WORK_TREE is set all the time and git-clone
interprets that as "I give you order to put the worktree here",
usually against the user's intention.
The solution in d95138e is reverted earlier, and instead we reuse
the solution from c056261 [4]. It fixed another setup-messed-
up-by-alias by saving and restoring env and spawning a new process,
but for git-clone and git-init only.
Now we conclude that setup-messed-up-by-alias is always evil. So the
env restoration is done for _all_ commands, including external ones,
whenever aliases are involved. It fixes what d95138e tried to fix,
without upsetting git-clone-inside-hooks.
The test from d95138e remains to verify it's not broken by this. A new
test is added to make sure git-clone-inside-hooks remains happy.
(*) GIT_WORK_TREE was not set _most of the time_. In some cases
GIT_WORK_TREE is set and git-clone will behave differently. The
use of GIT_WORK_TREE to direct git-clone to put work tree
elsewhere looks like a mistake because it causes surprises this
way. But that's a separate story.
[1] d95138e (setup: set env $GIT_WORK_TREE when work tree is set, like
$GIT_DIR - 2015-06-26)
[2] 2beebd2 (clone: create intermediate directories of destination
repo - 2008-06-25)
[3] 20ccef4 (make git-clone GIT_WORK_TREE aware - 2007-07-06)
[4] c056261 (git potty: restore environments after alias expansion -
2014-06-08)
Reported-by: Anthony Sottile <asottile@umich.edu>
Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2015-12-20 07:50:18 +00:00
|
|
|
{ "clone", cmd_clone },
|
2014-01-02 16:17:11 +00:00
|
|
|
{ "column", cmd_column, RUN_SETUP_GENTLY },
|
|
|
|
{ "commit", cmd_commit, RUN_SETUP | NEED_WORK_TREE },
|
2018-04-02 20:34:18 +00:00
|
|
|
{ "commit-graph", cmd_commit_graph, RUN_SETUP },
|
2022-08-19 16:03:52 +00:00
|
|
|
{ "commit-tree", cmd_commit_tree, RUN_SETUP },
|
2018-02-21 18:51:43 +00:00
|
|
|
{ "config", cmd_config, RUN_SETUP_GENTLY | DELAY_PAGER_CONFIG },
|
2014-01-02 16:17:11 +00:00
|
|
|
{ "count-objects", cmd_count_objects, RUN_SETUP },
|
2018-03-24 20:35:19 +00:00
|
|
|
{ "credential", cmd_credential, RUN_SETUP_GENTLY | NO_PARSEOPT },
|
2020-08-13 14:58:55 +00:00
|
|
|
{ "credential-cache", cmd_credential_cache },
|
|
|
|
{ "credential-cache--daemon", cmd_credential_cache_daemon },
|
|
|
|
{ "credential-store", cmd_credential_store },
|
2014-01-02 16:17:11 +00:00
|
|
|
{ "describe", cmd_describe, RUN_SETUP },
|
2022-08-12 20:10:15 +00:00
|
|
|
{ "diagnose", cmd_diagnose, RUN_SETUP_GENTLY },
|
2018-03-24 20:35:19 +00:00
|
|
|
{ "diff", cmd_diff, NO_PARSEOPT },
|
|
|
|
{ "diff-files", cmd_diff_files, RUN_SETUP | NEED_WORK_TREE | NO_PARSEOPT },
|
|
|
|
{ "diff-index", cmd_diff_index, RUN_SETUP | NO_PARSEOPT },
|
|
|
|
{ "diff-tree", cmd_diff_tree, RUN_SETUP | NO_PARSEOPT },
|
2019-03-14 11:25:04 +00:00
|
|
|
{ "difftool", cmd_difftool, RUN_SETUP_GENTLY },
|
2014-01-02 16:17:11 +00:00
|
|
|
{ "fast-export", cmd_fast_export, RUN_SETUP },
|
2020-08-13 14:59:45 +00:00
|
|
|
{ "fast-import", cmd_fast_import, RUN_SETUP | NO_PARSEOPT },
|
2014-01-02 16:17:11 +00:00
|
|
|
{ "fetch", cmd_fetch, RUN_SETUP },
|
2018-03-24 20:35:19 +00:00
|
|
|
{ "fetch-pack", cmd_fetch_pack, RUN_SETUP | NO_PARSEOPT },
|
2014-01-02 16:17:11 +00:00
|
|
|
{ "fmt-merge-msg", cmd_fmt_merge_msg, RUN_SETUP },
|
|
|
|
{ "for-each-ref", cmd_for_each_ref, RUN_SETUP },
|
2020-09-11 17:49:16 +00:00
|
|
|
{ "for-each-repo", cmd_for_each_repo, RUN_SETUP_GENTLY },
|
2014-01-02 16:17:11 +00:00
|
|
|
{ "format-patch", cmd_format_patch, RUN_SETUP },
|
|
|
|
{ "fsck", cmd_fsck, RUN_SETUP },
|
|
|
|
{ "fsck-objects", cmd_fsck, RUN_SETUP },
|
submodule--helper: don't use global --super-prefix in "absorbgitdirs"
The "--super-prefix" facility was introduced in [1] has always been a
transitory hack, which is why we've made it an error to supply it as
an option to "git" to commands that don't know about it.
That's been a good goal, as it has a global effect we haven't wanted
calls to get_super_prefix() from built-ins we didn't expect.
But it has meant that when we've had chains of different built-ins
using it all of the processes in that "chain" have needed to support
it, and worse processes that don't need it have needed to ask for
"SUPPORT_SUPER_PREFIX" because their parent process needs it.
That's how "fsmonitor--daemon" ended up with it, per [2] it's called
from (among other things) "submodule--helper absorbgitdirs", but as we
declared "submodule--helper" as "SUPPORT_SUPER_PREFIX" we needed to
declare "fsmonitor--daemon" as accepting it too, even though it
doesn't care about it.
But in the case of "absorbgitdirs" it only needed "--super-prefix" to
invoke itself recursively, and we'd never have another "in-between"
process in the chain. So we didn't need the bigger hammer of "git
--super-prefix", and the "setenv(GIT_SUPER_PREFIX_ENVIRONMENT, ...)"
that it entails.
Let's instead accept a hidden "--super-prefix" option to
"submodule--helper absorbgitdirs" itself.
Eventually (as with all other "--super-prefix" users) we'll want to
clean this code up so that this all happens in-process. I.e. needing
any variant of "--super-prefix" is itself a hack around our various
global state, and implicit reliance on "the_repository". This stepping
stone makes such an eventual change easier, as we'll need to deal with
less global state at that point.
The "fsmonitor--daemon" test adjusted here was added in [3]. To assert
that it didn't run into the "--super-prefix" message it was asserting
the output it didn't have. Let's instead assert the full output that
we *do* have, using the same pattern as a preceding change to
"t/t7412-submodule-absorbgitdirs.sh" used.
We could also remove the test entirely (as [4] did), but even though
the initial reason for having it is gone we're still getting some
marginal benefit from testing the "fsmonitor" and "submodule
absorbgitdirs" interaction, so let's keep it.
The change here to have either a NULL or non-"" string as a
"super_prefix" instead of the previous arrangement of "" or non-"" is
somewhat arbitrary. We could also decide to never have to check for
NULL.
As we'll be changing the rest of the "git --super-prefix" users to the
same pattern, leaving them all consistent makes sense. Why not pick ""
over NULL? Because that's how the "prefix" works[5], and having
"prefix" and "super_prefix" work the same way will be less
confusing. That "prefix" picked NULL instead of "" is itself
arbitrary, but as it's easy to make this small bit of our overall API
consistent, let's go with that.
1. 74866d75793 (git: make super-prefix option, 2016-10-07)
2. 53fcfbc84f6 (fsmonitor--daemon: allow --super-prefix argument,
2022-05-26)
3. 53fcfbc84f6 (fsmonitor--daemon: allow --super-prefix argument,
2022-05-26)
4. https://lore.kernel.org/git/20221109004708.97668-5-chooglen@google.com/
5. 9725c8dda20 (built-ins: trust the "prefix" from run_builtin(),
2022-02-16)
Signed-off-by: Glen Choo <chooglen@google.com>
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2022-12-20 12:39:51 +00:00
|
|
|
{ "fsmonitor--daemon", cmd_fsmonitor__daemon, RUN_SETUP },
|
2014-01-02 16:17:11 +00:00
|
|
|
{ "gc", cmd_gc, RUN_SETUP },
|
2018-03-24 20:35:19 +00:00
|
|
|
{ "get-tar-commit-id", cmd_get_tar_commit_id, NO_PARSEOPT },
|
2017-08-02 19:49:23 +00:00
|
|
|
{ "grep", cmd_grep, RUN_SETUP_GENTLY },
|
2014-01-02 16:17:11 +00:00
|
|
|
{ "hash-object", cmd_hash_object },
|
|
|
|
{ "help", cmd_help },
|
2021-12-22 03:59:27 +00:00
|
|
|
{ "hook", cmd_hook, RUN_SETUP },
|
2018-03-24 20:35:19 +00:00
|
|
|
{ "index-pack", cmd_index_pack, RUN_SETUP_GENTLY | NO_PARSEOPT },
|
setup.c: re-fix d95138e (setup: set env $GIT_WORK_TREE when ..
Commit d95138e [1] attempted to fix a .git file problem by
setting GIT_WORK_TREE whenever GIT_DIR is set. It sounded harmless
because we handle GIT_DIR and GIT_WORK_TREE side by side for most
commands, with two exceptions: git-init and git-clone.
"git clone" is not happy with d95138e. This command ignores GIT_DIR
but respects GIT_WORK_TREE [2] [3] which means it used to run fine
from a hook, where GIT_DIR was set but GIT_WORK_TREE was not (*).
With d95138e, GIT_WORK_TREE is set all the time and git-clone
interprets that as "I give you order to put the worktree here",
usually against the user's intention.
The solution in d95138e is reverted earlier, and instead we reuse
the solution from c056261 [4]. It fixed another setup-messed-
up-by-alias by saving and restoring env and spawning a new process,
but for git-clone and git-init only.
Now we conclude that setup-messed-up-by-alias is always evil. So the
env restoration is done for _all_ commands, including external ones,
whenever aliases are involved. It fixes what d95138e tried to fix,
without upsetting git-clone-inside-hooks.
The test from d95138e remains to verify it's not broken by this. A new
test is added to make sure git-clone-inside-hooks remains happy.
(*) GIT_WORK_TREE was not set _most of the time_. In some cases
GIT_WORK_TREE is set and git-clone will behave differently. The
use of GIT_WORK_TREE to direct git-clone to put work tree
elsewhere looks like a mistake because it causes surprises this
way. But that's a separate story.
[1] d95138e (setup: set env $GIT_WORK_TREE when work tree is set, like
$GIT_DIR - 2015-06-26)
[2] 2beebd2 (clone: create intermediate directories of destination
repo - 2008-06-25)
[3] 20ccef4 (make git-clone GIT_WORK_TREE aware - 2007-07-06)
[4] c056261 (git potty: restore environments after alias expansion -
2014-06-08)
Reported-by: Anthony Sottile <asottile@umich.edu>
Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2015-12-20 07:50:18 +00:00
|
|
|
{ "init", cmd_init_db },
|
|
|
|
{ "init-db", cmd_init_db },
|
2015-09-05 13:39:24 +00:00
|
|
|
{ "interpret-trailers", cmd_interpret_trailers, RUN_SETUP_GENTLY },
|
2014-01-02 16:17:11 +00:00
|
|
|
{ "log", cmd_log, RUN_SETUP },
|
2017-06-22 18:43:48 +00:00
|
|
|
{ "ls-files", cmd_ls_files, RUN_SETUP },
|
2014-01-02 16:17:11 +00:00
|
|
|
{ "ls-remote", cmd_ls_remote, RUN_SETUP_GENTLY },
|
|
|
|
{ "ls-tree", cmd_ls_tree, RUN_SETUP },
|
2022-08-19 16:03:52 +00:00
|
|
|
{ "mailinfo", cmd_mailinfo, RUN_SETUP_GENTLY },
|
2018-03-24 20:35:19 +00:00
|
|
|
{ "mailsplit", cmd_mailsplit, NO_PARSEOPT },
|
builtin/gc.c: let parse-options parse 'git maintenance's subcommands
'git maintenanze' parses its subcommands with a couple of if
statements. parse-options has just learned to parse subcommands, so
let's use that facility instead, with the benefits of shorter code,
handling missing or unknown subcommands, and listing subcommands for
Bash completion.
This change makes 'git maintenance' consistent with other commands in
that the help text shown for '-h' goes to standard output, not error,
in the exit code and error message on unknown subcommand, and the
error message on missing subcommand. There is a test checking these,
which is now updated accordingly.
Note that some of the functions implementing each subcommand don't
accept any parameters, so add the (unused) 'argc', '**argv' and
'*prefix' parameters to make them match the type expected by
parse-options, and thus avoid casting function pointers.
Signed-off-by: SZEDER Gábor <szeder.dev@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2022-08-19 16:04:03 +00:00
|
|
|
{ "maintenance", cmd_maintenance, RUN_SETUP },
|
2014-01-02 16:17:11 +00:00
|
|
|
{ "merge", cmd_merge, RUN_SETUP | NEED_WORK_TREE },
|
|
|
|
{ "merge-base", cmd_merge_base, RUN_SETUP },
|
|
|
|
{ "merge-file", cmd_merge_file, RUN_SETUP_GENTLY },
|
2018-03-24 20:35:19 +00:00
|
|
|
{ "merge-index", cmd_merge_index, RUN_SETUP | NO_PARSEOPT },
|
|
|
|
{ "merge-ours", cmd_merge_ours, RUN_SETUP | NO_PARSEOPT },
|
|
|
|
{ "merge-recursive", cmd_merge_recursive, RUN_SETUP | NEED_WORK_TREE | NO_PARSEOPT },
|
|
|
|
{ "merge-recursive-ours", cmd_merge_recursive, RUN_SETUP | NEED_WORK_TREE | NO_PARSEOPT },
|
|
|
|
{ "merge-recursive-theirs", cmd_merge_recursive, RUN_SETUP | NEED_WORK_TREE | NO_PARSEOPT },
|
|
|
|
{ "merge-subtree", cmd_merge_recursive, RUN_SETUP | NEED_WORK_TREE | NO_PARSEOPT },
|
2022-06-18 00:20:46 +00:00
|
|
|
{ "merge-tree", cmd_merge_tree, RUN_SETUP },
|
2022-08-19 16:03:52 +00:00
|
|
|
{ "mktag", cmd_mktag, RUN_SETUP },
|
2014-01-02 16:17:11 +00:00
|
|
|
{ "mktree", cmd_mktree, RUN_SETUP },
|
midx: disallow running outside of a repository
The multi-pack-index command supports working with arbitrary object
directories via the `--object-dir` flag. Though this has historically
worked in arbitrary repositories (including when the command itself was
run outside of a Git repository), this has been somewhat of an accident.
For example, running:
git multi-pack-index write --object-dir=/path/to/repo/objects
outside of a Git repository causes a BUG(). This is because the
top-level `cmd_multi_pack_index()` function stops parsing when it sees
"write", and then fills in the default object directory (the result of
calling `get_object_directory()`) before handing off to
`cmd_multi_pack_index_write()`. But there is no repository to
initialize, and so calling `get_object_directory()` results in a BUG()
(indicating that the current repository is not initialized).
Another case where this doesn't quite work as expected is when operating
in a SHA-256 repository. To see the failure, try this in your shell:
git init --object-format=sha256 repo
git -C repo commit --allow-empty base
git -C repo repack -d
git multi-pack-index --object-dir=$(pwd)/repo/.git/objects write
and observe that we cannot open the `.idx` file in "repo", because the
outermost process assumes that any repository that it works in also uses
the default value of `the_hash_algo` (at the time of writing, SHA-1).
There may be compelling reasons for trying to work around these bugs,
but working in arbitrary `--object-dir`'s is non-standard enough (and
likewise, these bugs prevalent enough) that I don't think any workflows
would be broken by abandoning this behavior.
Accordingly, restrict the `multi-pack-index` builtin to only work when
inside of a Git repository (i.e., its main utility becomes selecting
which alternate to operate in), which avoids both of the bugs above.
(Note that you can still trigger a bug when writing a MIDX in an
alternate which does not use the same object format as the repository
which it is an alternate of, but that is an unrelated bug to this one).
Signed-off-by: Taylor Blau <me@ttaylorr.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2021-08-31 20:51:53 +00:00
|
|
|
{ "multi-pack-index", cmd_multi_pack_index, RUN_SETUP },
|
2014-01-02 16:17:11 +00:00
|
|
|
{ "mv", cmd_mv, RUN_SETUP | NEED_WORK_TREE },
|
|
|
|
{ "name-rev", cmd_name_rev, RUN_SETUP },
|
|
|
|
{ "notes", cmd_notes, RUN_SETUP },
|
|
|
|
{ "pack-objects", cmd_pack_objects, RUN_SETUP },
|
2018-03-24 20:35:19 +00:00
|
|
|
{ "pack-redundant", cmd_pack_redundant, RUN_SETUP | NO_PARSEOPT },
|
2014-01-02 16:17:11 +00:00
|
|
|
{ "pack-refs", cmd_pack_refs, RUN_SETUP },
|
2018-03-24 20:35:19 +00:00
|
|
|
{ "patch-id", cmd_patch_id, RUN_SETUP_GENTLY | NO_PARSEOPT },
|
2014-01-02 16:17:11 +00:00
|
|
|
{ "pickaxe", cmd_blame, RUN_SETUP },
|
|
|
|
{ "prune", cmd_prune, RUN_SETUP },
|
|
|
|
{ "prune-packed", cmd_prune_packed, RUN_SETUP },
|
2015-06-14 08:41:51 +00:00
|
|
|
{ "pull", cmd_pull, RUN_SETUP | NEED_WORK_TREE },
|
2014-01-02 16:17:11 +00:00
|
|
|
{ "push", cmd_push, RUN_SETUP },
|
2018-08-13 11:33:02 +00:00
|
|
|
{ "range-diff", cmd_range_diff, RUN_SETUP | USE_PAGER },
|
read-tree: add "--super-prefix" option, eliminate global
The "--super-prefix" option to "git" was initially added in [1] for
use with "ls-files"[2], and shortly thereafter "submodule--helper"[3]
and "grep"[4]. It wasn't until [5] that "read-tree" made use of it.
At the time [5] made sense, but since then we've made "ls-files"
recurse in-process in [6], "grep" in [7], and finally
"submodule--helper" in the preceding commits.
Let's also remove it from "read-tree", which allows us to remove the
option to "git" itself.
We can do this because the only remaining user of it is the submodule
API, which will now invoke "read-tree" with its new "--super-prefix"
option. It will only do so when the "submodule_move_head()" function
is called.
That "submodule_move_head()" function was then only invoked by
"read-tree" itself, but now rather than setting an environment
variable to pass "--super-prefix" between cmd_read_tree() we:
- Set a new "super_prefix" in "struct unpack_trees_options". The
"super_prefixed()" function in "unpack-trees.c" added in [5] will now
use this, rather than get_super_prefix() looking up the environment
variable we set earlier in the same process.
- Add the same field to the "struct checkout", which is only needed to
ferry the "super_prefix" in the "struct unpack_trees_options" all the
way down to the "entry.c" callers of "submodule_move_head()".
Those calls which used the super prefix all originated in
"cmd_read_tree()". The only other caller is the "unlink_entry()"
caller in "builtin/checkout.c", which now passes a "NULL".
1. 74866d75793 (git: make super-prefix option, 2016-10-07)
2. e77aa336f11 (ls-files: optionally recurse into submodules, 2016-10-07)
3. 89c86265576 (submodule helper: support super prefix, 2016-12-08)
4. 0281e487fd9 (grep: optionally recurse into submodules, 2016-12-16)
5. 3d415425c7b (unpack-trees: support super-prefix option, 2017-01-17)
6. 188dce131fa (ls-files: use repository object, 2017-06-22)
7. f9ee2fcdfa0 (grep: recurse in-process using 'struct repository', 2017-08-02)
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2022-12-20 12:39:56 +00:00
|
|
|
{ "read-tree", cmd_read_tree, RUN_SETUP },
|
2019-07-24 21:15:00 +00:00
|
|
|
{ "rebase", cmd_rebase, RUN_SETUP | NEED_WORK_TREE },
|
2014-01-02 16:17:11 +00:00
|
|
|
{ "receive-pack", cmd_receive_pack },
|
|
|
|
{ "reflog", cmd_reflog, RUN_SETUP },
|
|
|
|
{ "remote", cmd_remote, RUN_SETUP },
|
2018-03-24 20:35:19 +00:00
|
|
|
{ "remote-ext", cmd_remote_ext, NO_PARSEOPT },
|
|
|
|
{ "remote-fd", cmd_remote_fd, NO_PARSEOPT },
|
2014-01-02 16:17:11 +00:00
|
|
|
{ "repack", cmd_repack, RUN_SETUP },
|
|
|
|
{ "replace", cmd_replace, RUN_SETUP },
|
2023-11-24 11:10:31 +00:00
|
|
|
{ "replay", cmd_replay, RUN_SETUP },
|
2014-01-02 16:17:11 +00:00
|
|
|
{ "rerere", cmd_rerere, RUN_SETUP },
|
|
|
|
{ "reset", cmd_reset, RUN_SETUP },
|
2019-04-25 09:45:45 +00:00
|
|
|
{ "restore", cmd_restore, RUN_SETUP | NEED_WORK_TREE },
|
2018-03-24 20:35:19 +00:00
|
|
|
{ "rev-list", cmd_rev_list, RUN_SETUP | NO_PARSEOPT },
|
|
|
|
{ "rev-parse", cmd_rev_parse, NO_PARSEOPT },
|
2014-01-02 16:17:11 +00:00
|
|
|
{ "revert", cmd_revert, RUN_SETUP | NEED_WORK_TREE },
|
|
|
|
{ "rm", cmd_rm, RUN_SETUP },
|
|
|
|
{ "send-pack", cmd_send_pack, RUN_SETUP },
|
|
|
|
{ "shortlog", cmd_shortlog, RUN_SETUP_GENTLY | USE_PAGER },
|
|
|
|
{ "show", cmd_show, RUN_SETUP },
|
|
|
|
{ "show-branch", cmd_show_branch, RUN_SETUP },
|
2020-05-25 19:59:12 +00:00
|
|
|
{ "show-index", cmd_show_index, RUN_SETUP_GENTLY },
|
2014-01-02 16:17:11 +00:00
|
|
|
{ "show-ref", cmd_show_ref, RUN_SETUP },
|
2023-03-27 07:55:02 +00:00
|
|
|
{ "sparse-checkout", cmd_sparse_checkout, RUN_SETUP },
|
2014-01-02 16:17:11 +00:00
|
|
|
{ "stage", cmd_add, RUN_SETUP | NEED_WORK_TREE },
|
2020-03-03 17:46:13 +00:00
|
|
|
{ "stash", cmd_stash, RUN_SETUP | NEED_WORK_TREE },
|
2014-01-02 16:17:11 +00:00
|
|
|
{ "status", cmd_status, RUN_SETUP | NEED_WORK_TREE },
|
|
|
|
{ "stripspace", cmd_stripspace },
|
2022-12-20 12:39:55 +00:00
|
|
|
{ "submodule--helper", cmd_submodule__helper, RUN_SETUP },
|
2019-03-29 10:39:05 +00:00
|
|
|
{ "switch", cmd_switch, RUN_SETUP | NEED_WORK_TREE },
|
2014-01-02 16:17:11 +00:00
|
|
|
{ "symbolic-ref", cmd_symbolic_ref, RUN_SETUP },
|
2017-08-02 19:40:53 +00:00
|
|
|
{ "tag", cmd_tag, RUN_SETUP | DELAY_PAGER_CONFIG },
|
2018-03-24 20:35:19 +00:00
|
|
|
{ "unpack-file", cmd_unpack_file, RUN_SETUP | NO_PARSEOPT },
|
|
|
|
{ "unpack-objects", cmd_unpack_objects, RUN_SETUP | NO_PARSEOPT },
|
2014-01-02 16:17:11 +00:00
|
|
|
{ "update-index", cmd_update_index, RUN_SETUP },
|
|
|
|
{ "update-ref", cmd_update_ref, RUN_SETUP },
|
|
|
|
{ "update-server-info", cmd_update_server_info, RUN_SETUP },
|
2018-03-24 20:35:19 +00:00
|
|
|
{ "upload-archive", cmd_upload_archive, NO_PARSEOPT },
|
|
|
|
{ "upload-archive--writer", cmd_upload_archive_writer, NO_PARSEOPT },
|
2018-03-14 18:31:41 +00:00
|
|
|
{ "upload-pack", cmd_upload_pack },
|
2018-03-24 20:35:19 +00:00
|
|
|
{ "var", cmd_var, RUN_SETUP_GENTLY | NO_PARSEOPT },
|
2014-06-23 07:05:49 +00:00
|
|
|
{ "verify-commit", cmd_verify_commit, RUN_SETUP },
|
2014-01-02 16:17:11 +00:00
|
|
|
{ "verify-pack", cmd_verify_pack },
|
|
|
|
{ "verify-tag", cmd_verify_tag, RUN_SETUP },
|
|
|
|
{ "version", cmd_version },
|
|
|
|
{ "whatchanged", cmd_whatchanged, RUN_SETUP },
|
2022-08-19 16:04:11 +00:00
|
|
|
{ "worktree", cmd_worktree, RUN_SETUP },
|
2014-01-02 16:17:11 +00:00
|
|
|
{ "write-tree", cmd_write_tree, RUN_SETUP },
|
|
|
|
};
|
|
|
|
|
2014-11-12 13:10:22 +00:00
|
|
|
static struct cmd_struct *get_builtin(const char *s)
|
2014-01-02 16:17:11 +00:00
|
|
|
{
|
|
|
|
int i;
|
|
|
|
for (i = 0; i < ARRAY_SIZE(commands); i++) {
|
2014-11-12 13:10:22 +00:00
|
|
|
struct cmd_struct *p = commands + i;
|
2014-01-02 16:17:11 +00:00
|
|
|
if (!strcmp(s, p->cmd))
|
2014-11-12 13:10:22 +00:00
|
|
|
return p;
|
2014-01-02 16:17:11 +00:00
|
|
|
}
|
2014-11-12 13:10:22 +00:00
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
|
|
|
int is_builtin(const char *s)
|
|
|
|
{
|
|
|
|
return !!get_builtin(s);
|
2014-01-02 16:17:11 +00:00
|
|
|
}
|
|
|
|
|
2018-05-20 18:39:58 +00:00
|
|
|
static void list_builtins(struct string_list *out, unsigned int exclude_option)
|
2017-05-30 05:18:43 +00:00
|
|
|
{
|
|
|
|
int i;
|
2018-03-24 20:35:19 +00:00
|
|
|
for (i = 0; i < ARRAY_SIZE(commands); i++) {
|
|
|
|
if (exclude_option &&
|
|
|
|
(commands[i].option & exclude_option))
|
|
|
|
continue;
|
2018-05-20 18:39:58 +00:00
|
|
|
string_list_append(out, commands[i].cmd);
|
2018-03-24 20:35:19 +00:00
|
|
|
}
|
2017-05-30 05:18:43 +00:00
|
|
|
}
|
|
|
|
|
2020-10-07 21:56:51 +00:00
|
|
|
void load_builtin_commands(const char *prefix, struct cmdnames *cmds)
|
|
|
|
{
|
|
|
|
const char *name;
|
|
|
|
int i;
|
|
|
|
|
|
|
|
/*
|
|
|
|
* Callers can ask for a subset of the commands based on a certain
|
|
|
|
* prefix, which is then dropped from the added names. The names in
|
|
|
|
* the `commands[]` array do not have the `git-` prefix, though,
|
|
|
|
* therefore we must expect the `prefix` to at least start with `git-`.
|
|
|
|
*/
|
|
|
|
if (!skip_prefix(prefix, "git-", &prefix))
|
|
|
|
BUG("prefix '%s' must start with 'git-'", prefix);
|
|
|
|
|
|
|
|
for (i = 0; i < ARRAY_SIZE(commands); i++)
|
|
|
|
if (skip_prefix(commands[i].cmd, prefix, &name))
|
|
|
|
add_cmdname(cmds, name, strlen(name));
|
|
|
|
}
|
|
|
|
|
2016-02-22 07:18:29 +00:00
|
|
|
#ifdef STRIP_EXTENSION
|
|
|
|
static void strip_extension(const char **argv)
|
|
|
|
{
|
|
|
|
size_t len;
|
|
|
|
|
|
|
|
if (strip_suffix(argv[0], STRIP_EXTENSION, &len))
|
|
|
|
argv[0] = xmemdupz(argv[0], len);
|
|
|
|
}
|
|
|
|
#else
|
|
|
|
#define strip_extension(cmd)
|
|
|
|
#endif
|
|
|
|
|
2014-01-02 16:15:44 +00:00
|
|
|
static void handle_builtin(int argc, const char **argv)
|
Teach the "git" command to handle some commands internally
This is another patch in the "prepare to do more in C" series, where the
git wrapper command is taught about the notion of handling some
functionality internally.
Right now, the only internal commands are "version" and "help", but the
point being that we can now easily extend it to handle some of the trivial
scripts internally. Things like "git log" and "git diff" wouldn't need
separate external scripts any more.
This also implies that to support the old "git-log" and "git-diff" syntax,
the "git" wrapper now automatically looks at the name it was executed as,
and if it is "git-xxxx", it will assume that it is to internally do what
"git xxxx" would do.
In other words, you can (once you implement an internal command) soft- or
hard-link that command to the "git" wrapper command, and it will do the
right thing, whether you use the "git xxxx" or the "git-xxxx" format.
There's one other change: the search order for external programs is
modified slightly, so that the first entry remains GIT_EXEC_DIR, but the
second entry is the same directory as the git wrapper itself was executed
out of - if we can figure it out from argv[0], of course.
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
2006-02-26 20:34:51 +00:00
|
|
|
{
|
2020-07-28 20:24:53 +00:00
|
|
|
struct strvec args = STRVEC_INIT;
|
2016-02-22 07:18:29 +00:00
|
|
|
const char *cmd;
|
2014-11-12 13:10:22 +00:00
|
|
|
struct cmd_struct *builtin;
|
2007-12-08 19:57:25 +00:00
|
|
|
|
2016-02-22 07:18:29 +00:00
|
|
|
strip_extension(argv);
|
|
|
|
cmd = argv[0];
|
Teach the "git" command to handle some commands internally
This is another patch in the "prepare to do more in C" series, where the
git wrapper command is taught about the notion of handling some
functionality internally.
Right now, the only internal commands are "version" and "help", but the
point being that we can now easily extend it to handle some of the trivial
scripts internally. Things like "git log" and "git diff" wouldn't need
separate external scripts any more.
This also implies that to support the old "git-log" and "git-diff" syntax,
the "git" wrapper now automatically looks at the name it was executed as,
and if it is "git-xxxx", it will assume that it is to internally do what
"git xxxx" would do.
In other words, you can (once you implement an internal command) soft- or
hard-link that command to the "git" wrapper command, and it will do the
right thing, whether you use the "git xxxx" or the "git-xxxx" format.
There's one other change: the search order for external programs is
modified slightly, so that the first entry remains GIT_EXEC_DIR, but the
second entry is the same directory as the git wrapper itself was executed
out of - if we can figure it out from argv[0], of course.
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
2006-02-26 20:34:51 +00:00
|
|
|
|
2016-08-26 17:58:36 +00:00
|
|
|
/* Turn "git cmd --help" into "git help --exclude-guides cmd" */
|
2006-04-15 18:13:49 +00:00
|
|
|
if (argc > 1 && !strcmp(argv[1], "--help")) {
|
2016-08-26 17:58:36 +00:00
|
|
|
int i;
|
|
|
|
|
2006-04-15 18:13:49 +00:00
|
|
|
argv[1] = argv[0];
|
|
|
|
argv[0] = cmd = "help";
|
2016-08-26 17:58:36 +00:00
|
|
|
|
|
|
|
for (i = 0; i < argc; i++) {
|
2020-07-28 20:24:53 +00:00
|
|
|
strvec_push(&args, argv[i]);
|
2016-08-26 17:58:36 +00:00
|
|
|
if (!i)
|
2020-07-28 20:24:53 +00:00
|
|
|
strvec_push(&args, "--exclude-guides");
|
2016-08-26 17:58:36 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
argc++;
|
2020-07-29 00:37:20 +00:00
|
|
|
argv = args.v;
|
2006-04-15 18:13:49 +00:00
|
|
|
}
|
|
|
|
|
2014-11-12 13:10:22 +00:00
|
|
|
builtin = get_builtin(cmd);
|
git: simplify environment save/restore logic
The only code that cares about the value of the global variable
saved_env_before_alias after the previous fix is handle_builtin()
that turns into a glorified no-op when the variable is true, so the
logic could safely be lifted to its caller, i.e. the caller can
refrain from calling it when the variable is set.
This variable tells us if save_env_before_alias() was called (with
or without matching restore_env()), but the sole caller of the
function, handle_alias(), always calls it as the first thing, so we
can consider that the variable essentially keeps track of the fact
that handle_alias() has ever been called.
It turns out that handle_builtin() and handle_alias() are called
only from one function in a way that the value of the variable
matters, which is run_argv(), and it already keeps track of the
fact that it already called handle_alias().
So we can simplify the whole thing by:
- Change handle_builtin() to always make a direct call to the
builtin implementation it finds, and make sure the caller
refrains from calling it if handle_alias() has ever been
called;
- Remove saved_env_before_alias variable, and instead use the
local "done_alias" variable maintained inside run_argv() to
make the same decision.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2016-01-27 06:52:02 +00:00
|
|
|
if (builtin)
|
|
|
|
exit(run_builtin(builtin, argc, argv));
|
2020-07-28 20:24:53 +00:00
|
|
|
strvec_clear(&args);
|
Teach the "git" command to handle some commands internally
This is another patch in the "prepare to do more in C" series, where the
git wrapper command is taught about the notion of handling some
functionality internally.
Right now, the only internal commands are "version" and "help", but the
point being that we can now easily extend it to handle some of the trivial
scripts internally. Things like "git log" and "git diff" wouldn't need
separate external scripts any more.
This also implies that to support the old "git-log" and "git-diff" syntax,
the "git" wrapper now automatically looks at the name it was executed as,
and if it is "git-xxxx", it will assume that it is to internally do what
"git xxxx" would do.
In other words, you can (once you implement an internal command) soft- or
hard-link that command to the "git" wrapper command, and it will do the
right thing, whether you use the "git xxxx" or the "git-xxxx" format.
There's one other change: the search order for external programs is
modified slightly, so that the first entry remains GIT_EXEC_DIR, but the
second entry is the same directory as the git wrapper itself was executed
out of - if we can figure it out from argv[0], of course.
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
2006-02-26 20:34:51 +00:00
|
|
|
}
|
|
|
|
|
2007-12-02 06:09:22 +00:00
|
|
|
static void execv_dashed_external(const char **argv)
|
|
|
|
{
|
2017-01-07 01:16:24 +00:00
|
|
|
struct child_process cmd = CHILD_PROCESS_INIT;
|
2009-01-28 07:38:14 +00:00
|
|
|
int status;
|
2007-12-02 06:09:22 +00:00
|
|
|
|
git.c: ignore pager.* when launching builtin as dashed external
When running, e.g., `git -c alias.bar=foo bar`, we expand the alias and
execute `git-foo` as a dashed external. This is true even if git foo is
a builtin. That is on purpose, and is motivated in a comment which was
added in commit 441981bc ("git: simplify environment save/restore
logic", 2016-01-26).
Shortly before we launch a dashed external, and unless we have already
found out whether we should use a pager, we check `pager.foo`. This was
added in commit 92058e4d ("support pager.* for external commands",
2011-08-18). If the dashed external is a builtin, this does not match
that commit's intention and is arguably wrong, since it would be cleaner
if we let the "dashed external builtin" handle `pager.foo`.
This has not mattered in practice, but a recent patch taught `git-tag`
to ignore `pager.tag` under certain circumstances. But, when started
using an alias, it doesn't get the chance to do so, as outlined above.
That recent patch added a test to document this breakage.
Do not check `pager.foo` before launching a builtin as a dashed
external, i.e., if we recognize the name of the external as a builtin.
Change the test to use `test_expect_success`.
Signed-off-by: Martin Ågren <martin.agren@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2017-08-02 19:40:55 +00:00
|
|
|
if (use_pager == -1 && !is_builtin(argv[0]))
|
2011-08-18 22:01:32 +00:00
|
|
|
use_pager = check_pager_config(argv[0]);
|
2010-07-14 22:55:12 +00:00
|
|
|
commit_pager_choice();
|
|
|
|
|
2020-07-28 20:24:53 +00:00
|
|
|
strvec_pushf(&cmd.args, "git-%s", argv[0]);
|
|
|
|
strvec_pushv(&cmd.args, argv + 1);
|
2017-01-07 01:16:24 +00:00
|
|
|
cmd.clean_on_exit = 1;
|
execv_dashed_external: wait for child on signal death
When you hit ^C to interrupt a git command going to a pager,
this usually leaves the pager running. But when a dashed
external is in use, the pager ends up in a funny state and
quits (but only after eating one more character from the
terminal!). This fixes it.
Explaining the reason will require a little background.
When git runs a pager, it's important for the git process to
hang around and wait for the pager to finish, even though it
has no more data to feed it. This is because git spawns the
pager as a child, and thus the git process is the session
leader on the terminal. After it dies, the pager will finish
its current read from the terminal (eating the one
character), and then get EIO trying to read again.
When you hit ^C, that sends SIGINT to git and to the pager,
and it's a similar situation. The pager ignores it, but the
git process needs to hang around until the pager is done. We
addressed that long ago in a3da882120 (pager: do
wait_for_pager on signal death, 2009-01-22).
But when you have a dashed external (or an alias pointing to
a builtin, which will re-exec git for the builtin), there's
an extra process in the mix. For instance, running:
$ git -c alias.l=log l
will end up with a process tree like:
git (parent)
\
git-log (child)
\
less (pager)
If you hit ^C, SIGINT goes to all of them. The pager ignores
it, and the child git process will end up in wait_for_pager().
But the parent git process will die, and the usual EIO
trouble happens.
So we really want the parent git process to wait_for_pager(),
but of course it doesn't know anything about the pager at
all, since it was started by the child. However, we can
have it wait on the git-log child, which in turn is waiting
on the pager. And that's what this patch does.
There are a few design decisions here worth explaining:
1. The new feature is attached to run-command's
clean_on_exit feature. Partly this is convenience,
since that feature already has a signal handler that
deals with child cleanup.
But it's also a meaningful connection. The main reason
that dashed externals use clean_on_exit is to bind the
two processes together. If somebody kills the parent
with a signal, we propagate that to the child (in this
instance with SIGINT, we do propagate but it doesn't
matter because the original signal went to the whole
process group). Likewise, we do not want the parent
to go away until the child has done so.
In a traditional Unix world, we'd probably accomplish
this binding by just having the parent execve() the
child directly. But since that doesn't work on Windows,
everything goes through run_command's more spawn-like
interface.
2. We do _not_ automatically waitpid() on any
clean_on_exit children. For dashed externals this makes
sense; we know that the parent is doing nothing but
waiting for the child to exit anyway. But with other
children, it's possible that the child, after getting
the signal, could be waiting on the parent to do
something (like closing a descriptor). If we were to
wait on such a child, we'd end up in a deadlock. So
this errs on the side of caution, and lets callers
enable the feature explicitly.
3. When we send children the cleanup signal, we send all
the signals first, before waiting on any children. This
is to avoid the case where one child might be waiting
on another one to exit, causing a deadlock. We inform
all of them that it's time to die before reaping any.
In practice, there is only ever one dashed external run
from a given process, so this doesn't matter much now.
But it future-proofs us if other callers start using
the wait_after_clean mechanism.
There's no automated test here, because it would end up racy
and unportable. But it's easy to reproduce the situation by
running the log command given above and hitting ^C.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2017-01-07 01:22:23 +00:00
|
|
|
cmd.wait_after_clean = 1;
|
2017-01-07 01:16:24 +00:00
|
|
|
cmd.silent_exec_failure = 1;
|
2019-02-22 22:25:01 +00:00
|
|
|
cmd.trace2_child_class = "dashed";
|
2007-12-02 06:09:22 +00:00
|
|
|
|
2019-02-22 22:25:01 +00:00
|
|
|
trace2_cmd_name("_run_dashed_");
|
|
|
|
|
|
|
|
/*
|
|
|
|
* The code in run_command() logs trace2 child_start/child_exit
|
|
|
|
* events, so we do not need to report exec/exec_result events here.
|
|
|
|
*/
|
2020-07-29 00:37:20 +00:00
|
|
|
trace_argv_printf(cmd.args.v, "trace: exec:");
|
2007-12-02 06:09:22 +00:00
|
|
|
|
2009-01-28 07:38:14 +00:00
|
|
|
/*
|
2017-01-07 01:17:48 +00:00
|
|
|
* If we fail because the command is not found, it is
|
|
|
|
* OK to return. Otherwise, we just pass along the status code,
|
|
|
|
* or our usual generic code if we were not even able to exec
|
|
|
|
* the program.
|
2009-01-28 07:38:14 +00:00
|
|
|
*/
|
2017-01-07 01:16:24 +00:00
|
|
|
status = run_command(&cmd);
|
2019-02-22 22:25:01 +00:00
|
|
|
|
|
|
|
/*
|
|
|
|
* If the child process ran and we are now going to exit, emit a
|
|
|
|
* generic string as our trace2 command verb to indicate that we
|
|
|
|
* launched a dashed command.
|
|
|
|
*/
|
2017-01-07 01:17:48 +00:00
|
|
|
if (status >= 0)
|
2009-07-04 19:26:39 +00:00
|
|
|
exit(status);
|
2017-01-07 01:17:48 +00:00
|
|
|
else if (errno != ENOENT)
|
|
|
|
exit(128);
|
2007-12-02 06:09:22 +00:00
|
|
|
}
|
|
|
|
|
2009-01-04 17:16:01 +00:00
|
|
|
static int run_argv(int *argcp, const char ***argv)
|
|
|
|
{
|
|
|
|
int done_alias = 0;
|
2018-09-16 07:50:00 +00:00
|
|
|
struct string_list cmd_list = STRING_LIST_INIT_NODUP;
|
2018-09-16 07:50:01 +00:00
|
|
|
struct string_list_item *seen;
|
2009-01-04 17:16:01 +00:00
|
|
|
|
|
|
|
while (1) {
|
git: simplify environment save/restore logic
The only code that cares about the value of the global variable
saved_env_before_alias after the previous fix is handle_builtin()
that turns into a glorified no-op when the variable is true, so the
logic could safely be lifted to its caller, i.e. the caller can
refrain from calling it when the variable is set.
This variable tells us if save_env_before_alias() was called (with
or without matching restore_env()), but the sole caller of the
function, handle_alias(), always calls it as the first thing, so we
can consider that the variable essentially keeps track of the fact
that handle_alias() has ever been called.
It turns out that handle_builtin() and handle_alias() are called
only from one function in a way that the value of the variable
matters, which is run_argv(), and it already keeps track of the
fact that it already called handle_alias().
So we can simplify the whole thing by:
- Change handle_builtin() to always make a direct call to the
builtin implementation it finds, and make sure the caller
refrains from calling it if handle_alias() has ever been
called;
- Remove saved_env_before_alias variable, and instead use the
local "done_alias" variable maintained inside run_argv() to
make the same decision.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2016-01-27 06:52:02 +00:00
|
|
|
/*
|
|
|
|
* If we tried alias and futzed with our environment,
|
|
|
|
* it no longer is safe to invoke builtins directly in
|
|
|
|
* general. We have to spawn them as dashed externals.
|
|
|
|
*
|
|
|
|
* NEEDSWORK: if we can figure out cases
|
|
|
|
* where it is safe to do, we can avoid spawning a new
|
|
|
|
* process.
|
|
|
|
*/
|
|
|
|
if (!done_alias)
|
|
|
|
handle_builtin(*argcp, *argv);
|
2019-02-22 22:25:01 +00:00
|
|
|
else if (get_builtin(**argv)) {
|
2022-10-30 11:51:14 +00:00
|
|
|
struct child_process cmd = CHILD_PROCESS_INIT;
|
2019-02-22 22:25:01 +00:00
|
|
|
int i;
|
|
|
|
|
|
|
|
/*
|
|
|
|
* The current process is committed to launching a
|
|
|
|
* child process to run the command named in (**argv)
|
|
|
|
* and exiting. Log a generic string as the trace2
|
|
|
|
* command verb to indicate this. Note that the child
|
|
|
|
* process will log the actual verb when it runs.
|
|
|
|
*/
|
|
|
|
trace2_cmd_name("_run_git_alias_");
|
|
|
|
|
|
|
|
commit_pager_choice();
|
|
|
|
|
2022-10-30 11:51:14 +00:00
|
|
|
strvec_push(&cmd.args, "git");
|
2019-02-22 22:25:01 +00:00
|
|
|
for (i = 0; i < *argcp; i++)
|
2022-10-30 11:51:14 +00:00
|
|
|
strvec_push(&cmd.args, (*argv)[i]);
|
2019-02-22 22:25:01 +00:00
|
|
|
|
2022-10-30 11:51:14 +00:00
|
|
|
trace_argv_printf(cmd.args.v, "trace: exec:");
|
2019-02-22 22:25:01 +00:00
|
|
|
|
|
|
|
/*
|
|
|
|
* if we fail because the command is not found, it is
|
|
|
|
* OK to return. Otherwise, we just pass along the status code.
|
|
|
|
*/
|
2022-10-30 11:51:14 +00:00
|
|
|
cmd.silent_exec_failure = 1;
|
|
|
|
cmd.clean_on_exit = 1;
|
|
|
|
cmd.wait_after_clean = 1;
|
|
|
|
cmd.trace2_child_class = "git_alias";
|
|
|
|
i = run_command(&cmd);
|
2019-02-22 22:25:01 +00:00
|
|
|
if (i >= 0 || errno != ENOENT)
|
|
|
|
exit(i);
|
|
|
|
die("could not execute builtin %s", **argv);
|
|
|
|
}
|
|
|
|
|
2009-01-04 17:16:01 +00:00
|
|
|
/* .. then try the external ones */
|
|
|
|
execv_dashed_external(*argv);
|
|
|
|
|
2018-09-16 07:50:01 +00:00
|
|
|
seen = unsorted_string_list_lookup(&cmd_list, *argv[0]);
|
|
|
|
if (seen) {
|
|
|
|
int i;
|
|
|
|
struct strbuf sb = STRBUF_INIT;
|
|
|
|
for (i = 0; i < cmd_list.nr; i++) {
|
|
|
|
struct string_list_item *item = &cmd_list.items[i];
|
|
|
|
|
|
|
|
strbuf_addf(&sb, "\n %s", item->string);
|
|
|
|
if (item == seen)
|
|
|
|
strbuf_addstr(&sb, " <==");
|
|
|
|
else if (i == cmd_list.nr - 1)
|
|
|
|
strbuf_addstr(&sb, " ==>");
|
|
|
|
}
|
2018-09-16 07:50:00 +00:00
|
|
|
die(_("alias loop detected: expansion of '%s' does"
|
2018-09-16 07:50:01 +00:00
|
|
|
" not terminate:%s"), cmd_list.items[0].string, sb.buf);
|
2018-09-16 07:50:00 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
string_list_append(&cmd_list, *argv[0]);
|
|
|
|
|
|
|
|
/*
|
|
|
|
* It could be an alias -- this works around the insanity
|
2009-01-04 17:16:01 +00:00
|
|
|
* of overriding "git log" with "git show" by having
|
|
|
|
* alias.log = show
|
|
|
|
*/
|
2014-06-08 09:37:10 +00:00
|
|
|
if (!handle_alias(argcp, argv))
|
2009-01-04 17:16:01 +00:00
|
|
|
break;
|
|
|
|
done_alias = 1;
|
|
|
|
}
|
|
|
|
|
2018-09-16 07:50:00 +00:00
|
|
|
string_list_clear(&cmd_list, 0);
|
|
|
|
|
2009-01-04 17:16:01 +00:00
|
|
|
return done_alias;
|
|
|
|
}
|
|
|
|
|
add an extra level of indirection to main()
There are certain startup tasks that we expect every git
process to do. In some cases this is just to improve the
quality of the program (e.g., setting up gettext()). In
others it is a requirement for using certain functions in
libgit.a (e.g., system_path() expects that you have called
git_extract_argv0_path()).
Most commands are builtins and are covered by the git.c
version of main(). However, there are still a few external
commands that use their own main(). Each of these has to
remember to include the correct startup sequence, and we are
not always consistent.
Rather than just fix the inconsistencies, let's make this
harder to get wrong by providing a common main() that can
run this standard startup.
We basically have two options to do this:
- the compat/mingw.h file already does something like this by
adding a #define that replaces the definition of main with a
wrapper that calls mingw_startup().
The upside is that the code in each program doesn't need
to be changed at all; it's rewritten on the fly by the
preprocessor.
The downside is that it may make debugging of the startup
sequence a bit more confusing, as the preprocessor is
quietly inserting new code.
- the builtin functions are all of the form cmd_foo(),
and git.c's main() calls them.
This is much more explicit, which may make things more
obvious to somebody reading the code. It's also more
flexible (because of course we have to figure out _which_
cmd_foo() to call).
The downside is that each of the builtins must define
cmd_foo(), instead of just main().
This patch chooses the latter option, preferring the more
explicit approach, even though it is more invasive. We
introduce a new file common-main.c, with the "real" main. It
expects to call cmd_main() from whatever other objects it is
linked against.
We link common-main.o against anything that links against
libgit.a, since we know that such programs will need to do
this setup. Note that common-main.o can't actually go inside
libgit.a, as the linker would not pick up its main()
function automatically (it has no callers).
The rest of the patch is just adjusting all of the various
external programs (mostly in t/helper) to use cmd_main().
I've provided a global declaration for cmd_main(), which
means that all of the programs also need to match its
signature. In particular, many functions need to switch to
"const char **" instead of "char **" for argv. This effect
ripples out to a few other variables and functions, as well.
This makes the patch even more invasive, but the end result
is much better. We should be treating argv strings as const
anyway, and now all programs conform to the same signature
(which also matches the way builtins are defined).
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2016-07-01 05:58:58 +00:00
|
|
|
int cmd_main(int argc, const char **argv)
|
2005-11-15 23:31:25 +00:00
|
|
|
{
|
2009-01-18 12:00:10 +00:00
|
|
|
const char *cmd;
|
2015-03-02 12:02:37 +00:00
|
|
|
int done_help = 0;
|
Teach the "git" command to handle some commands internally
This is another patch in the "prepare to do more in C" series, where the
git wrapper command is taught about the notion of handling some
functionality internally.
Right now, the only internal commands are "version" and "help", but the
point being that we can now easily extend it to handle some of the trivial
scripts internally. Things like "git log" and "git diff" wouldn't need
separate external scripts any more.
This also implies that to support the old "git-log" and "git-diff" syntax,
the "git" wrapper now automatically looks at the name it was executed as,
and if it is "git-xxxx", it will assume that it is to internally do what
"git xxxx" would do.
In other words, you can (once you implement an internal command) soft- or
hard-link that command to the "git" wrapper command, and it will do the
right thing, whether you use the "git xxxx" or the "git-xxxx" format.
There's one other change: the search order for external programs is
modified slightly, so that the first entry remains GIT_EXEC_DIR, but the
second entry is the same directory as the git wrapper itself was executed
out of - if we can figure it out from argv[0], of course.
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
2006-02-26 20:34:51 +00:00
|
|
|
|
common-main: call git_extract_argv0_path()
Every program which links against libgit.a must call this
function, or risk hitting an assert() in system_path() that
checks whether we have configured argv0_path (though only
when RUNTIME_PREFIX is defined, so essentially only on
Windows).
Looking at the diff, you can see that putting it into the
common main() saves us having to do it individually in each
of the external commands. But what you can't see are the
cases where we _should_ have been doing so, but weren't
(e.g., git-credential-store, and all of the t/helper test
programs).
This has been an accident-waiting-to-happen for a long time,
but wasn't triggered until recently because it involves one
of those programs actually calling system_path(). That
happened with git-credential-store in v2.8.0 with ae5f677
(lazily load core.sharedrepository, 2016-03-11). The
program:
- takes a lock file, which...
- opens a tempfile, which...
- calls adjust_shared_perm to fix permissions, which...
- lazy-loads the config (as of ae5f677), which...
- calls system_path() to find the location of
/etc/gitconfig
On systems with RUNTIME_PREFIX, this means credential-store
reliably hits that assert() and cannot be used.
We never noticed in the test suite, because we set
GIT_CONFIG_NOSYSTEM there, which skips the system_path()
lookup entirely. But if we were to tweak git_config() to
find /etc/gitconfig even when we aren't going to open it,
then the test suite shows multiple failures (for
credential-store, and for some other test helpers). I didn't
include that tweak here because it's way too specific to
this particular call to be worth carrying around what is
essentially dead code.
The implementation is fairly straightforward, with one
exception: there is exactly one caller (git.c) that actually
cares about the result of the function, and not the
side-effect of setting up argv0_path. We can accommodate
that by simply replacing the value of argv[0] in the array
we hand down to cmd_main().
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2016-07-01 06:04:04 +00:00
|
|
|
cmd = argv[0];
|
2009-01-18 12:00:11 +00:00
|
|
|
if (!cmd)
|
2009-01-18 12:00:10 +00:00
|
|
|
cmd = "git-help";
|
common-main: stop munging argv[0] path
Since 650c44925 (common-main: call git_extract_argv0_path(),
2016-07-01), the argv[0] that is seen in cmd_main() of
individual programs is always the basename of the
executable, as common-main strips off the full path. This
can produce confusing results for git-daemon, which wants to
re-exec itself.
For instance, if the program was originally run as
"/usr/lib/git/git-daemon", it will try just re-execing
"git-daemon", which will find the first instance in $PATH.
If git's exec-path has not been prepended to $PATH, we may
find the git-daemon from a different version (or no
git-daemon at all).
Normally this isn't a problem. Git commands are run as "git
daemon", the git wrapper puts the exec-path at the front of
$PATH, and argv[0] is already "daemon" anyway. But running
git-daemon via its full exec-path, while not really a
recommended method, did work prior to 650c44925. Let's make
it work again.
The real goal of 650c44925 was not to munge argv[0], but to
reliably set the argv0_path global. The only reason it
munges at all is that one caller, the git.c wrapper,
piggy-backed on that computation to find the command
basename. Instead, let's leave argv[0] untouched in
common-main, and have git.c do its own basename computation.
While we're at it, let's drop the return value from
git_extract_argv0_path(). It was only ever used in this one
callsite, and its dual purposes is what led to this
confusion in the first place.
Note that by changing the interface, the compiler can
confirm for us that there are no other callers storing the
return value. But the compiler can't tell us whether any of
the cmd_main() functions (besides git.c) were relying on the
basename munging. However, we can observe that prior to
650c44925, no other cmd_main() functions did that munging,
and no new cmd_main() functions have been introduced since
then. So we can't be regressing any of those cases.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2016-11-27 04:31:13 +00:00
|
|
|
else {
|
|
|
|
const char *slash = find_last_dir_sep(cmd);
|
|
|
|
if (slash)
|
|
|
|
cmd = slash + 1;
|
|
|
|
}
|
2005-11-15 23:31:25 +00:00
|
|
|
|
2014-07-12 00:07:01 +00:00
|
|
|
trace_command_performance(argv);
|
|
|
|
|
Teach the "git" command to handle some commands internally
This is another patch in the "prepare to do more in C" series, where the
git wrapper command is taught about the notion of handling some
functionality internally.
Right now, the only internal commands are "version" and "help", but the
point being that we can now easily extend it to handle some of the trivial
scripts internally. Things like "git log" and "git diff" wouldn't need
separate external scripts any more.
This also implies that to support the old "git-log" and "git-diff" syntax,
the "git" wrapper now automatically looks at the name it was executed as,
and if it is "git-xxxx", it will assume that it is to internally do what
"git xxxx" would do.
In other words, you can (once you implement an internal command) soft- or
hard-link that command to the "git" wrapper command, and it will do the
right thing, whether you use the "git xxxx" or the "git-xxxx" format.
There's one other change: the search order for external programs is
modified slightly, so that the first entry remains GIT_EXEC_DIR, but the
second entry is the same directory as the git wrapper itself was executed
out of - if we can figure it out from argv[0], of course.
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
2006-02-26 20:34:51 +00:00
|
|
|
/*
|
|
|
|
* "git-xxxx" is the same as "git xxxx", but we obviously:
|
|
|
|
*
|
|
|
|
* - cannot take flags in between the "git" and the "xxxx".
|
|
|
|
* - cannot execute it externally (since it would just do
|
|
|
|
* the same thing over again)
|
|
|
|
*
|
2014-01-02 16:15:44 +00:00
|
|
|
* So we just directly call the builtin handler, and die if
|
|
|
|
* that one cannot handle it.
|
Teach the "git" command to handle some commands internally
This is another patch in the "prepare to do more in C" series, where the
git wrapper command is taught about the notion of handling some
functionality internally.
Right now, the only internal commands are "version" and "help", but the
point being that we can now easily extend it to handle some of the trivial
scripts internally. Things like "git log" and "git diff" wouldn't need
separate external scripts any more.
This also implies that to support the old "git-log" and "git-diff" syntax,
the "git" wrapper now automatically looks at the name it was executed as,
and if it is "git-xxxx", it will assume that it is to internally do what
"git xxxx" would do.
In other words, you can (once you implement an internal command) soft- or
hard-link that command to the "git" wrapper command, and it will do the
right thing, whether you use the "git xxxx" or the "git-xxxx" format.
There's one other change: the search order for external programs is
modified slightly, so that the first entry remains GIT_EXEC_DIR, but the
second entry is the same directory as the git wrapper itself was executed
out of - if we can figure it out from argv[0], of course.
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
2006-02-26 20:34:51 +00:00
|
|
|
*/
|
use skip_prefix to avoid magic numbers
It's a common idiom to match a prefix and then skip past it
with a magic number, like:
if (starts_with(foo, "bar"))
foo += 3;
This is easy to get wrong, since you have to count the
prefix string yourself, and there's no compiler check if the
string changes. We can use skip_prefix to avoid the magic
numbers here.
Note that some of these conversions could be much shorter.
For example:
if (starts_with(arg, "--foo=")) {
bar = arg + 6;
continue;
}
could become:
if (skip_prefix(arg, "--foo=", &bar))
continue;
However, I have left it as:
if (skip_prefix(arg, "--foo=", &v)) {
bar = v;
continue;
}
to visually match nearby cases which need to actually
process the string. Like:
if (skip_prefix(arg, "--foo=", &v)) {
bar = atoi(v);
continue;
}
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2014-06-18 19:47:50 +00:00
|
|
|
if (skip_prefix(cmd, "git-", &cmd)) {
|
Teach the "git" command to handle some commands internally
This is another patch in the "prepare to do more in C" series, where the
git wrapper command is taught about the notion of handling some
functionality internally.
Right now, the only internal commands are "version" and "help", but the
point being that we can now easily extend it to handle some of the trivial
scripts internally. Things like "git log" and "git diff" wouldn't need
separate external scripts any more.
This also implies that to support the old "git-log" and "git-diff" syntax,
the "git" wrapper now automatically looks at the name it was executed as,
and if it is "git-xxxx", it will assume that it is to internally do what
"git xxxx" would do.
In other words, you can (once you implement an internal command) soft- or
hard-link that command to the "git" wrapper command, and it will do the
right thing, whether you use the "git xxxx" or the "git-xxxx" format.
There's one other change: the search order for external programs is
modified slightly, so that the first entry remains GIT_EXEC_DIR, but the
second entry is the same directory as the git wrapper itself was executed
out of - if we can figure it out from argv[0], of course.
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
2006-02-26 20:34:51 +00:00
|
|
|
argv[0] = cmd;
|
2014-01-02 16:15:44 +00:00
|
|
|
handle_builtin(argc, argv);
|
2018-11-10 05:16:00 +00:00
|
|
|
die(_("cannot handle %s as a builtin"), cmd);
|
Teach the "git" command to handle some commands internally
This is another patch in the "prepare to do more in C" series, where the
git wrapper command is taught about the notion of handling some
functionality internally.
Right now, the only internal commands are "version" and "help", but the
point being that we can now easily extend it to handle some of the trivial
scripts internally. Things like "git log" and "git diff" wouldn't need
separate external scripts any more.
This also implies that to support the old "git-log" and "git-diff" syntax,
the "git" wrapper now automatically looks at the name it was executed as,
and if it is "git-xxxx", it will assume that it is to internally do what
"git xxxx" would do.
In other words, you can (once you implement an internal command) soft- or
hard-link that command to the "git" wrapper command, and it will do the
right thing, whether you use the "git xxxx" or the "git-xxxx" format.
There's one other change: the search order for external programs is
modified slightly, so that the first entry remains GIT_EXEC_DIR, but the
second entry is the same directory as the git wrapper itself was executed
out of - if we can figure it out from argv[0], of course.
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
2006-02-26 20:34:51 +00:00
|
|
|
}
|
2005-11-15 23:31:25 +00:00
|
|
|
|
Teach the "git" command to handle some commands internally
This is another patch in the "prepare to do more in C" series, where the
git wrapper command is taught about the notion of handling some
functionality internally.
Right now, the only internal commands are "version" and "help", but the
point being that we can now easily extend it to handle some of the trivial
scripts internally. Things like "git log" and "git diff" wouldn't need
separate external scripts any more.
This also implies that to support the old "git-log" and "git-diff" syntax,
the "git" wrapper now automatically looks at the name it was executed as,
and if it is "git-xxxx", it will assume that it is to internally do what
"git xxxx" would do.
In other words, you can (once you implement an internal command) soft- or
hard-link that command to the "git" wrapper command, and it will do the
right thing, whether you use the "git xxxx" or the "git-xxxx" format.
There's one other change: the search order for external programs is
modified slightly, so that the first entry remains GIT_EXEC_DIR, but the
second entry is the same directory as the git wrapper itself was executed
out of - if we can figure it out from argv[0], of course.
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
2006-02-26 20:34:51 +00:00
|
|
|
/* Look for flags.. */
|
2006-07-25 18:24:22 +00:00
|
|
|
argv++;
|
|
|
|
argc--;
|
2007-06-08 20:57:55 +00:00
|
|
|
handle_options(&argv, &argc, NULL);
|
2022-10-08 16:21:37 +00:00
|
|
|
|
|
|
|
if (!argc) {
|
2007-10-27 08:36:49 +00:00
|
|
|
/* The user didn't specify a command; give them help */
|
git --paginate: do not commit pager choice too early
When git is passed the --paginate option, starting up a pager requires
deciding what pager to start, which requires access to the core.pager
configuration.
At the relevant moment, the repository has not been searched for yet.
Attempting to access the configuration at this point results in
git_dir being set to .git [*], which is almost certainly not what was
wanted. In particular, when run from a subdirectory of the toplevel,
git --paginate does not respect the core.pager setting from the
current repository.
[*] unless GIT_DIR or GIT_CONFIG is set
So delay the pager startup when possible:
1. run_argv() already commits pager choice inside run_builtin() if a
command is found. For commands that use RUN_SETUP, waiting until
then fixes the problem described above: once git knows where to
look, it happily respects the core.pager setting.
2. list_common_cmds_help() prints out 29 lines and exits. This can
benefit from pagination, so we need to commit the pager choice
before writing this output.
Luckily ‘git’ without subcommand has no other reason to access a
repository, so it would be intuitive to ignore repository-local
configuration in this case. Simpler for now to choose a pager
using the funny code that notices a repository that happens to be
at .git. That this accesses a repository when it is very
convenient to is a bug but not an important one.
3. help_unknown_cmd() prints out a few lines to stderr. It is not
important to paginate this, so don’t.
Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com>
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
Acked-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2010-06-26 19:26:37 +00:00
|
|
|
commit_pager_choice();
|
2018-11-10 05:16:00 +00:00
|
|
|
printf(_("usage: %s\n\n"), git_usage_string);
|
2007-10-27 08:36:49 +00:00
|
|
|
list_common_cmds_help();
|
2013-03-10 15:10:20 +00:00
|
|
|
printf("\n%s\n", _(git_more_info_string));
|
2007-10-27 08:36:49 +00:00
|
|
|
exit(1);
|
2005-11-15 23:31:25 +00:00
|
|
|
}
|
2022-10-08 16:21:37 +00:00
|
|
|
|
|
|
|
if (!strcmp("--version", argv[0]) || !strcmp("-v", argv[0]))
|
|
|
|
argv[0] = "version";
|
|
|
|
else if (!strcmp("--help", argv[0]) || !strcmp("-h", argv[0]))
|
|
|
|
argv[0] = "help";
|
|
|
|
|
2006-07-25 18:24:22 +00:00
|
|
|
cmd = argv[0];
|
Teach the "git" command to handle some commands internally
This is another patch in the "prepare to do more in C" series, where the
git wrapper command is taught about the notion of handling some
functionality internally.
Right now, the only internal commands are "version" and "help", but the
point being that we can now easily extend it to handle some of the trivial
scripts internally. Things like "git log" and "git diff" wouldn't need
separate external scripts any more.
This also implies that to support the old "git-log" and "git-diff" syntax,
the "git" wrapper now automatically looks at the name it was executed as,
and if it is "git-xxxx", it will assume that it is to internally do what
"git xxxx" would do.
In other words, you can (once you implement an internal command) soft- or
hard-link that command to the "git" wrapper command, and it will do the
right thing, whether you use the "git xxxx" or the "git-xxxx" format.
There's one other change: the search order for external programs is
modified slightly, so that the first entry remains GIT_EXEC_DIR, but the
second entry is the same directory as the git wrapper itself was executed
out of - if we can figure it out from argv[0], of course.
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
2006-02-26 20:34:51 +00:00
|
|
|
|
|
|
|
/*
|
2007-10-28 11:17:20 +00:00
|
|
|
* We use PATH to find git commands, but we prepend some higher
|
2009-04-17 18:13:30 +00:00
|
|
|
* precedence paths: the "--exec-path" option, the GIT_EXEC_PATH
|
2007-10-28 11:17:20 +00:00
|
|
|
* environment, and the $(gitexecdir) from the Makefile at build
|
|
|
|
* time.
|
Teach the "git" command to handle some commands internally
This is another patch in the "prepare to do more in C" series, where the
git wrapper command is taught about the notion of handling some
functionality internally.
Right now, the only internal commands are "version" and "help", but the
point being that we can now easily extend it to handle some of the trivial
scripts internally. Things like "git log" and "git diff" wouldn't need
separate external scripts any more.
This also implies that to support the old "git-log" and "git-diff" syntax,
the "git" wrapper now automatically looks at the name it was executed as,
and if it is "git-xxxx", it will assume that it is to internally do what
"git xxxx" would do.
In other words, you can (once you implement an internal command) soft- or
hard-link that command to the "git" wrapper command, and it will do the
right thing, whether you use the "git xxxx" or the "git-xxxx" format.
There's one other change: the search order for external programs is
modified slightly, so that the first entry remains GIT_EXEC_DIR, but the
second entry is the same directory as the git wrapper itself was executed
out of - if we can figure it out from argv[0], of course.
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
2006-02-26 20:34:51 +00:00
|
|
|
*/
|
2008-07-21 19:19:52 +00:00
|
|
|
setup_path();
|
2005-11-15 23:31:25 +00:00
|
|
|
|
2006-06-06 01:09:40 +00:00
|
|
|
while (1) {
|
2015-03-02 12:02:37 +00:00
|
|
|
int was_alias = run_argv(&argc, &argv);
|
2009-01-04 17:16:01 +00:00
|
|
|
if (errno != ENOENT)
|
2006-06-06 01:09:40 +00:00
|
|
|
break;
|
2009-01-04 17:16:01 +00:00
|
|
|
if (was_alias) {
|
2018-02-13 13:19:15 +00:00
|
|
|
fprintf(stderr, _("expansion of alias '%s' failed; "
|
|
|
|
"'%s' is not a git command\n"),
|
2007-02-11 00:33:57 +00:00
|
|
|
cmd, argv[0]);
|
|
|
|
exit(1);
|
|
|
|
}
|
2009-01-04 17:16:01 +00:00
|
|
|
if (!done_help) {
|
|
|
|
cmd = argv[0] = help_unknown_cmd(cmd);
|
|
|
|
done_help = 1;
|
|
|
|
} else
|
|
|
|
break;
|
2007-02-11 00:33:57 +00:00
|
|
|
}
|
2005-12-01 12:48:35 +00:00
|
|
|
|
2018-02-13 13:19:15 +00:00
|
|
|
fprintf(stderr, _("failed to run command '%s': %s\n"),
|
2006-06-28 09:17:21 +00:00
|
|
|
cmd, strerror(errno));
|
2005-11-15 23:31:25 +00:00
|
|
|
|
|
|
|
return 1;
|
|
|
|
}
|