2010-02-19 10:18:58 +03:00
|
|
|
#!/bin/sh
|
|
|
|
|
|
|
|
test_description='Test automatic use of a pager.'
|
|
|
|
|
|
|
|
. ./test-lib.sh
|
2010-06-12 20:36:51 +04:00
|
|
|
. "$TEST_DIRECTORY"/lib-pager.sh
|
2010-10-16 22:36:56 +04:00
|
|
|
. "$TEST_DIRECTORY"/lib-terminal.sh
|
2010-02-19 10:18:58 +03:00
|
|
|
|
|
|
|
test_expect_success 'setup' '
|
2010-10-04 00:00:14 +04:00
|
|
|
sane_unset GIT_PAGER GIT_PAGER_IN_USE &&
|
2011-08-18 09:02:06 +04:00
|
|
|
test_unconfig core.pager &&
|
2010-04-15 04:38:07 +04:00
|
|
|
|
|
|
|
PAGER="cat >paginated.out" &&
|
|
|
|
export PAGER &&
|
|
|
|
|
2010-02-19 10:18:58 +03:00
|
|
|
test_commit initial
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success TTY 'some commands use a pager' '
|
2011-12-14 12:22:03 +04:00
|
|
|
rm -f paginated.out &&
|
2010-02-20 11:50:25 +03:00
|
|
|
test_terminal git log &&
|
2010-02-19 10:18:58 +03:00
|
|
|
test -e paginated.out
|
|
|
|
'
|
|
|
|
|
2010-08-06 07:12:46 +04:00
|
|
|
test_expect_failure TTY 'pager runs from subdir' '
|
|
|
|
echo subdir/paginated.out >expected &&
|
|
|
|
mkdir -p subdir &&
|
|
|
|
rm -f paginated.out subdir/paginated.out &&
|
|
|
|
(
|
|
|
|
cd subdir &&
|
|
|
|
test_terminal git log
|
|
|
|
) &&
|
|
|
|
{
|
|
|
|
ls paginated.out subdir/paginated.out ||
|
|
|
|
:
|
|
|
|
} >actual &&
|
|
|
|
test_cmp expected actual
|
|
|
|
'
|
|
|
|
|
2014-01-07 06:14:05 +04:00
|
|
|
test_expect_success TTY 'LESS and LV envvars are set for pagination' '
|
|
|
|
(
|
|
|
|
sane_unset LESS LV &&
|
2014-01-30 20:42:32 +04:00
|
|
|
PAGER="env >pager-env.out; wc" &&
|
2014-01-07 06:14:05 +04:00
|
|
|
export PAGER &&
|
|
|
|
|
|
|
|
test_terminal git log
|
|
|
|
) &&
|
|
|
|
grep ^LESS= pager-env.out &&
|
|
|
|
grep ^LV= pager-env.out
|
|
|
|
'
|
|
|
|
|
2010-02-19 10:18:58 +03:00
|
|
|
test_expect_success TTY 'some commands do not use a pager' '
|
2011-12-14 12:22:03 +04:00
|
|
|
rm -f paginated.out &&
|
2010-02-20 11:50:25 +03:00
|
|
|
test_terminal git rev-list HEAD &&
|
2010-02-19 10:18:58 +03:00
|
|
|
! test -e paginated.out
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'no pager when stdout is a pipe' '
|
2011-12-14 12:22:03 +04:00
|
|
|
rm -f paginated.out &&
|
2010-02-19 10:18:58 +03:00
|
|
|
git log | cat &&
|
|
|
|
! test -e paginated.out
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'no pager when stdout is a regular file' '
|
2011-12-14 12:22:03 +04:00
|
|
|
rm -f paginated.out &&
|
2010-04-15 04:38:07 +04:00
|
|
|
git log >file &&
|
2010-02-19 10:18:58 +03:00
|
|
|
! test -e paginated.out
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success TTY 'git --paginate rev-list uses a pager' '
|
2011-12-14 12:22:03 +04:00
|
|
|
rm -f paginated.out &&
|
2010-02-20 11:50:25 +03:00
|
|
|
test_terminal git --paginate rev-list HEAD &&
|
2010-02-19 10:18:58 +03:00
|
|
|
test -e paginated.out
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'no pager even with --paginate when stdout is a pipe' '
|
2011-12-14 12:22:03 +04:00
|
|
|
rm -f file paginated.out &&
|
2010-02-19 10:18:58 +03:00
|
|
|
git --paginate log | cat &&
|
|
|
|
! test -e paginated.out
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success TTY 'no pager with --no-pager' '
|
2011-12-14 12:22:03 +04:00
|
|
|
rm -f paginated.out &&
|
2010-02-20 11:50:25 +03:00
|
|
|
test_terminal git --no-pager log &&
|
2010-02-19 10:18:58 +03:00
|
|
|
! test -e paginated.out
|
|
|
|
'
|
|
|
|
|
2010-08-06 07:06:39 +04:00
|
|
|
test_expect_success TTY 'configuration can disable pager' '
|
|
|
|
rm -f paginated.out &&
|
2011-08-18 09:02:06 +04:00
|
|
|
test_unconfig pager.grep &&
|
2010-08-06 07:06:39 +04:00
|
|
|
test_terminal git grep initial &&
|
|
|
|
test -e paginated.out &&
|
|
|
|
|
|
|
|
rm -f paginated.out &&
|
2011-08-18 09:02:06 +04:00
|
|
|
test_config pager.grep false &&
|
2010-08-06 07:06:39 +04:00
|
|
|
test_terminal git grep initial &&
|
|
|
|
! test -e paginated.out
|
|
|
|
'
|
|
|
|
|
2010-08-16 12:08:34 +04:00
|
|
|
test_expect_success TTY 'git config uses a pager if configured to' '
|
2010-08-06 07:15:09 +04:00
|
|
|
rm -f paginated.out &&
|
2011-08-18 09:02:06 +04:00
|
|
|
test_config pager.config true &&
|
2010-08-06 07:15:09 +04:00
|
|
|
test_terminal git config --list &&
|
|
|
|
test -e paginated.out
|
|
|
|
'
|
|
|
|
|
2010-08-16 12:08:34 +04:00
|
|
|
test_expect_success TTY 'configuration can enable pager (from subdir)' '
|
2010-08-06 07:12:46 +04:00
|
|
|
rm -f paginated.out &&
|
|
|
|
mkdir -p subdir &&
|
2011-08-18 09:02:06 +04:00
|
|
|
test_config pager.bundle true &&
|
2010-08-06 07:12:46 +04:00
|
|
|
|
|
|
|
git bundle create test.bundle --all &&
|
|
|
|
rm -f paginated.out subdir/paginated.out &&
|
|
|
|
(
|
|
|
|
cd subdir &&
|
|
|
|
test_terminal git bundle unbundle ../test.bundle
|
|
|
|
) &&
|
|
|
|
{
|
|
|
|
test -e paginated.out ||
|
|
|
|
test -e subdir/paginated.out
|
|
|
|
}
|
|
|
|
'
|
|
|
|
|
2010-02-19 10:18:58 +03:00
|
|
|
# A colored commit log will begin with an appropriate ANSI escape
|
|
|
|
# for the first color; the text "commit" comes later.
|
|
|
|
colorful() {
|
2010-04-15 04:38:07 +04:00
|
|
|
read firstline <$1
|
2010-06-21 22:18:54 +04:00
|
|
|
! expr "$firstline" : "[a-zA-Z]" >/dev/null
|
2010-02-19 10:18:58 +03:00
|
|
|
}
|
|
|
|
|
|
|
|
test_expect_success 'tests can detect color' '
|
2011-12-14 12:22:03 +04:00
|
|
|
rm -f colorful.log colorless.log &&
|
2010-04-15 04:38:07 +04:00
|
|
|
git log --no-color >colorless.log &&
|
|
|
|
git log --color >colorful.log &&
|
2010-02-19 10:18:58 +03:00
|
|
|
! colorful colorless.log &&
|
|
|
|
colorful colorful.log
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'no color when stdout is a regular file' '
|
2010-04-15 04:38:07 +04:00
|
|
|
rm -f colorless.log &&
|
2011-12-14 12:22:03 +04:00
|
|
|
test_config color.ui auto &&
|
2010-04-15 04:38:07 +04:00
|
|
|
git log >colorless.log &&
|
2010-02-19 10:18:58 +03:00
|
|
|
! colorful colorless.log
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success TTY 'color when writing to a pager' '
|
2010-04-15 04:38:07 +04:00
|
|
|
rm -f paginated.out &&
|
2011-12-14 12:22:03 +04:00
|
|
|
test_config color.ui auto &&
|
2014-03-18 22:54:05 +04:00
|
|
|
test_terminal env TERM=vt100 git log &&
|
2010-02-19 10:18:58 +03:00
|
|
|
colorful paginated.out
|
|
|
|
'
|
|
|
|
|
color: delay auto-color decision until point of use
When we read a color value either from a config file or from
the command line, we use git_config_colorbool to convert it
from the tristate always/never/auto into a single yes/no
boolean value.
This has some timing implications with respect to starting
a pager.
If we start (or decide not to start) the pager before
checking the colorbool, everything is fine. Either isatty(1)
will give us the right information, or we will properly
check for pager_in_use().
However, if we decide to start a pager after we have checked
the colorbool, things are not so simple. If stdout is a tty,
then we will have already decided to use color. However, the
user may also have configured color.pager not to use color
with the pager. In this case, we need to actually turn off
color. Unfortunately, the pager code has no idea which color
variables were turned on (and there are many of them
throughout the code, and they may even have been manipulated
after the colorbool selection by something like "--color" on
the command line).
This bug can be seen any time a pager is started after
config and command line options are checked. This has
affected "git diff" since 89d07f7 (diff: don't run pager if
user asked for a diff style exit code, 2007-08-12). It has
also affect the log family since 1fda91b (Fix 'git log'
early pager startup error case, 2010-08-24).
This patch splits the notion of parsing a colorbool and
actually checking the configuration. The "use_color"
variables now have an additional possible value,
GIT_COLOR_AUTO. Users of the variable should use the new
"want_color()" wrapper, which will lazily determine and
cache the auto-color decision.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2011-08-18 09:04:23 +04:00
|
|
|
test_expect_success TTY 'colors are suppressed by color.pager' '
|
|
|
|
rm -f paginated.out &&
|
|
|
|
test_config color.ui auto &&
|
|
|
|
test_config color.pager false &&
|
2014-03-18 22:54:05 +04:00
|
|
|
test_terminal env TERM=vt100 git log &&
|
color: delay auto-color decision until point of use
When we read a color value either from a config file or from
the command line, we use git_config_colorbool to convert it
from the tristate always/never/auto into a single yes/no
boolean value.
This has some timing implications with respect to starting
a pager.
If we start (or decide not to start) the pager before
checking the colorbool, everything is fine. Either isatty(1)
will give us the right information, or we will properly
check for pager_in_use().
However, if we decide to start a pager after we have checked
the colorbool, things are not so simple. If stdout is a tty,
then we will have already decided to use color. However, the
user may also have configured color.pager not to use color
with the pager. In this case, we need to actually turn off
color. Unfortunately, the pager code has no idea which color
variables were turned on (and there are many of them
throughout the code, and they may even have been manipulated
after the colorbool selection by something like "--color" on
the command line).
This bug can be seen any time a pager is started after
config and command line options are checked. This has
affected "git diff" since 89d07f7 (diff: don't run pager if
user asked for a diff style exit code, 2007-08-12). It has
also affect the log family since 1fda91b (Fix 'git log'
early pager startup error case, 2010-08-24).
This patch splits the notion of parsing a colorbool and
actually checking the configuration. The "use_color"
variables now have an additional possible value,
GIT_COLOR_AUTO. Users of the variable should use the new
"want_color()" wrapper, which will lazily determine and
cache the auto-color decision.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2011-08-18 09:04:23 +04:00
|
|
|
! colorful paginated.out
|
|
|
|
'
|
|
|
|
|
2010-02-19 10:18:58 +03:00
|
|
|
test_expect_success 'color when writing to a file intended for a pager' '
|
2010-04-15 04:38:07 +04:00
|
|
|
rm -f colorful.log &&
|
2011-12-14 12:22:03 +04:00
|
|
|
test_config color.ui auto &&
|
2010-04-15 04:38:07 +04:00
|
|
|
(
|
|
|
|
TERM=vt100 &&
|
|
|
|
GIT_PAGER_IN_USE=true &&
|
|
|
|
export TERM GIT_PAGER_IN_USE &&
|
|
|
|
git log >colorful.log
|
|
|
|
) &&
|
2010-02-19 10:18:58 +03:00
|
|
|
colorful colorful.log
|
|
|
|
'
|
|
|
|
|
2011-08-18 09:02:29 +04:00
|
|
|
test_expect_success TTY 'colors are sent to pager for external commands' '
|
|
|
|
test_config alias.externallog "!git log" &&
|
|
|
|
test_config color.ui auto &&
|
2014-03-18 22:54:05 +04:00
|
|
|
test_terminal env TERM=vt100 git -p externallog &&
|
2011-08-18 09:02:29 +04:00
|
|
|
colorful paginated.out
|
|
|
|
'
|
|
|
|
|
2010-06-26 23:23:02 +04:00
|
|
|
# Use this helper to make it easy for the caller of your
|
|
|
|
# terminal-using function to specify whether it should fail.
|
|
|
|
# If you write
|
|
|
|
#
|
|
|
|
# your_test() {
|
|
|
|
# parse_args "$@"
|
|
|
|
#
|
|
|
|
# $test_expectation "$cmd - behaves well" "
|
|
|
|
# ...
|
|
|
|
# $full_command &&
|
|
|
|
# ...
|
|
|
|
# "
|
|
|
|
# }
|
|
|
|
#
|
|
|
|
# then your test can be used like this:
|
|
|
|
#
|
|
|
|
# your_test expect_(success|failure) [test_must_fail] 'git foo'
|
|
|
|
#
|
|
|
|
parse_args() {
|
|
|
|
test_expectation="test_$1"
|
|
|
|
shift
|
|
|
|
if test "$1" = test_must_fail
|
|
|
|
then
|
|
|
|
full_command="test_must_fail test_terminal "
|
|
|
|
shift
|
|
|
|
else
|
|
|
|
full_command="test_terminal "
|
|
|
|
fi
|
|
|
|
cmd=$1
|
|
|
|
full_command="$full_command $1"
|
|
|
|
}
|
2010-04-15 04:38:07 +04:00
|
|
|
|
2010-06-26 23:24:50 +04:00
|
|
|
test_default_pager() {
|
|
|
|
parse_args "$@"
|
|
|
|
|
2010-10-16 22:36:59 +04:00
|
|
|
$test_expectation SIMPLEPAGER,TTY "$cmd - default pager is used by default" "
|
2010-10-04 00:00:14 +04:00
|
|
|
sane_unset PAGER GIT_PAGER &&
|
2011-08-18 09:02:06 +04:00
|
|
|
test_unconfig core.pager &&
|
2011-12-14 12:22:03 +04:00
|
|
|
rm -f default_pager_used &&
|
2010-06-26 23:24:50 +04:00
|
|
|
cat >\$less <<-\EOF &&
|
|
|
|
#!/bin/sh
|
|
|
|
wc >default_pager_used
|
|
|
|
EOF
|
|
|
|
chmod +x \$less &&
|
|
|
|
(
|
|
|
|
PATH=.:\$PATH &&
|
|
|
|
export PATH &&
|
|
|
|
$full_command
|
|
|
|
) &&
|
|
|
|
test -e default_pager_used
|
|
|
|
"
|
|
|
|
}
|
2010-02-19 10:18:58 +03:00
|
|
|
|
2010-06-26 23:24:50 +04:00
|
|
|
test_PAGER_overrides() {
|
|
|
|
parse_args "$@"
|
2010-04-15 04:38:07 +04:00
|
|
|
|
2010-06-26 23:24:50 +04:00
|
|
|
$test_expectation TTY "$cmd - PAGER overrides default pager" "
|
2010-10-04 00:00:14 +04:00
|
|
|
sane_unset GIT_PAGER &&
|
2011-08-18 09:02:06 +04:00
|
|
|
test_unconfig core.pager &&
|
2011-12-14 12:22:03 +04:00
|
|
|
rm -f PAGER_used &&
|
2010-06-26 23:24:50 +04:00
|
|
|
PAGER='wc >PAGER_used' &&
|
|
|
|
export PAGER &&
|
|
|
|
$full_command &&
|
|
|
|
test -e PAGER_used
|
|
|
|
"
|
|
|
|
}
|
2010-04-15 04:38:07 +04:00
|
|
|
|
2010-06-26 23:24:50 +04:00
|
|
|
test_core_pager_overrides() {
|
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 23:26:37 +04:00
|
|
|
if_local_config=
|
|
|
|
used_if_wanted='overrides PAGER'
|
|
|
|
test_core_pager "$@"
|
|
|
|
}
|
2010-02-19 10:18:58 +03:00
|
|
|
|
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 23:26:37 +04:00
|
|
|
test_local_config_ignored() {
|
|
|
|
if_local_config='! '
|
|
|
|
used_if_wanted='is not used'
|
|
|
|
test_core_pager "$@"
|
|
|
|
}
|
2010-04-15 04:38:07 +04:00
|
|
|
|
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 23:26:37 +04:00
|
|
|
test_core_pager() {
|
2010-06-26 23:24:50 +04:00
|
|
|
parse_args "$@"
|
|
|
|
|
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 23:26:37 +04:00
|
|
|
$test_expectation TTY "$cmd - repository-local core.pager setting $used_if_wanted" "
|
2010-10-04 00:00:14 +04:00
|
|
|
sane_unset GIT_PAGER &&
|
2011-12-14 12:22:03 +04:00
|
|
|
rm -f core.pager_used &&
|
2010-06-26 23:24:50 +04:00
|
|
|
PAGER=wc &&
|
|
|
|
export PAGER &&
|
2011-08-18 09:02:06 +04:00
|
|
|
test_config core.pager 'wc >core.pager_used' &&
|
2010-06-26 23:24:50 +04:00
|
|
|
$full_command &&
|
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 23:26:37 +04:00
|
|
|
${if_local_config}test -e core.pager_used
|
2010-06-26 23:24:50 +04:00
|
|
|
"
|
|
|
|
}
|
|
|
|
|
2010-06-26 23:25:37 +04:00
|
|
|
test_core_pager_subdir() {
|
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 23:26:37 +04:00
|
|
|
if_local_config=
|
|
|
|
used_if_wanted='overrides PAGER'
|
|
|
|
test_pager_subdir_helper "$@"
|
|
|
|
}
|
|
|
|
|
|
|
|
test_no_local_config_subdir() {
|
|
|
|
if_local_config='! '
|
|
|
|
used_if_wanted='is not used'
|
|
|
|
test_pager_subdir_helper "$@"
|
|
|
|
}
|
|
|
|
|
|
|
|
test_pager_subdir_helper() {
|
2010-06-26 23:25:37 +04:00
|
|
|
parse_args "$@"
|
|
|
|
|
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 23:26:37 +04:00
|
|
|
$test_expectation TTY "$cmd - core.pager $used_if_wanted from subdirectory" "
|
2010-10-04 00:00:14 +04:00
|
|
|
sane_unset GIT_PAGER &&
|
2010-06-26 23:25:37 +04:00
|
|
|
rm -f core.pager_used &&
|
2011-12-14 12:22:03 +04:00
|
|
|
rm -fr sub &&
|
2010-06-26 23:25:37 +04:00
|
|
|
PAGER=wc &&
|
|
|
|
stampname=\$(pwd)/core.pager_used &&
|
|
|
|
export PAGER stampname &&
|
2011-08-18 09:02:06 +04:00
|
|
|
test_config core.pager 'wc >\"\$stampname\"' &&
|
2010-06-26 23:25:37 +04:00
|
|
|
mkdir sub &&
|
|
|
|
(
|
|
|
|
cd sub &&
|
|
|
|
$full_command
|
|
|
|
) &&
|
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 23:26:37 +04:00
|
|
|
${if_local_config}test -e core.pager_used
|
2010-06-26 23:25:37 +04:00
|
|
|
"
|
|
|
|
}
|
|
|
|
|
2010-06-26 23:24:50 +04:00
|
|
|
test_GIT_PAGER_overrides() {
|
|
|
|
parse_args "$@"
|
|
|
|
|
|
|
|
$test_expectation TTY "$cmd - GIT_PAGER overrides core.pager" "
|
2011-12-14 12:22:03 +04:00
|
|
|
rm -f GIT_PAGER_used &&
|
2011-08-18 09:02:06 +04:00
|
|
|
test_config core.pager wc &&
|
2010-06-26 23:24:50 +04:00
|
|
|
GIT_PAGER='wc >GIT_PAGER_used' &&
|
|
|
|
export GIT_PAGER &&
|
|
|
|
$full_command &&
|
|
|
|
test -e GIT_PAGER_used
|
|
|
|
"
|
|
|
|
}
|
2010-04-15 04:38:07 +04:00
|
|
|
|
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 23:26:37 +04:00
|
|
|
test_doesnt_paginate() {
|
|
|
|
parse_args "$@"
|
|
|
|
|
|
|
|
$test_expectation TTY "no pager for '$cmd'" "
|
2011-12-14 12:22:03 +04:00
|
|
|
rm -f GIT_PAGER_used &&
|
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 23:26:37 +04:00
|
|
|
GIT_PAGER='wc >GIT_PAGER_used' &&
|
|
|
|
export GIT_PAGER &&
|
|
|
|
$full_command &&
|
|
|
|
! test -e GIT_PAGER_used
|
|
|
|
"
|
|
|
|
}
|
|
|
|
|
2010-07-15 02:55:12 +04:00
|
|
|
test_pager_choices() {
|
|
|
|
test_default_pager expect_success "$@"
|
|
|
|
test_PAGER_overrides expect_success "$@"
|
|
|
|
test_core_pager_overrides expect_success "$@"
|
|
|
|
test_core_pager_subdir expect_success "$@"
|
|
|
|
test_GIT_PAGER_overrides expect_success "$@"
|
|
|
|
}
|
|
|
|
|
|
|
|
test_expect_success 'setup: some aliases' '
|
|
|
|
git config alias.aliasedlog log &&
|
|
|
|
git config alias.true "!true"
|
2010-02-19 10:18:58 +03:00
|
|
|
'
|
|
|
|
|
2010-07-15 02:55:12 +04:00
|
|
|
test_pager_choices 'git log'
|
|
|
|
test_pager_choices 'git -p log'
|
|
|
|
test_pager_choices 'git aliasedlog'
|
|
|
|
|
|
|
|
test_default_pager expect_success 'git -p aliasedlog'
|
|
|
|
test_PAGER_overrides expect_success 'git -p aliasedlog'
|
|
|
|
test_core_pager_overrides expect_success 'git -p aliasedlog'
|
|
|
|
test_core_pager_subdir expect_failure 'git -p aliasedlog'
|
|
|
|
test_GIT_PAGER_overrides expect_success 'git -p aliasedlog'
|
|
|
|
|
|
|
|
test_default_pager expect_success 'git -p true'
|
|
|
|
test_PAGER_overrides expect_success 'git -p true'
|
|
|
|
test_core_pager_overrides expect_success 'git -p true'
|
|
|
|
test_core_pager_subdir expect_failure 'git -p true'
|
|
|
|
test_GIT_PAGER_overrides expect_success 'git -p true'
|
|
|
|
|
|
|
|
test_default_pager expect_success test_must_fail 'git -p request-pull'
|
|
|
|
test_PAGER_overrides expect_success test_must_fail 'git -p request-pull'
|
|
|
|
test_core_pager_overrides expect_success test_must_fail 'git -p request-pull'
|
|
|
|
test_core_pager_subdir expect_failure test_must_fail 'git -p request-pull'
|
|
|
|
test_GIT_PAGER_overrides expect_success test_must_fail 'git -p request-pull'
|
2010-06-26 23:24:50 +04:00
|
|
|
|
|
|
|
test_default_pager expect_success test_must_fail 'git -p'
|
|
|
|
test_PAGER_overrides expect_success test_must_fail 'git -p'
|
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 23:26:37 +04:00
|
|
|
test_local_config_ignored expect_failure test_must_fail 'git -p'
|
|
|
|
test_no_local_config_subdir expect_success test_must_fail 'git -p'
|
2010-06-26 23:24:50 +04:00
|
|
|
test_GIT_PAGER_overrides expect_success test_must_fail 'git -p'
|
|
|
|
|
2010-07-15 02:55:12 +04:00
|
|
|
test_doesnt_paginate expect_failure test_must_fail 'git -p nonsense'
|
2010-02-19 10:18:58 +03:00
|
|
|
|
2010-08-06 07:01:37 +04:00
|
|
|
test_pager_choices 'git shortlog'
|
|
|
|
test_expect_success 'setup: configure shortlog not to paginate' '
|
|
|
|
git config pager.shortlog false
|
|
|
|
'
|
|
|
|
test_doesnt_paginate expect_success 'git shortlog'
|
|
|
|
test_no_local_config_subdir expect_success 'git shortlog'
|
|
|
|
test_default_pager expect_success 'git -p shortlog'
|
|
|
|
test_core_pager_subdir expect_success 'git -p shortlog'
|
|
|
|
|
2010-08-16 04:36:12 +04:00
|
|
|
test_core_pager_subdir expect_success test_must_fail \
|
|
|
|
'git -p apply </dev/null'
|
|
|
|
|
2010-11-17 20:04:12 +03:00
|
|
|
test_expect_success TTY 'command-specific pager' '
|
2011-08-18 09:00:47 +04:00
|
|
|
sane_unset PAGER GIT_PAGER &&
|
2010-11-17 20:04:12 +03:00
|
|
|
echo "foo:initial" >expect &&
|
|
|
|
>actual &&
|
2011-08-18 09:02:06 +04:00
|
|
|
test_unconfig core.pager &&
|
|
|
|
test_config pager.log "sed s/^/foo:/ >actual" &&
|
2010-11-17 20:04:12 +03:00
|
|
|
test_terminal git log --format=%s -1 &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success TTY 'command-specific pager overrides core.pager' '
|
2011-08-18 09:00:47 +04:00
|
|
|
sane_unset PAGER GIT_PAGER &&
|
2010-11-17 20:04:12 +03:00
|
|
|
echo "foo:initial" >expect &&
|
|
|
|
>actual &&
|
2015-03-20 13:07:15 +03:00
|
|
|
test_config core.pager "exit 1" &&
|
2011-08-18 09:02:06 +04:00
|
|
|
test_config pager.log "sed s/^/foo:/ >actual" &&
|
2010-11-17 20:04:12 +03:00
|
|
|
test_terminal git log --format=%s -1 &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success TTY 'command-specific pager overridden by environment' '
|
|
|
|
GIT_PAGER="sed s/^/foo:/ >actual" && export GIT_PAGER &&
|
|
|
|
>actual &&
|
|
|
|
echo "foo:initial" >expect &&
|
2011-08-18 09:02:06 +04:00
|
|
|
test_config pager.log "exit 1" &&
|
2010-11-17 20:04:12 +03:00
|
|
|
test_terminal git log --format=%s -1 &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
2011-08-19 02:01:32 +04:00
|
|
|
test_expect_success 'setup external command' '
|
|
|
|
cat >git-external <<-\EOF &&
|
|
|
|
#!/bin/sh
|
|
|
|
git "$@"
|
|
|
|
EOF
|
|
|
|
chmod +x git-external
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success TTY 'command-specific pager works for external commands' '
|
|
|
|
sane_unset PAGER GIT_PAGER &&
|
|
|
|
echo "foo:initial" >expect &&
|
|
|
|
>actual &&
|
|
|
|
test_config pager.external "sed s/^/foo:/ >actual" &&
|
2016-01-07 16:51:50 +03:00
|
|
|
test_terminal git --exec-path="$(pwd)" external log --format=%s -1 &&
|
2011-08-19 02:01:32 +04:00
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success TTY 'sub-commands of externals use their own pager' '
|
|
|
|
sane_unset PAGER GIT_PAGER &&
|
|
|
|
echo "foo:initial" >expect &&
|
|
|
|
>actual &&
|
|
|
|
test_config pager.log "sed s/^/foo:/ >actual" &&
|
|
|
|
test_terminal git --exec-path=. external log --format=%s -1 &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success TTY 'external command pagers override sub-commands' '
|
|
|
|
sane_unset PAGER GIT_PAGER &&
|
|
|
|
>expect &&
|
|
|
|
>actual &&
|
|
|
|
test_config pager.external false &&
|
|
|
|
test_config pager.log "sed s/^/log:/ >actual" &&
|
|
|
|
test_terminal git --exec-path=. external log --format=%s -1 &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
config: silence warnings for command names with invalid keys
When we are running the git command "foo", we may have to
look up the config keys "pager.foo" and "alias.foo". These
config schemes are mis-designed, as the command names can be
anything, but the config syntax has some restrictions. For
example:
$ git foo_bar
error: invalid key: pager.foo_bar
error: invalid key: alias.foo_bar
git: 'foo_bar' is not a git command. See 'git --help'.
You cannot name an alias with an underscore. And if you have
an external command with one, you cannot configure its
pager.
In the long run, we may develop a different config scheme
for these features. But in the near term (and because we'll
need to support the existing scheme indefinitely), we should
at least squelch the error messages shown above.
These errors come from git_config_parse_key. Ideally we
would pass a "quiet" flag to the config machinery, but there
are many layers between the pager code and the key parsing.
Passing a flag through all of those would be an invasive
change.
Instead, let's provide a config function to report on
whether a key is syntactically valid, and have the pager and
alias code skip lookup for bogus keys. We can build this
easily around the existing git_config_parse_key, with two
minor modifications:
1. We now handle a NULL store_key, to validate but not
write out the normalized key.
2. We accept a "quiet" flag to avoid writing to stderr.
This doesn't need to be a full-blown public "flags"
field, because we can make the existing implementation
a static helper function, keeping the mess contained
inside config.c.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2015-08-24 09:11:33 +03:00
|
|
|
test_expect_success 'command with underscores does not complain' '
|
|
|
|
write_script git-under_score <<-\EOF &&
|
|
|
|
echo ok
|
|
|
|
EOF
|
|
|
|
git --exec-path=. under_score >actual 2>&1 &&
|
|
|
|
echo ok >expect &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
2010-02-19 10:18:58 +03:00
|
|
|
test_done
|