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