diff options
author | Ben Straub <bs@github.com> | 2014-01-28 01:12:31 +0400 |
---|---|---|
committer | Ben Straub <bs@github.com> | 2014-01-31 03:51:00 +0400 |
commit | a2311f92c29e0fee5348d580c3318cf6724cd765 (patch) | |
tree | f4af06ba4ea36749b3f4a76789cd27aa13a457f6 | |
parent | 94f263f59be5be74945367b9793c57f297ed4a44 (diff) |
Ensure updating HEAD updates reflog
-rw-r--r-- | src/refdb_fs.c | 7 | ||||
-rw-r--r-- | tests/repo/head.c | 23 |
2 files changed, 27 insertions, 3 deletions
diff --git a/src/refdb_fs.c b/src/refdb_fs.c index 53c42458f..9ae784782 100644 --- a/src/refdb_fs.c +++ b/src/refdb_fs.c @@ -1434,12 +1434,12 @@ success: static int reflog_append(refdb_fs_backend *backend, const git_reference *ref, const git_signature *who, const char *message) { int error; - git_oid old_id, new_id; + git_oid old_id, new_id = {{0}}; git_buf buf = GIT_BUF_INIT, path = GIT_BUF_INIT; git_repository *repo = backend->repo; /* Creation of symbolic references doesn't get a reflog entry */ - if (ref->type == GIT_REF_SYMBOLIC) + if (ref->type == GIT_REF_SYMBOLIC && strcmp(ref->name, GIT_HEAD_FILE)) return 0; error = git_reference_name_to_id(&old_id, repo, ref->name); @@ -1450,7 +1450,8 @@ static int reflog_append(refdb_fs_backend *backend, const git_reference *ref, co if (error < 0) return error; - git_oid_cpy(&new_id, git_reference_target(ref)); + if (git_reference_target(ref) != NULL) + git_oid_cpy(&new_id, git_reference_target(ref)); if ((error = serialize_reflog_entry(&buf, &old_id, &new_id, who, message)) < 0) goto cleanup; diff --git a/tests/repo/head.c b/tests/repo/head.c index 06e837d70..df7059fd5 100644 --- a/tests/repo/head.c +++ b/tests/repo/head.c @@ -194,3 +194,26 @@ void test_repo_head__can_tell_if_an_unborn_head_is_detached(void) cl_assert_equal_i(false, git_repository_head_detached(repo)); } + +void test_repo_head__setting_head_updates_reflog(void) +{ + git_reflog *log; + const git_reflog_entry *entry1, *entry2, *entry3; + git_object *tag; + + cl_git_pass(git_repository_set_head(repo, "refs/heads/haacked", NULL, "message1")); + cl_git_pass(git_repository_set_head(repo, "refs/heads/unborn", NULL, "message2")); + cl_git_pass(git_revparse_single(&tag, repo, "tags/test")); + cl_git_pass(git_repository_set_head_detached(repo, git_object_id(tag), NULL, "message3")); + + cl_git_pass(git_reflog_read(&log, repo, "HEAD")); + entry1 = git_reflog_entry_byindex(log, 2); + entry2 = git_reflog_entry_byindex(log, 1); + entry3 = git_reflog_entry_byindex(log, 0); + cl_assert_equal_s("message1", git_reflog_entry_message(entry1)); + cl_assert_equal_s("message2", git_reflog_entry_message(entry2)); + cl_assert_equal_s("message3", git_reflog_entry_message(entry3)); + + git_reflog_free(log); + git_object_free(tag); +} |