From 69998b0c6ff724bf620480140ccce648fec7d6a9 Mon Sep 17 00:00:00 2001 From: Che-Liang Chiou Date: Wed, 11 Jan 2012 11:28:42 +0800 Subject: Represent git-submodule as nested projects We need a representation of git-submodule in repo; otherwise repo will not sync submodules, and leave workspace in a broken state. Of course this will not be a problem if all projects are owned by the owner of the manifest file, who may simply choose not to use git-submodule in all projects. However, this is not possible in practice because manifest file owner is unlikely to own all upstream projects. As git submodules are simply git repositories, it is natural to treat them as plain repo projects that live inside a repo project. That is, we could use recursively declared projects to denote the is-submodule relation of git repositories. The behavior of repo remains the same to projects that do not have a sub-project within. As for parent projects, repo fetches them and their sub-projects as normal projects, and then checks out subprojects at the commit specified in parent's commit object. The sub-project is fetched at a path relative to parent project's working directory; so the path specified in manifest file should match that of .gitmodules file. If a submodule is not registered in repo manifest, repo will derive its properties from itself and its parent project, which might not always be correct. In such cases, the subproject is called a derived subproject. To a user, a sub-project is merely a git-submodule; so all tips of working with a git-submodule apply here, too. For example, you should not run `repo sync` in a parent repository if its submodule is dirty. Change-Id: I541e9e2ac1a70304272dbe09724572aa1004eb5c --- subcmds/sync.py | 21 ++++++++++++++++++++- 1 file changed, 20 insertions(+), 1 deletion(-) (limited to 'subcmds/sync.py') diff --git a/subcmds/sync.py b/subcmds/sync.py index e68a025e..90e2908f 100644 --- a/subcmds/sync.py +++ b/subcmds/sync.py @@ -503,12 +503,31 @@ uncommitted changes are present' % project.relpath to_fetch.append(rp) to_fetch.extend(all_projects) - self._Fetch(to_fetch, opt) + fetched = self._Fetch(to_fetch, opt) _PostRepoFetch(rp, opt.no_repo_verify) if opt.network_only: # bail out now; the rest touches the working tree return + # Iteratively fetch missing and/or nested unregistered submodules + previously_missing_set = set() + while True: + self.manifest._Unload() + all = self.GetProjects(args, missing_ok=True) + missing = [] + for project in all: + if project.gitdir not in fetched: + missing.append(project) + if not missing: + break + # Stop us from non-stopped fetching actually-missing repos: If set of + # missing repos has not been changed from last fetch, we break. + missing_set = set(p.name for p in missing) + if previously_missing_set == missing_set: + break + previously_missing_set = missing_set + fetched.update(self._Fetch(missing, opt)) + if self.manifest.IsMirror: # bail out now, we have no working tree return -- cgit v1.2.3-54-g00ecf