# [PATCH 1/2] gitignore.txt: clarify default core.excludesfile path 4 messages from 2018-06-27 to 2018-06-27. Participants: Todd Zullinger, Junio C Hamano. Thread: https://gitlist.dev/t/48785 ## Todd Zullinger, 2018-06-27 04:46 Subject: [PATCH 1/2] gitignore.txt: clarify default core.excludesfile path Message-ID: <20180627044652.12080-1-tmz@pobox.com> URL: https://gitlist.dev/e/20180627044652.12080-1-tmz%40pobox.com ``` The default core.excludesfile path is $XDG_CONFIG_HOME/git/ignore. $HOME/.config/git/ignore is used if XDG_CONFIG_HOME is empty or unset, as described later in the document. Signed-off-by: Todd Zullinger --- Documentation/gitignore.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/gitignore.txt b/Documentation/gitignore.txt index ff5d7f9ed6..d107daaffd 100644 --- a/Documentation/gitignore.txt +++ b/Documentation/gitignore.txt @@ -7,7 +7,7 @@ gitignore - Specifies intentionally untracked files to ignore SYNOPSIS -------- -$HOME/.config/git/ignore, $GIT_DIR/info/exclude, .gitignore +$XDG_CONFIG_HOME/git/ignore, $GIT_DIR/info/exclude, .gitignore DESCRIPTION ----------- -- 2.18.0 ``` ## Todd Zullinger, 2018-06-27 04:46 Subject: [PATCH 2/2] dir.c: fix typos in core.excludesfile comment Message-ID: <20180627044652.12080-2-tmz@pobox.com> URL: https://gitlist.dev/e/20180627044652.12080-2-tmz%40pobox.com In-Reply-To: <20180627044652.12080-1-tmz@pobox.com> ``` Make it easier to find references to core.excludesfile and the default $XDG_CONFIG_HOME/git/ignore path. Signed-off-by: Todd Zullinger --- I noticed the typo in core.excludesfile and $XDG_CONFIG_HOME while I was verifing the previous change to clarify the documentation matched the code. Fixing these minor issues in the comments will hopefully make it easier for others to find the right places in the code in the future. dir.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dir.c b/dir.c index fe9bf58e4c..363a4837ae 100644 --- a/dir.c +++ b/dir.c @@ -2497,7 +2497,7 @@ void setup_standard_excludes(struct dir_struct *dir) { dir->exclude_per_dir = ".gitignore"; - /* core.excludefile defaulting to $XDG_HOME/git/ignore */ + /* core.excludesfile defaulting to $XDG_CONFIG_HOME/git/ignore */ if (!excludes_file) excludes_file = xdg_config_home("ignore"); if (excludes_file && !access_or_warn(excludes_file, R_OK, 0)) -- 2.18.0 ``` ## Junio C Hamano, 2018-06-27 19:16 Subject: Re: [PATCH 1/2] gitignore.txt: clarify default core.excludesfile path Message-ID: URL: https://gitlist.dev/e/xmqqvaa4dopx.fsf%40gitster-ct.c.googlers.com In-Reply-To: <20180627044652.12080-1-tmz@pobox.com> ``` Todd Zullinger writes: > The default core.excludesfile path is $XDG_CONFIG_HOME/git/ignore. > $HOME/.config/git/ignore is used if XDG_CONFIG_HOME is empty or unset, ... because $HOME/.config is the default value for XDG_CONFIG_HOME when it is unset, that is? If so, the change makes sense. > as described later in the document. > Signed-off-by: Todd Zullinger > --- > Documentation/gitignore.txt | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Documentation/gitignore.txt b/Documentation/gitignore.txt > index ff5d7f9ed6..d107daaffd 100644 > --- a/Documentation/gitignore.txt > +++ b/Documentation/gitignore.txt > @@ -7,7 +7,7 @@ gitignore - Specifies intentionally untracked files to ignore > > SYNOPSIS > -------- > -$HOME/.config/git/ignore, $GIT_DIR/info/exclude, .gitignore > +$XDG_CONFIG_HOME/git/ignore, $GIT_DIR/info/exclude, .gitignore > > DESCRIPTION > ----------- ``` ## Todd Zullinger, 2018-06-27 20:07 Subject: Re: [PATCH 1/2] gitignore.txt: clarify default core.excludesfile path Message-ID: <20180627200743.GL20217@zaya.teonanacatl.net> URL: https://gitlist.dev/e/20180627200743.GL20217%40zaya.teonanacatl.net In-Reply-To: ``` Junio C Hamano wrote: > Todd Zullinger writes: > >> The default core.excludesfile path is $XDG_CONFIG_HOME/git/ignore. >> $HOME/.config/git/ignore is used if XDG_CONFIG_HOME is empty or unset, > > ... because $HOME/.config is the default value for XDG_CONFIG_HOME > when it is unset, that is? If so, the change makes sense. Indeed, that's the fallback path. Thanks, -- Todd ```