On Tue, May 23, 2017 at 2:55 AM, Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> wrote: > On Tue, May 23, 2017 at 11:17 AM, Johannes Schindelin > <Johannes.Schindelin@xxxxxx> wrote: >> Hi Ævar, >> >> On Mon, 22 May 2017, Ævar Arnfjörð Bjarmason wrote: >> >>> When I was adding the sha1collisiondetection submodule to git.git I >>> noticed that building git would dirty the submodule. >>> >>> This is because our own Makefile adds .depend/ directories. I hacked >>> around it by just getting the upstream project accept carrying an ignore >>> rule for that around: >>> https://github.com/cr-marcstevens/sha1collisiondetection/commit/e8397b26 >>> >>> A workaround for this is to have the Makefile add such a rule to >>> .git/modules/sha1collisiondetection/info/exclude, but that's less >>> convenient than being able to distribute it as a normal .gitignore rule. >>> >>> The submodule.<name>.ignore config provides an overly big hammer to >>> solve this, it would be better if we had something like >>> submodule.<name>.gitignore=<path>. Then we could have e.g. >>> .gitignore.sha1collisiondetection which would be added to whatever rules >>> the repo's own .gitignore provides. >> >> While I have nothing but the utmost respect for Stefan and Brandon for >> trying to improve submodules, maybe it would be a wiser idea to imitate >> the same strategy with sha1dc as we use with git-gui and gitk, i.e. >> perform a subtree merge instead of adding it as a submodule. It's not like >> 570kB will kill us. Actually that is a very valid bug report outside that series for the behavior of submodules. In a world where you use a submodule to track say a third party library, the current behavior of .gitignore applying to each repo makes sense. When it is no third party, but a first party lib, then it is sensible to expect that the building/testing infrastructure works across the whole repo set, and the user wants just one central place to specify things, such as ignoring certain files or applying .gitattributes. This topic came up in various forms on the mailing list, most often for config that ought to be applied across all repos[1]. That said I have no good idea yet how to fix this issue without introducing the ultimate user confusion. The conditional include of config files (by Duy as part of 2.13) seems like an interesting approach, which we could build on top of. We currently have a main config and a per-working-tree config, so I would expect we'd introduce another config file that is included by all submodules by default. It could be located in the superproject at ".git/config.super". This config file could then specify [submodule] recursiveIgnore = [yes/no] recursiveAttributes = [yes/no] In that way commands run from within the submodule as well as from the superproject would realize that the submodule needs to lookup the superproject and use the attribute/ignore/config settings from there. [1] Here the example for URL.insteadOf https://public-inbox.org/git/CAPZ477MCsBsfbqKzp69MT_brwz-0aes6twJofQrhizUBV7ZoeA@xxxxxxxxxxxxxx/ > > The submodule/.gitignore bug/feature-request being reported here isn't > something that impacts the ab/sha1dc series in practice. > > It was something I noticed while working with an earlier commit in > that repo, but that's a commit that'll never be pinned by the > git.git:sha1collisiondetection submodule. Thanks for the bug report. As outlined above, we'd still need to bikeshed how to fix it properly I'd think. Thanks, Stefan