From 04cd4198b9f75a6c6c4e84029e42907baaafce5a Mon Sep 17 00:00:00 2001 From: Ben Peart Date: Fri, 16 Mar 2018 17:47:27 -0400 Subject: [PATCH 1/4] read-cache: add post-indexchanged hook --- cache.h | 3 ++- read-cache.c | 11 +++++++++-- unpack-trees.c | 2 ++ 3 files changed, 13 insertions(+), 3 deletions(-) diff --git a/cache.h b/cache.h index e2c31d4986b63c..5366238af9b4fb 100644 --- a/cache.h +++ b/cache.h @@ -329,7 +329,8 @@ struct index_state { struct cache_time timestamp; unsigned name_hash_initialized : 1, initialized : 1, - drop_cache_tree : 1; + drop_cache_tree : 1, + updated_workdir : 1; struct hashmap name_hash; struct hashmap dir_hash; struct object_id oid; diff --git a/read-cache.c b/read-cache.c index 363fc664937ac5..c938b622fdcf33 100644 --- a/read-cache.c +++ b/read-cache.c @@ -18,6 +18,7 @@ #include "commit.h" #include "blob.h" #include "resolve-undo.h" +#include "run-command.h" #include "strbuf.h" #include "varint.h" #include "split-index.h" @@ -3058,8 +3059,14 @@ static int do_write_locked_index(struct index_state *istate, struct lock_file *l if (ret) return ret; if (flags & COMMIT_LOCK) - return commit_locked_index(lock); - return close_lock_file_gently(lock); + ret = commit_locked_index(lock); + else + ret = close_lock_file_gently(lock); + + run_hook_le(NULL, "post-indexchanged", istate->updated_workdir ? "1" : "0", NULL); + istate->updated_workdir = 0; + + return ret; } static int write_split_index(struct index_state *istate, diff --git a/unpack-trees.c b/unpack-trees.c index 11d04f014c5fbc..b139c5656f8319 100644 --- a/unpack-trees.c +++ b/unpack-trees.c @@ -1654,6 +1654,8 @@ int unpack_trees(unsigned len, struct tree_desc *t, struct unpack_trees_options WRITE_TREE_SILENT | WRITE_TREE_REPAIR); } + + o->result.updated_workdir = 1; discard_index(o->dst_index); *o->dst_index = o->result; } else { From 7454daddc4f3dda07b98f9a5e136a94e1a42c989 Mon Sep 17 00:00:00 2001 From: Ben Peart Date: Mon, 19 Mar 2018 13:16:48 -0400 Subject: [PATCH 2/4] read-cache: post-indexchanged hook add skip-worktree bit changing support --- builtin/reset.c | 1 + builtin/update-index.c | 2 ++ cache.h | 3 ++- read-cache.c | 5 ++++- 4 files changed, 9 insertions(+), 2 deletions(-) diff --git a/builtin/reset.c b/builtin/reset.c index 4e7ab9c4c947ff..32ba2e63f43dad 100644 --- a/builtin/reset.c +++ b/builtin/reset.c @@ -470,6 +470,7 @@ int cmd_reset(int argc, const char **argv, const char *prefix) "use '--quiet' to avoid this. Set the config setting reset.quiet to true\n" "to make this the default.\n"), t_delta_in_ms / 1000.0); } + the_index.updated_skipworktree = 1; } } else { int err = reset_index(&oid, reset_type, quiet); diff --git a/builtin/update-index.c b/builtin/update-index.c index e19da77edcaa02..7c74b41bda3e4c 100644 --- a/builtin/update-index.c +++ b/builtin/update-index.c @@ -1069,6 +1069,8 @@ int cmd_update_index(int argc, const char **argv, const char *prefix) entries = read_cache(); if (entries < 0) die("cache corrupted"); + + the_index.updated_skipworktree = 1; /* * Custom copy of parse_options() because we want to handle diff --git a/cache.h b/cache.h index 5366238af9b4fb..cf849fe505cd59 100644 --- a/cache.h +++ b/cache.h @@ -330,7 +330,8 @@ struct index_state { unsigned name_hash_initialized : 1, initialized : 1, drop_cache_tree : 1, - updated_workdir : 1; + updated_workdir : 1, + updated_skipworktree : 1; struct hashmap name_hash; struct hashmap dir_hash; struct object_id oid; diff --git a/read-cache.c b/read-cache.c index c938b622fdcf33..df50a8b727c0a6 100644 --- a/read-cache.c +++ b/read-cache.c @@ -3063,8 +3063,11 @@ static int do_write_locked_index(struct index_state *istate, struct lock_file *l else ret = close_lock_file_gently(lock); - run_hook_le(NULL, "post-indexchanged", istate->updated_workdir ? "1" : "0", NULL); + run_hook_le(NULL, "post-indexchanged", + istate->updated_workdir ? "1" : "0", + istate->updated_skipworktree ? "1" : "0", NULL); istate->updated_workdir = 0; + istate->updated_skipworktree = 0; return ret; } From 5e370406b9558ab22cd59597bbf50ee8a41fff29 Mon Sep 17 00:00:00 2001 From: Ben Peart Date: Mon, 19 Mar 2018 14:50:18 -0400 Subject: [PATCH 3/4] read-cache: add test for post-indexchanged hook --- t/t7113-post-index-changed-hook.sh | 144 +++++++++++++++++++++++++++++ 1 file changed, 144 insertions(+) create mode 100755 t/t7113-post-index-changed-hook.sh diff --git a/t/t7113-post-index-changed-hook.sh b/t/t7113-post-index-changed-hook.sh new file mode 100755 index 00000000000000..71e3b2859109c8 --- /dev/null +++ b/t/t7113-post-index-changed-hook.sh @@ -0,0 +1,144 @@ +#!/bin/sh + +test_description='post index changed hook' + +. ./test-lib.sh + +test_expect_success 'setup' ' + mkdir -p dir1 && + touch dir1/file1.txt && + echo testing >dir1/file2.txt && + git add . && + git commit -m "initial" +' + +test_expect_success 'test status, add, commit, others trigger hook without flags set' ' + mkdir -p .git/hooks && + write_script .git/hooks/post-indexchanged <<-\EOF && + if test "$1" -eq 1; then + echo "Invalid combination of flags passed to hook; updated_workdir is set." >testfailure + exit 1 + fi + if test "$2" -eq 1; then + echo "Invalid combination of flags passed to hook; updated_skipworktree is set." >testfailure + exit 1 + fi + if test -f ".git/index.lock"; then + echo ".git/index.lock exists" >testfailure + exit 3 + fi + if ! test -f ".git/index"; then + echo ".git/index does not exist" >testfailure + exit 3 + fi + echo "success" >testsuccess + EOF + mkdir -p dir2 && + touch dir2/file1.txt && + touch dir2/file2.txt && + : force index to be dirty && + test-tool chmtime +60 dir1/file1.txt && + git status && + test_path_is_file testsuccess && rm -f testsuccess && + test_path_is_missing testfailure && + git add . && + test_path_is_file testsuccess && rm -f testsuccess && + test_path_is_missing testfailure && + git commit -m "second" && + test_path_is_file testsuccess && rm -f testsuccess && + test_path_is_missing testfailure && + git checkout -- dir1/file1.txt && + test_path_is_file testsuccess && rm -f testsuccess && + test_path_is_missing testfailure && + git update-index && + test_path_is_missing testsuccess && + test_path_is_missing testfailure && + git reset --soft && + test_path_is_missing testsuccess && + test_path_is_missing testfailure +' + +test_expect_success 'test checkout and reset trigger the hook' ' + write_script .git/hooks/post-indexchanged <<-\EOF && + if test "$1" -eq 1 && test "$2" -eq 1; then + echo "Invalid combination of flags passed to hook; updated_workdir and updated_skipworktree are both set." >testfailure + exit 1 + fi + if test "$1" -eq 0 && test "$2" -eq 0; then + echo "Invalid combination of flags passed to hook; neither updated_workdir or updated_skipworktree are set." >testfailure + exit 2 + fi + if test "$1" -eq 1; then + if test -f ".git/index.lock"; then + echo "updated_workdir set but .git/index.lock exists" >testfailure + exit 3 + fi + if ! test -f ".git/index"; then + echo "updated_workdir set but .git/index does not exist" >testfailure + exit 3 + fi + else + echo "update_workdir should be set for checkout" >testfailure + exit 4 + fi + echo "success" >testsuccess + EOF + : force index to be dirty && + test-tool chmtime +60 dir1/file1.txt && + git checkout master && + test_path_is_file testsuccess && rm -f testsuccess && + test_path_is_missing testfailure && + test-tool chmtime +60 dir1/file1.txt && + git checkout HEAD && + test_path_is_file testsuccess && rm -f testsuccess && + test_path_is_missing testfailure && + test-tool chmtime +60 dir1/file1.txt && + git reset --hard && + test_path_is_file testsuccess && rm -f testsuccess && + test_path_is_missing testfailure && + git checkout -B test && + test_path_is_file testsuccess && rm -f testsuccess && + test_path_is_missing testfailure +' + +test_expect_success 'test reset --mixed and update-index triggers the hook' ' + write_script .git/hooks/post-indexchanged <<-\EOF && + if test "$1" -eq 1 && test "$2" -eq 1; then + echo "Invalid combination of flags passed to hook; updated_workdir and updated_skipworktree are both set." >testfailure + exit 1 + fi + if test "$1" -eq 0 && test "$2" -eq 0; then + echo "Invalid combination of flags passed to hook; neither updated_workdir or updated_skipworktree are set." >testfailure + exit 2 + fi + if test "$2" -eq 1; then + if test -f ".git/index.lock"; then + echo "updated_skipworktree set but .git/index.lock exists" >testfailure + exit 3 + fi + if ! test -f ".git/index"; then + echo "updated_skipworktree set but .git/index does not exist" >testfailure + exit 3 + fi + else + echo "updated_skipworktree should be set for reset --mixed and update-index" >testfailure + exit 4 + fi + echo "success" >testsuccess + EOF + : force index to be dirty && + test-tool chmtime +60 dir1/file1.txt && + git reset --mixed HEAD~1 && + test_path_is_file testsuccess && rm -f testsuccess && + test_path_is_missing testfailure && + git hash-object -w --stdin expect && + git update-index --cacheinfo 100644 "$(cat expect)" dir1/file1.txt && + test_path_is_file testsuccess && rm -f testsuccess && + test_path_is_missing testfailure && + git update-index --skip-worktree dir1/file2.txt && + git update-index --remove dir1/file2.txt && + test_path_is_file testsuccess && rm -f testsuccess && + test_path_is_missing testfailure +' + +test_done From 66887e1c035953045a7e9d681794cf4a5bdd7574 Mon Sep 17 00:00:00 2001 From: Kevin Willford Date: Tue, 30 Oct 2018 13:38:47 -0600 Subject: [PATCH 4/4] Add documentation for the post-indexchanged hook --- Documentation/githooks.txt | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+) diff --git a/Documentation/githooks.txt b/Documentation/githooks.txt index 87f9ad2a77a97a..1a90bc01d704c9 100644 --- a/Documentation/githooks.txt +++ b/Documentation/githooks.txt @@ -512,6 +512,24 @@ of the working directory and are separated by a single NUL. Full paths The exit status determines whether git will use the data from the hook. On error, git will abort the command with an error message. +post-indexchanged +~~~~~~~~~~~~~~~~~ + +This hook is invoked when the index is written in read-cache.c +do_write_locked_index. + +The first parameter passed to the hook is the indicator for the +working directory being updated. "1" meaning working directory +was updated or "0" when the working directory was not updated. + +The second parameter passed to the hook is the indicator for whether +or not the index was updated and the skip-worktree bit could have +changed. "1" meaning skip-worktree bits could have been updated +and "0" meaning they were not. + +Only one parameter should be set to "1" when the hook runs. The hook +running passing "1", "1" should not be possible. + GIT --- Part of the linkgit:git[1] suite