From de45869e1dc28fa99951709b9485a35dbcea4444 Mon Sep 17 00:00:00 2001 From: "Shawn O. Pearce" Date: Mon, 4 Jan 2010 15:00:45 -0800 Subject: [PATCH] Fix RefWriter creation of info/refs to omit HEAD We really mean to omit HEAD here, but botched the difference between getOrigName and getName on the Ref object. We tested on the wrong value, picking up the target of the symbolic ref and therefore included it twice. Change-Id: If780c65166ccada2e63a4f42bbab752a56b16564 Signed-off-by: Shawn O. Pearce --- org.eclipse.jgit/src/org/eclipse/jgit/lib/RefWriter.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/org.eclipse.jgit/src/org/eclipse/jgit/lib/RefWriter.java b/org.eclipse.jgit/src/org/eclipse/jgit/lib/RefWriter.java index 704cbc052..c48449933 100644 --- a/org.eclipse.jgit/src/org/eclipse/jgit/lib/RefWriter.java +++ b/org.eclipse.jgit/src/org/eclipse/jgit/lib/RefWriter.java @@ -1,6 +1,6 @@ /* * Copyright (C) 2008, Charles O'Farrell - * Copyright (C) 2009, Google Inc. + * Copyright (C) 2009-2010, Google Inc. * Copyright (C) 2009, Robin Rosenberg * Copyright (C) 2008, Shawn O. Pearce * and other copyright owners as documented in the project's IP log. @@ -85,7 +85,7 @@ public void writeInfoRefs() throws IOException { final StringWriter w = new StringWriter(); final char[] tmp = new char[Constants.OBJECT_ID_STRING_LENGTH]; for (final Ref r : refs) { - if (Constants.HEAD.equals(r.getName())) { + if (Constants.HEAD.equals(r.getOrigName())) { // Historically HEAD has never been published through // the INFO_REFS file. This is a mistake, but its the // way things are.