On Sat, 23 Feb 2019, Johannes Schindelin wrote: > Hi, > > On Sat, 23 Feb 2019, Johannes Schindelin wrote: > > > On Sat, 23 Feb 2019, Junio C Hamano wrote: > > > > > "Robert P. J. Day" <rpjday@xxxxxxxxxxxxxx> writes: > > > > > > > am i misreading something? and if not, is there a reason git clean > > > > does not consult core.excludesFile? > > > > > > Can you ask "git log" and "git blame" whch of core.excludesFile and > > > "clean -x" features came earlier and by how big a difference? > > > > Or maybe we can have a look why the `core.excludesfile` regression test > > case in t7300 does not catch this? > > > > https://github.com/git/git/blob/v2.21.0-rc2/t/t7300-clean.sh#L408-L417 > > I actually doubt that `git clean` ignores `core.excludesFile`: in > https://github.com/git/git/blob/v2.21.0-rc2/config.c#L1297-L1298, > `git_default_core_config()` (which is called via the `git_clean_config()` > -> `git_color_default_config()` -> `git_default_config()` chain from > `cmd_clean()`) does interpret `core.excludesFile`: > > if (!strcmp(var, "core.excludesfile")) > return git_config_pathname(&excludes_file, var, value); > > Then, `cmd_clean()` goes on to parse the options, setting the `ignored` > variable upon `-x` and then doing > [this](https://github.com/git/git/blob/v2.21.0-rc2/builtin/clean.c#L957-L958): > > if (!ignored) > setup_standard_excludes(&dir); > > This function specifically looks at `excludes_file` in > https://github.com/git/git/blob/v2.21.0-rc2/dir.c#L2481-L2483: > > if (excludes_file && !access_or_warn(excludes_file, R_OK, 0)) > add_excludes_from_file_1(dir, excludes_file, > dir->untracked ? &dir->ss_excludes_file : NULL); > > So I am quite puzzled by the claim that `git clean` might not consult > `core.excludesFile`. > > Robert, care to come up with an example demonstrating where it does not? sorry i wasn't clear, all i was pointing out was that "man git-clean" *explicitly* mentioned two locations related to cleaning: -x Donʼt use the standard ignore rules read from .gitignore (per directory) and $GIT_DIR/info/exclude, ... without additionally *explicitly* mentioning core.excludesFile. if the man page simply said something like, "using the standard ignore processing" and left it at that, it would be fine, but to list two of the locations without the third is potentially confusing. rday -- ======================================================================== Robert P. J. Day Ottawa, Ontario, CANADA http://crashcourse.ca/dokuwiki Twitter: http://twitter.com/rpjday LinkedIn: http://ca.linkedin.com/in/rpjday ========================================================================
