Signed-off-by: Pierre Habouzit <madcoder@xxxxxxxxxx> --- builtin-reflog.c | 1 + t/t1301-shared-repo.sh | 15 +++++++++++++++ 2 files changed, 16 insertions(+), 0 deletions(-) Some people like to have logAllRefUpdates even for bare repositories, and if shared, git-gc breaks them. diff --git a/builtin-reflog.c b/builtin-reflog.c index 897d1dc..430929f 100644 --- a/builtin-reflog.c +++ b/builtin-reflog.c @@ -308,6 +308,7 @@ static int expire_reflog(const char *ref, const unsigned char *sha1, int unused, } else if (cmd->updateref && commit_ref(lock)) { status |= error("Couldn't set %s", lock->ref_name); } + adjust_shared_perm(log_file); } free(newlog_path); free(log_file); diff --git a/t/t1301-shared-repo.sh b/t/t1301-shared-repo.sh index 5e4252a..6c78c8b 100755 --- a/t/t1301-shared-repo.sh +++ b/t/t1301-shared-repo.sh @@ -83,4 +83,19 @@ do done +test_expect_success 'git reflog expire honors core.sharedRepository' ' + git config core.sharedRepository group && + git reflog expire --all && + actual="$(ls -l .git/logs/refs/heads/master)" && + case "$actual" in + -rw-rw-*) + : happy + ;; + *) + echo Ooops, .git/logs/refs/heads/master is not 0662 [$actual] + false + ;; + esac +' + test_done -- 1.5.6.rc3.149.gaacb0.dirty -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html