From 227e2c0ecd782beb1bf6cb613a02060dfe08bd52 Mon Sep 17 00:00:00 2001 From: Jonas Fonseca Date: Thu, 27 Nov 2008 17:45:54 +0100 Subject: [PATCH] IO API: use for the main, log, diff, tree and blob views This introduces an incompatibility with previous versions. --- NEWS | 7 ++++ manual.txt | 27 ++++++------- tig.1.txt | 4 +- tig.c | 111 ++++++++++++++++++++++++++++++++++------------------- 4 files changed, 93 insertions(+), 56 deletions(-) diff --git a/NEWS b/NEWS index 3532b5d..0463230 100644 --- a/NEWS +++ b/NEWS @@ -4,6 +4,13 @@ Release notes tig master ---------- +Incompatibilities: + + - Commands from the environment (e.g. TIG_MAIN_CMD) will no longer have + access to shell features, such as subshells and variable expansion. + The easiest way to upgrade your configuration is to put such commands + in a script file. + Improvements: - Display repository references in the sorted order: tags, heads, diff --git a/manual.txt b/manual.txt index 8ac7cd2..5a295e0 100644 --- a/manual.txt +++ b/manual.txt @@ -137,8 +137,9 @@ reloaded, if the commit ID changed. The head ID is used when opening the main and log view to indicate from what revision to show history. Some of the commands used or provided by tig can be configured. This goes for -the <>. These user-defined commands can -use arguments that refer to the current browsing state by using one of the +some of the <> as well as the +<>. These user-defined commands can use +arguments that refer to the current browsing state by using one of the following variables. `-----------------------`----------------------------------------------------- @@ -237,13 +238,12 @@ for example you prefer commits in the main view to be sorted by date and only show 500 commits, use: ----------------------------------------------------------------------------- -$ TIG_MAIN_CMD="git log --date-order -n500 --pretty=raw %s" tig +$ TIG_MAIN_CMD="git log --date-order -n500 --pretty=raw %(head)" tig ----------------------------------------------------------------------------- Or set the variable permanently in your environment. -Notice, how `%s` is used to specify the commit reference. There can be a -maximum of 5 `%s` ref specifications. +Notice, how `%(head)` is used to specify the commit reference. -- @@ -252,7 +252,7 @@ TIG_DIFF_CMD:: The command used for the diff view. Defaults to: ----------------------------------------------------------------------------- git show --pretty=fuller --no-color --root - --patch-with-stat --find-copies-harder -C %s + --patch-with-stat --find-copies-harder -C %(commit) ----------------------------------------------------------------------------- TIG_LOG_CMD:: @@ -261,7 +261,7 @@ TIG_LOG_CMD:: author and committer shown in the log view be sure to pass `\--pretty=fuller` to git log. Defaults to: ----------------------------------------------------------------------------- -git log --no-color --cc --stat -n100 %s +git log --no-color --cc --stat -n100 %(head) ----------------------------------------------------------------------------- TIG_MAIN_CMD:: @@ -270,7 +270,7 @@ TIG_MAIN_CMD:: the option: `\--pretty=raw` since the main view parser expects to read that format. ----------------------------------------------------------------------------- -git log --no-color --pretty=raw --parents --topo-order %s +git log --no-color --pretty=raw --parents --topo-order %(head) ----------------------------------------------------------------------------- -- @@ -283,19 +283,16 @@ Tree Commands TIG_TREE_CMD:: - The command used for the tree view. Takes two arguments, the first is - the revision ID and the second is the path of the directory tree, - empty for the root directory. Defaults to: + The command used for the tree view. Defaults to: ----------------------------------------------------------------------------- -git ls-tree %s %s +git ls-tree %(commit) %(directory) ----------------------------------------------------------------------------- TIG_BLOB_CMD:: - The command used for the blob view. Takes one argument which is the - blob ID. Defaults to: + The command used for the blob view. Defaults to: ----------------------------------------------------------------------------- -git cat-file blob %s +git cat-file blob %(blob) ----------------------------------------------------------------------------- -- diff --git a/tig.1.txt b/tig.1.txt index 6a7e266..3a914cc 100644 --- a/tig.1.txt +++ b/tig.1.txt @@ -94,7 +94,9 @@ ENVIRONMENT VARIABLES --------------------- In addition to environment variables used by git (e.g. GIT_DIR), tig defines -the following: +the ones below. The command related environment variables have access to the +internal state of tig via replacement variabls, such as `%(commit)` and +`%(blob)`. See manpage:tigrc[5] for a full list. TIGRC_USER:: Path of the user configuration file (defaults to `~/.tigrc`). diff --git a/tig.c b/tig.c index 6665d61..858f930 100644 --- a/tig.c +++ b/tig.c @@ -122,31 +122,9 @@ static int load_refs(void); #define TIG_LS_REMOTE \ "git ls-remote . 2>/dev/null" -#define TIG_DIFF_CMD \ - "git show --pretty=fuller --no-color --root --patch-with-stat --find-copies-harder -C %s 2>/dev/null" - -#define TIG_LOG_CMD \ - "git log --no-color --cc --stat -n100 %s 2>/dev/null" - #define TIG_MAIN_BASE \ "git log --no-color --pretty=raw --parents --topo-order" -#define TIG_MAIN_CMD \ - TIG_MAIN_BASE " %s 2>/dev/null" - -#define TIG_TREE_CMD \ - "git ls-tree %s %s" - -#define TIG_BLOB_CMD \ - "git cat-file blob %s" - -/* XXX: Needs to be defined to the empty string. */ -#define TIG_HELP_CMD "" -#define TIG_PAGER_CMD "" -#define TIG_STATUS_CMD "" -#define TIG_STAGE_CMD "" -#define TIG_BLAME_CMD "" - /* Some ascii-shorthands fitted into the ncurses namespace. */ #define KEY_TAB '\t' #define KEY_RETURN '\r' @@ -359,6 +337,18 @@ argv_from_string(const char *argv[SIZEOF_ARG], int *argc, char *cmd) return *argc < SIZEOF_ARG; } +static void +argv_from_env(const char **argv, const char *name) +{ + char *env = argv ? getenv(name) : NULL; + int argc = 0; + + if (env && *env) + env = strdup(env); + if (env && !argv_from_string(argv, &argc, env)) + die("Too many arguments in the `%s` environment variable", name); +} + /* * Executing external commands. @@ -486,6 +476,12 @@ run_io_format(struct io *io, const char *cmd, ...) return io->sh[0] ? start_io(io) : FALSE; } +static bool +run_io_rd(struct io *io, const char **argv, enum format_flags flags) +{ + return init_io_rd(io, argv, NULL, flags) && start_io(io); +} + static bool io_eof(struct io *io) { @@ -1594,7 +1590,6 @@ static char ref_head[SIZEOF_REF] = "HEAD"; struct view { const char *name; /* View name */ - const char *cmd_fmt; /* Default command line format */ const char *cmd_env; /* Command line set via environment */ const char *id; /* Points to either of ref_{head,commit,blob} */ @@ -1643,6 +1638,8 @@ struct view { struct view_ops { /* What type of content being displayed. Used in the title bar. */ const char *type; + /* Default command arguments. */ + const char **argv; /* Open and reads in all view content. */ bool (*open)(struct view *view); /* Read one line; updates view->line. */ @@ -1659,6 +1656,7 @@ struct view_ops { static struct view_ops blame_ops; static struct view_ops blob_ops; +static struct view_ops diff_ops; static struct view_ops help_ops; static struct view_ops log_ops; static struct view_ops main_ops; @@ -1667,16 +1665,16 @@ static struct view_ops stage_ops; static struct view_ops status_ops; static struct view_ops tree_ops; -#define VIEW_STR(name, cmd, env, ref, ops, map, git) \ - { name, cmd, #env, ref, ops, map, git } +#define VIEW_STR(name, env, ref, ops, map, git) \ + { name, #env, ref, ops, map, git } #define VIEW_(id, name, ops, git, ref) \ - VIEW_STR(name, TIG_##id##_CMD, TIG_##id##_CMD, ref, ops, KEYMAP_##id, git) + VIEW_STR(name, TIG_##id##_CMD, ref, ops, KEYMAP_##id, git) static struct view views[] = { VIEW_(MAIN, "main", &main_ops, TRUE, ref_head), - VIEW_(DIFF, "diff", &pager_ops, TRUE, ref_commit), + VIEW_(DIFF, "diff", &diff_ops, TRUE, ref_commit), VIEW_(LOG, "log", &log_ops, TRUE, ref_head), VIEW_(TREE, "tree", &tree_ops, TRUE, ref_commit), VIEW_(BLOB, "blob", &blob_ops, TRUE, ref_blob), @@ -2548,22 +2546,14 @@ begin_update(struct view *view, bool refresh) return FALSE; } else if (view == VIEW(REQ_VIEW_TREE)) { - const char *format = view->cmd_env ? view->cmd_env : view->cmd_fmt; - char path[SIZEOF_STR]; - if (strcmp(view->vid, view->id)) - opt_path[0] = path[0] = 0; - else if (sq_quote(path, 0, opt_path) >= sizeof(path)) - return FALSE; + opt_path[0] = 0; - if (!run_io_format(&view->io, format, view->id, path)) + if (!run_io_rd(&view->io, view->ops->argv, FORMAT_ALL)) return FALSE; } else { - const char *format = view->cmd_env ? view->cmd_env : view->cmd_fmt; - const char *id = view->id; - - if (!run_io_format(&view->io, format, id, id, id, id, id)) + if (!run_io_rd(&view->io, view->ops->argv, FORMAT_ALL)) return FALSE; /* Put the current ref_* value to the view title ref @@ -3315,6 +3305,7 @@ pager_select(struct view *view, struct line *line) static struct view_ops pager_ops = { "line", NULL, + NULL, pager_read, pager_draw, pager_request, @@ -3322,6 +3313,10 @@ static struct view_ops pager_ops = { pager_select, }; +static const char *log_argv[SIZEOF_ARG] = { + "git", "log", "--no-color", "--cc", "--stat", "-n100", "%(head)", NULL +}; + static enum request log_request(struct view *view, enum request request, struct line *line) { @@ -3337,6 +3332,7 @@ log_request(struct view *view, enum request request, struct line *line) static struct view_ops log_ops = { "line", + log_argv, NULL, pager_read, pager_draw, @@ -3345,6 +3341,21 @@ static struct view_ops log_ops = { pager_select, }; +static const char *diff_argv[SIZEOF_ARG] = { + "git", "show", "--pretty=fuller", "--no-color", "--root", + "--patch-with-stat", "--find-copies-harder", "-C", "%(commit)", NULL +}; + +static struct view_ops diff_ops = { + "line", + diff_argv, + NULL, + pager_read, + pager_draw, + pager_request, + pager_grep, + pager_select, +}; /* * Help backend @@ -3430,6 +3441,7 @@ help_open(struct view *view) static struct view_ops help_ops = { "line", + NULL, help_open, NULL, pager_draw, @@ -3698,8 +3710,13 @@ tree_select(struct view *view, struct line *line) string_copy_rev(view->ref, text); } +static const char *tree_argv[SIZEOF_ARG] = { + "git", "ls-tree", "%(commit)", "%(directory)", NULL +}; + static struct view_ops tree_ops = { "file", + tree_argv, NULL, tree_read, pager_draw, @@ -3716,8 +3733,13 @@ blob_read(struct view *view, char *line) return add_line_text(view, line, LINE_DEFAULT) != NULL; } +static const char *blob_argv[SIZEOF_ARG] = { + "git", "cat-file", "blob", "%(blob)", NULL +}; + static struct view_ops blob_ops = { "line", + blob_argv, NULL, blob_read, pager_draw, @@ -4077,6 +4099,7 @@ blame_select(struct view *view, struct line *line) static struct view_ops blame_ops = { "line", + NULL, blame_open, blame_read, blame_draw, @@ -4476,7 +4499,7 @@ status_enter(struct view *view, struct line *line) } split = view_is_displayed(view) ? OPEN_SPLIT : 0; - open_view(view, REQ_VIEW_STAGE, OPEN_RELOAD | split); + open_view(view, REQ_VIEW_STAGE, OPEN_REFRESH | split); if (view_is_displayed(VIEW(REQ_VIEW_STAGE))) { if (status) { stage_status = *status; @@ -4812,6 +4835,7 @@ status_grep(struct view *view, struct line *line) static struct view_ops status_ops = { "file", + NULL, status_open, NULL, status_draw, @@ -5067,6 +5091,7 @@ stage_request(struct view *view, enum request request, struct line *line) static struct view_ops stage_ops = { "line", NULL, + NULL, pager_read, pager_draw, stage_request, @@ -5295,6 +5320,11 @@ update_rev_graph(struct rev_graph *graph) * Main view backend */ +static const char *main_argv[SIZEOF_ARG] = { + "git", "log", "--no-color", "--pretty=raw", "--parents", + "--topo-order", "%(head)", NULL +}; + static bool main_draw(struct view *view, struct line *line, unsigned int lineno) { @@ -5575,6 +5605,7 @@ main_select(struct view *view, struct line *line) static struct view_ops main_ops = { "commit", + main_argv, NULL, main_read, main_draw, @@ -6356,7 +6387,7 @@ main(int argc, const char *argv[]) die("Failed to load refs."); foreach_view (view, i) - view->cmd_env = getenv(view->cmd_env); + argv_from_env(view->ops->argv, view->cmd_env); init_display(); -- 2.32.0.93.g670b81a890