diff options
| author | Junio C Hamano <gitster@pobox.com> | 2018-09-24 17:30:51 (GMT) |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2018-09-24 17:30:51 (GMT) |
| commit | faadedb19538a9968dd2d607bcab00be1916bd31 (patch) | |
| tree | 8636f755fa6b929248d66075040e6b98a51e047b | |
| parent | 5b39d49515bc07bc36bda8bc330b41bf06d5042e (diff) | |
| parent | 84d938b732242e81ec68647994070eb7c34d061f (diff) | |
| download | git-faadedb19538a9968dd2d607bcab00be1916bd31.zip git-faadedb19538a9968dd2d607bcab00be1916bd31.tar.gz git-faadedb19538a9968dd2d607bcab00be1916bd31.tar.bz2 | |
Merge branch 'nd/attr-pathspec-fix'
"git add ':(attr:foo)'" is not supported and is supposed to be
rejected while the command line arguments are parsed, but we fail
to reject such a command line upfront.
* nd/attr-pathspec-fix:
add: do not accept pathspec magic 'attr'
| -rw-r--r-- | builtin/add.c | 2 | ||||
| -rwxr-xr-x | t/t6135-pathspec-with-attrs.sh | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/builtin/add.c b/builtin/add.c index 9916498..0b64bcd 100644 --- a/builtin/add.c +++ b/builtin/add.c @@ -454,7 +454,7 @@ int cmd_add(int argc, const char **argv, const char *prefix) * Check the "pathspec '%s' did not match any files" block * below before enabling new magic. */ - parse_pathspec(&pathspec, 0, + parse_pathspec(&pathspec, PATHSPEC_ATTR, PATHSPEC_PREFER_FULL | PATHSPEC_SYMLINK_LEADING_PATH, prefix, argv); diff --git a/t/t6135-pathspec-with-attrs.sh b/t/t6135-pathspec-with-attrs.sh index 77b8cef..e436a73 100755 --- a/t/t6135-pathspec-with-attrs.sh +++ b/t/t6135-pathspec-with-attrs.sh @@ -166,7 +166,7 @@ test_expect_success 'fail if attr magic is used places not implemented' ' # though, but git-add is convenient as it has its own internal pathspec # parsing. test_must_fail git add ":(attr:labelB)" 2>actual && - test_i18ngrep "unsupported magic" actual + test_i18ngrep "magic not supported" actual ' test_expect_success 'abort on giving invalid label on the command line' ' |