From 2444aa231a3a9618a413b7e42e8120b42258702d Mon Sep 17 00:00:00 2001 From: Dave Borowitz Date: Mon, 25 Jun 2012 10:46:36 -0700 Subject: [PATCH] Use the working tree's .gitmodules in SubmoduleWalk.forIndex() This was broken in fe1f1b8f8aba60fdd1ad6f0f72e9c9180978cc60, which preferred the index over the working tree when both were present. Change-Id: I97dcf9a088adcbd0187fa7eec9ef34445ce3a981 Signed-off-by: Kevin Sawicki --- .../jgit/submodule/SubmoduleWalkTest.java | 10 +++---- .../eclipse/jgit/submodule/SubmoduleWalk.java | 28 ++++++------------- 2 files changed, 14 insertions(+), 24 deletions(-) diff --git a/org.eclipse.jgit.test/tst/org/eclipse/jgit/submodule/SubmoduleWalkTest.java b/org.eclipse.jgit.test/tst/org/eclipse/jgit/submodule/SubmoduleWalkTest.java index 0669dd199..cb3c7d88f 100644 --- a/org.eclipse.jgit.test/tst/org/eclipse/jgit/submodule/SubmoduleWalkTest.java +++ b/org.eclipse.jgit.test/tst/org/eclipse/jgit/submodule/SubmoduleWalkTest.java @@ -299,13 +299,13 @@ public void indexWithGitmodules() throws Exception { final Config gitmodules = new Config(); gitmodules.setString(CONFIG_SUBMODULE_SECTION, path, CONFIG_KEY_PATH, "sub"); - gitmodules.setString(CONFIG_SUBMODULE_SECTION, path, CONFIG_KEY_URL, - "git://example.com/sub"); - final RevBlob gitmodulesBlob = testDb.blob(gitmodules.toText()); - - // Different config in the working tree. + // Different config in the index should be overridden by the working tree. gitmodules.setString(CONFIG_SUBMODULE_SECTION, path, CONFIG_KEY_URL, "git://example.com/bad"); + final RevBlob gitmodulesBlob = testDb.blob(gitmodules.toText()); + + gitmodules.setString(CONFIG_SUBMODULE_SECTION, path, CONFIG_KEY_URL, + "git://example.com/sub"); writeTrashFile(DOT_GIT_MODULES, gitmodules.toText()); DirCache cache = db.lockDirCache(); diff --git a/org.eclipse.jgit/src/org/eclipse/jgit/submodule/SubmoduleWalk.java b/org.eclipse.jgit/src/org/eclipse/jgit/submodule/SubmoduleWalk.java index 323965f7c..e8022b69d 100644 --- a/org.eclipse.jgit/src/org/eclipse/jgit/submodule/SubmoduleWalk.java +++ b/org.eclipse.jgit/src/org/eclipse/jgit/submodule/SubmoduleWalk.java @@ -94,8 +94,6 @@ public static SubmoduleWalk forIndex(Repository repository) try { DirCache index = repository.readDirCache(); generator.setTree(new DirCacheIterator(index)); - generator.setRootTree(new DirCacheIterator(index)); - generator.useWorkingTree = true; } catch (IOException e) { generator.release(); throw e; @@ -304,8 +302,6 @@ else if (submoduleUrl.startsWith("../")) { private String path; - private boolean useWorkingTree; - /** * Create submodule generator * @@ -388,7 +384,14 @@ public SubmoduleWalk setRootTree(final AnyObjectId id) throws IOException { * @throws ConfigInvalidException */ public SubmoduleWalk loadModulesConfig() throws IOException, ConfigInvalidException { - if (rootTree != null) { + if (rootTree == null) { + File modulesFile = new File(repository.getWorkTree(), + Constants.DOT_GIT_MODULES); + FileBasedConfig config = new FileBasedConfig(modulesFile, + repository.getFS()); + config.load(); + modulesConfig = config; + } else { TreeWalk configWalk = new TreeWalk(repository); try { configWalk.addTree(rootTree); @@ -411,10 +414,7 @@ public SubmoduleWalk loadModulesConfig() throws IOException, ConfigInvalidExcept return this; } } - if (!useWorkingTree) { - modulesConfig = new Config(); - return this; - } + modulesConfig = new Config(); } finally { if (idx > 0) rootTree.next(idx); @@ -423,16 +423,6 @@ public SubmoduleWalk loadModulesConfig() throws IOException, ConfigInvalidExcept configWalk.release(); } } - if (repository.isBare()) { - modulesConfig = new Config(); - } else { - File modulesFile = new File(repository.getWorkTree(), - Constants.DOT_GIT_MODULES); - FileBasedConfig config = new FileBasedConfig(modulesFile, - repository.getFS()); - config.load(); - modulesConfig = config; - } return this; }