Skip to content

Commit

Permalink
Merge branch 'sg/lock-file-commit-error' into maint
Browse files Browse the repository at this point in the history
Cosmetic improvement to lock-file error messages.

* sg/lock-file-commit-error:
  Make error message after failing commit_lock_file() less confusing
  • Loading branch information
Junio C Hamano committed Dec 11, 2015
2 parents 7605881 + 08a3651 commit abca668
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 5 deletions.
6 changes: 4 additions & 2 deletions config.c
Original file line number Diff line number Diff line change
Expand Up @@ -2144,7 +2144,8 @@ int git_config_set_multivar_in_file(const char *config_filename,
}

if (commit_lock_file(lock) < 0) {
error("could not commit config file %s", config_filename);
error("could not write config file %s: %s", config_filename,
strerror(errno));
ret = CONFIG_NO_WRITE;
lock = NULL;
goto out_free;
Expand Down Expand Up @@ -2330,7 +2331,8 @@ int git_config_rename_section_in_file(const char *config_filename,
fclose(config_file);
unlock_and_out:
if (commit_lock_file(lock) < 0)
ret = error("could not commit config file %s", config_filename);
ret = error("could not write config file %s: %s",
config_filename, strerror(errno));
out:
free(filename_buf);
return ret;
Expand Down
3 changes: 2 additions & 1 deletion credential-store.c
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,8 @@ static void rewrite_credential_file(const char *fn, struct credential *c,
print_line(extra);
parse_credential_file(fn, c, NULL, print_line);
if (commit_lock_file(&credential_lock) < 0)
die_errno("unable to commit credential store");
die_errno("unable to write credential store: %s",
strerror(errno));
}

static void store_credential_file(const char *fn, struct credential *c)
Expand Down
2 changes: 1 addition & 1 deletion fast-import.c
Original file line number Diff line number Diff line change
Expand Up @@ -1821,7 +1821,7 @@ static void dump_marks(void)

dump_marks_helper(f, 0, marks);
if (commit_lock_file(&mark_lock)) {
failure |= error("Unable to commit marks file %s: %s",
failure |= error("Unable to write file %s: %s",
export_marks_file, strerror(errno));
return;
}
Expand Down
2 changes: 1 addition & 1 deletion refs.c
Original file line number Diff line number Diff line change
Expand Up @@ -4643,7 +4643,7 @@ int reflog_expire(const char *refname, const unsigned char *sha1,
get_lock_file_path(lock->lk));
rollback_lock_file(&reflog_lock);
} else if (commit_lock_file(&reflog_lock)) {
status |= error("unable to commit reflog '%s' (%s)",
status |= error("unable to write reflog %s: %s",
log_file, strerror(errno));
} else if (update && commit_ref(lock)) {
status |= error("couldn't set %s", lock->ref_name);
Expand Down

0 comments on commit abca668

Please sign in to comment.