git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH v2 9/9] compat: die when unable to set core.precomposeunicode

From
Patrick Steinhardt <ps@pks.im>
Date
Jan 28, 2016, 09:00 UTC
Message-ID
<1453971637-22273-10-git-send-email-ps@pks.im>
In-Reply-To
<1453971637-22273-1-git-send-email-ps@pks.im>

When calling `git_config_set` to set 'core.precomposeunicode' we ignore the return value of the function, which may indicate that we were unable to write the value back to disk.

As surrounding code is already dying when an error occurs we simply die and print an error message when we are unable to write the config value.

Signed-off-by: Patrick Steinhardt <ps@pks.im>
---
 compat/precompose_utf8.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/compat/precompose_utf8.c b/compat/precompose_utf8.c
index 079070f..9ff1ebe 100644
--- a/compat/precompose_utf8.c
+++ b/compat/precompose_utf8.c
@@ -50,7 +50,8 @@ void probe_utf8_pathname_composition(void)
 		close(output_fd);
 		git_path_buf(&path, "%s", auml_nfd);
 		precomposed_unicode = access(path.buf, R_OK) ? 0 : 1;
-		git_config_set("core.precomposeunicode", precomposed_unicode ? "true" : "false");
+		git_config_set_or_die("core.precomposeunicode",
+				      precomposed_unicode ? "true" : "false");
 		git_path_buf(&path, "%s", auml_nfc);
 		if (unlink(path.buf))
 			die_errno(_("failed to unlink '%s'"), path.buf);
-- 
2.7.0
Previous: Patrick SteinhardtNext: Jeff King
Message 10 of 13 in “Handle errors when setting configs”
  1. 0/9 Handle errors when setting configsPatrick Steinhardt, Jan 28, 2016
  2. 1/9 config: introduce set_or_die wrappersPatrick Steinhardt, Jan 28, 2016
  3. 2/9 branch: return error code for install_branch_configPatrick Steinhardt, Jan 28, 2016
  4. 3/9 remote: handle config errors in set_urlPatrick Steinhardt, Jan 28, 2016
  5. 4/9 clone: handle config errors in cmd_clonePatrick Steinhardt, Jan 28, 2016
  6. 5/9 branch: handle config errors when unsetting upstreamPatrick Steinhardt, Jan 28, 2016
  7. 6/9 init-db: handle config errorsPatrick Steinhardt, Jan 28, 2016
  8. 7/9 sequencer: handle config errors when saving optsPatrick Steinhardt, Jan 28, 2016
  9. 8/9 submodule--helper: handle config errorsPatrick Steinhardt, Jan 28, 2016
  10. 9/9 compat: die when unable to set core.precomposeunicodePatrick Steinhardt, Jan 28, 2016
  11. Jeff KingJan 29, 2016
  12. Junio C HamanoJan 29, 2016
  13. Patrick SteinhardtJan 30, 2016

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.