On 08/10, Stefan Beller wrote:
> > > + cfg_file = xstrfmt("%s/config", subrepo.gitdir);
> >
> > As I mentioned here:
> > https://public-inbox.org/git/20180807230637.247200-1-bmwill@xxxxxxxxxx/T/#t
> >
> > This lines should probably be more like:
> >
> > cfg_file = repo_git_path(&subrepo, "config");
> >
>
> Why? You did not mention the benefits for writing it this way
> here or on the reference. Care to elaborate?
Its more future proof especially because we have the difference bettwen
commondir and gitdir for worktrees. Using the "repo_git_path" function
handles path rewritting when using worktrees. Here (when working with
worktrees) "subrepo.gitdir" refers to the worktree specific gitdir while
"subrepo.commondir" refers to the global common gitdir where the
repository config actually lives.
--
Brandon Williams
