On Sun, Jan 18, 2009 at 22:55, Johannes Schindelin <Johannes.Schindelin@xxxxxx> wrote: > Hi, > > On Sun, 18 Jan 2009, Lars Hjemli wrote: > >> So, would you rather have something like `git archive --submodules=foo >> --submodules=bar HEAD` to explicitly tell which submodule paths to >> include in the archive when executed in a bare repo? > > That does not quite say what you tried to do, does it? You tried to > traverse submodules whose commit can be found in the object database. > > Setting aside the fact that we usually try to avoid accessing unreachable > objects, which your handling does not do, our "git submodule" does not do > that either; it only handles submodules that are checked out. > > Now, this behavior might be wanted, in bare as well as in non-bare > repositories, but I think it should be triggered by an option, such as > "--submodules=look-in-superprojects-odb". Sorry, but if your concern is whether to traverse a submodule in a bare repo when the submodule isn't checked out (yeah, contradiction in terms), I just don't see the point. For non-bare repositories the policy has always been to ignore submodules which isn't checked out, but for bare repositories there is no obvious way (for me, at least) to apply the same policy. Therefore I proposed to traverse all submodules where the linked commit is reachable, but as you pointed out this would be wrong for non-bare repositories. I then modified my proposal to include a check on is_bare_repository(): If we're not in a bare repository, read_tree_recursive() is only allowed to recurse into checked out submodules. But if we're in a bare repository, read_tree_recursive() is allowed to recurse into any submodule with a reachable commit. Now then, if `--submodules=look-in-superprojects-odb` should be required to trigger the latter behavior, running `git archive --submodules HEAD` in a bare repository would always produce identical output as `git archive HEAD` and this is why I don't understand the gain of 'look-in-superprojects-odb' (I thought you wanted to limit which of the reachable submodules to recurse into). -- larsh -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
