diff options
| author | James McCoy <jamessan@jamessan.com> | 2021-01-16 23:40:36 -0500 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-01-16 23:40:36 -0500 |
| commit | 0f2d37f11fac7eed4617684bad2771e864818a8f (patch) | |
| tree | 6c6ec42fa412c609719d490f3a84462123bbdb79 /src | |
| parent | abbfaf286f26989b18377d4238f3cea9dc281149 (diff) | |
| parent | 640febaca71bb4bb0adee399c2cb094f57441430 (diff) | |
| download | rneovim-0f2d37f11fac7eed4617684bad2771e864818a8f.tar.gz rneovim-0f2d37f11fac7eed4617684bad2771e864818a8f.tar.bz2 rneovim-0f2d37f11fac7eed4617684bad2771e864818a8f.zip | |
Merge pull request #13774 from mjlbach/log_warn_cache
log.c: warn instead of error on failed stdpath('cache') creation
Diffstat (limited to 'src')
| -rw-r--r-- | src/nvim/log.c | 16 |
1 files changed, 7 insertions, 9 deletions
diff --git a/src/nvim/log.c b/src/nvim/log.c index 98df2c799a..324382a0f7 100644 --- a/src/nvim/log.c +++ b/src/nvim/log.c @@ -22,7 +22,6 @@ #include "nvim/os/time.h" #define LOG_FILE_ENV "NVIM_LOG_FILE" -#define LOGERR "E5010: " /// Cached location of the expanded log file path decided by log_path_init(). static char log_file_path[MAXPATHL + 1] = { 0 }; @@ -73,15 +72,10 @@ static bool log_path_init(void) // Make kXDGCacheHome if it does not exist. char *cachehome = get_xdg_home(kXDGCacheHome); char *failed_dir = NULL; + bool log_dir_failure = false; if (!os_isdir((char_u *)cachehome)) { - int ret; - if ((ret = os_mkdir_recurse(cachehome, 0700, &failed_dir)) != 0) { - EMSG3(_(LOGERR "Failed to create directory %s " - "for writing logs: %s"), - failed_dir, os_strerror(ret)); - } + log_dir_failure = (os_mkdir_recurse(cachehome, 0700, &failed_dir) != 0); } - XFREE_CLEAR(failed_dir); XFREE_CLEAR(cachehome); // Invalid $NVIM_LOG_FILE or failed to expand; fall back to default. char *defaultpath = stdpaths_user_cache_subpath("log"); @@ -97,6 +91,11 @@ static bool log_path_init(void) return false; } os_setenv(LOG_FILE_ENV, log_file_path, true); + if (log_dir_failure) { + WLOG("Failed to create directory %s for writing logs: %s", + failed_dir, os_strerror(log_dir_failure)); + } + XFREE_CLEAR(failed_dir); } return true; } @@ -337,4 +336,3 @@ static bool v_do_log_to_file(FILE *log_file, int log_level, return true; } - |