Em qui, 14 de jul de 2022 18:27, Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> escreveu: > > > Here's a cleaned up version of what I have, which I figure is probably > better linked-to than contributing to my E-Mail quota :): > > https://github.com/git/git/compare/master...avar:git:avar/doc-config-includes Thanks for sharing your version! > The one thing I'd like you to reconsider is to drop the idea of adding > these "ifndef::git-grep[]" defines and the like. In your version it > yields an arguably better result. > > But I think what we should be going for is the more general direction > outlined above, at which point that becomes quite a mess of > ifdefs. I.e. config/gc/rerere.txt would need to know what it's going to > get include in, which would be N number of manpages in the genreal case, > not just "main or config" as this series leaves it. > > I think the solution I have to that in 1/9 in that first series is a > better trade-off, i.e. we just (eventually, your series doesn't need to > do that) include some standard wording saying that what you're looking > at in git-CMD(1) is transcluded as-is from the relevant part of > git-config(1). I.e.: > > Everything below this line in this section is selectively included > from the linkgit:git-config[1] documentation. The content is the same > as what's found there: > > What do you think about doing that instead? I like the includes/* idea, and I agree that it is a more sensible way forward than the many 'ifndef[]::git-cmd.txt's :) Your linked changes also cover a wider range of cmds than my series does. So I'd be happy to have them as a replacement to this series.