Skip to content

Commit

Permalink
Merge pull request #3773 from dscho/tone-down-fsmonitor-warning
Browse files Browse the repository at this point in the history
Try to advise about core.useBuiltinFSMonitor's deprecation only once
  • Loading branch information
dscho authored Apr 7, 2022
2 parents c61dcf9 + 80da228 commit 8c57cba
Showing 1 changed file with 6 additions and 3 deletions.
9 changes: 6 additions & 3 deletions fsmonitor-settings.c
Original file line number Diff line number Diff line change
Expand Up @@ -59,9 +59,12 @@ static int check_deprecated_builtin_config(struct repository *r)
* the check resulted the config being set by this (deprecated) setting.
*/
if(!repo_config_get_bool(r, "core.useBuiltinFSMonitor", &core_use_builtin_fsmonitor)) {
advise_if_enabled(ADVICE_USE_CORE_FSMONITOR_CONFIG,
_("core.useBuiltinFSMonitor will be deprecated "
"soon; use core.fsmonitor instead"));
if (!git_env_bool("GIT_SUPPRESS_USEBUILTINFSMONITOR_ADVICE", 0)) {
advise_if_enabled(ADVICE_USE_CORE_FSMONITOR_CONFIG,
_("core.useBuiltinFSMonitor will be deprecated "
"soon; use core.fsmonitor instead"));
setenv("GIT_SUPPRESS_USEBUILTINFSMONITOR_ADVICE", "1", 1);
}
if (core_use_builtin_fsmonitor) {
fsm_settings__set_ipc(r);
return 1;
Expand Down

0 comments on commit 8c57cba

Please sign in to comment.