On Mon, Apr 29, 2013 at 11:42 PM, Nguyễn Thái Ngọc Duy <pclouds@xxxxxxxxx> wrote: > This patch separates submodule odb sources from ordinary alternate > sources. The new sources can be accessed with ODB_EXTALT (e.g. via > read_sha1_file_extended). > > ODB_EXTALT is only added to odb_default in certain cases. Basically: > > - External commands do not access submodule odb by default > - unpack-objects, index-pack and rev-list do not > - All other builtin commands do > > unpack-objects, index-pack and rev-list take new objects from outside > and have to make sure the repository is still in good state. They > should not pay attention to submodule's odb, especially rev-list > because it does connectivity check. > > External commands also do not have default access to submodule odb, > simply because I see no reasons why the should. They don't usually s/the should/they should/ > play a big role in the user front, where submodule integration happens > and requires looking into submodule odb. > > The die() in add_submodule_odb() may be too strong. There might be a > use case where somebody wants to add_submodule_odb() and look some up > with read_sha1_file_extended() even if odb_default does not contain > ODB_EXTALT. Right now such a use case may need to work around die() by > temporarily adding ODB_EXTALT to odb_default. Not nice, but as no such s/as// perhaps? > use case exists yet to worry about. > > Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@xxxxxxxxx> -- 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