Browse Source

Fix StashApplyCommand for stashes containing untracked changes.

If there are untracked changes, apply only the untracked tree
after a successful merge. The merge tree from merging untracked
with HEAD would also contain files already reset before (changes
in tracked files) and try to reset those again,leading to false
checkout conflicts.

Bug: 505804
Change-Id: Iaced4d277623334d11e3d1cca5969590d7c5093e
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>
stable-4.7
Thomas Wolf 8 years ago
parent
commit
46af7192a2
  1. 17
      org.eclipse.jgit.test/tst/org/eclipse/jgit/api/StashApplyCommandTest.java
  2. 10
      org.eclipse.jgit/src/org/eclipse/jgit/api/StashApplyCommand.java

17
org.eclipse.jgit.test/tst/org/eclipse/jgit/api/StashApplyCommandTest.java

@ -736,4 +736,21 @@ public class StashApplyCommandTest extends RepositoryTestCase {
} }
assertEquals("working-directory", read(path)); assertEquals("working-directory", read(path));
} }
@Test
public void untrackedAndTrackedChanges() throws Exception {
writeTrashFile(PATH, "changed");
String path = "untracked.txt";
writeTrashFile(path, "untracked");
git.stashCreate().setIncludeUntracked(true).call();
assertTrue(PATH + " should exist", check(PATH));
assertEquals(PATH + " should have been reset", "content", read(PATH));
assertFalse(path + " should not exist", check(path));
git.stashApply().setStashRef("stash@{0}").call();
assertTrue(PATH + " should exist", check(PATH));
assertEquals(PATH + " should have new content", "changed", read(PATH));
assertTrue(path + " should exist", check(path));
assertEquals(path + " should have new content", "untracked",
read(path));
}
} }

10
org.eclipse.jgit/src/org/eclipse/jgit/api/StashApplyCommand.java

@ -232,20 +232,20 @@ public class StashApplyCommand extends GitCommand<ObjectId> {
untrackedMerger.setBase(null); untrackedMerger.setBase(null);
boolean ok = untrackedMerger.merge(headCommit, boolean ok = untrackedMerger.merge(headCommit,
untrackedCommit); untrackedCommit);
if (ok) if (ok) {
try { try {
RevTree untrackedTree = revWalk RevTree untrackedTree = revWalk
.parseTree(untrackedMerger .parseTree(untrackedCommit);
.getResultTreeId());
resetUntracked(untrackedTree); resetUntracked(untrackedTree);
} catch (CheckoutConflictException e) { } catch (CheckoutConflictException e) {
throw new StashApplyFailureException( throw new StashApplyFailureException(
JGitText.get().stashApplyConflict); JGitText.get().stashApplyConflict, e);
} }
else } else {
throw new StashApplyFailureException( throw new StashApplyFailureException(
JGitText.get().stashApplyConflict); JGitText.get().stashApplyConflict);
} }
}
} else { } else {
throw new StashApplyFailureException( throw new StashApplyFailureException(
JGitText.get().stashApplyConflict); JGitText.get().stashApplyConflict);

Loading…
Cancel
Save