# Re: [PATCH] Oops, get the commit first, then the tree. 2 messages from 2006-12-03 to 2006-12-03. Participants: Shawn Pearce, Robin Rosenberg. Thread: https://gitlist.dev/t/43387 ## Robin Rosenberg, 2006-12-03 00:45 Subject: [PATCH] Oops, get the commit first, then the tree. Message-ID: <200612030145.45567.robin.rosenberg@dewire.com> URL: https://gitlist.dev/e/200612030145.45567.robin.rosenberg%40dewire.com ``` This made the eclipse plugin unable to connect to Git. Signed-off-by: Robin Rosenberg --- .../src/org/spearce/jgit/lib/Repository.java | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/org.spearce.jgit/src/org/spearce/jgit/lib/Repository.java b/org.spearce.jgit/src/org/spearce/jgit/lib/Repository.java index bfdcd3e..2344817 100644 --- a/org.spearce.jgit/src/org/spearce/jgit/lib/Repository.java +++ b/org.spearce.jgit/src/org/spearce/jgit/lib/Repository.java @@ -168,7 +168,8 @@ public class Repository { } public Tree mapTree(final String revstr) throws IOException { - final ObjectId id = resolve(revstr); + Commit commit = mapCommit(revstr); + final ObjectId id = commit.getTreeId(); return id != null ? mapTree(id) : null; } -- 1.4.4.gf05d ``` ## Shawn Pearce, 2006-12-03 02:34 Subject: Re: [PATCH] Oops, get the commit first, then the tree. Message-ID: <20061203023407.GC26668@spearce.org> URL: https://gitlist.dev/e/20061203023407.GC26668%40spearce.org In-Reply-To: <200612030145.45567.robin.rosenberg@dewire.com> ``` Robin Rosenberg wrote: > This made the eclipse plugin unable to connect to Git. ... > --- a/org.spearce.jgit/src/org/spearce/jgit/lib/Repository.java > +++ b/org.spearce.jgit/src/org/spearce/jgit/lib/Repository.java > @@ -168,7 +168,8 @@ public class Repository { > } > > public Tree mapTree(final String revstr) throws IOException { > - final ObjectId id = resolve(revstr); > + Commit commit = mapCommit(revstr); > + final ObjectId id = commit.getTreeId(); > return id != null ? mapTree(id) : null; I don't want to apply the fix above because mapTree(String) should be able to take an arbitrary SHA1 expression and return the Tree object corresponding to that. Your change only permits commits to be resolved; but if the revstr resolved to a tag we'd need to peel back the tag to get to the tree. Ideally that should be done in mapTree(ObjectId), based on the type of thing ObjectId points to. I think the bug was caused by a recent breakage in Repository.mapTree (42691339). I fixed it in 020e0ee but neglected to push it out. Actually I'm surprised mapTree(ObjectId) even works before 020e0ee; I just don't see how its right... -- ```