github
/
jgit
mirror of https://gitee.com/github-mirror-repo/jgit.git
1
0
Fork 0
Code Issues Releases Wiki Activity
Browse Source

Kill GitIndex 

A few places were still using GitIndex. Replacing it was fairly
simple, but there is a difference in test outcome in
ReadTreeTest.testUntrackedConflicts. I believe the new behavior
is good, since we do not update neither the index, not the worktree.

Change-Id: I4be5357b7b3139dded17f77e07a140addb213ea7
Signed-off-by: Robin Rosenberg <robin.rosenberg@dewire.com>
stable-1.2
Robin Rosenberg 13 years ago committed by Christian Halstrick
parent
83c172f0f7
commit
6e9fdce9b9
4 changed files with 0 additions and 1592 deletions
Whitespace
Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL
Split View
Diff Options
Show Stats Download Patch File Download Diff File
  1. 481
      org.eclipse.jgit.test/exttst/org/eclipse/jgit/lib/T0007_GitIndexTest.java
  2. 1072
      org.eclipse.jgit/src/org/eclipse/jgit/lib/GitIndex.java
  3. 24
      org.eclipse.jgit/src/org/eclipse/jgit/lib/Repository.java
  4. 15
      org.eclipse.jgit/src/org/eclipse/jgit/lib/TreeEntry.java

481
org.eclipse.jgit.test/exttst/org/eclipse/jgit/lib/T0007_GitIndexTest.java
Unescape View File

@ -1,481 +0,0 @@
/*
* Copyright (C) 2007, Robin Rosenberg <robin.rosenberg@dewire.com>
* Copyright (C) 2008, Shawn O. Pearce <spearce@spearce.org>
* and other copyright owners as documented in the project's IP log.
*
* This program and the accompanying materials are made available
* under the terms of the Eclipse Distribution License v1.0 which
* accompanies this distribution, is reproduced below, and is
* available at http://www.eclipse.org/org/documents/edl-v10.php
*
* All rights reserved.
*
* Redistribution and use in source and binary forms, with or
* without modification, are permitted provided that the following
* conditions are met:
*
* - Redistributions of source code must retain the above copyright
* notice, this list of conditions and the following disclaimer.
*
* - Redistributions in binary form must reproduce the above
* copyright notice, this list of conditions and the following
* disclaimer in the documentation and/or other materials provided
* with the distribution.
*
* - Neither the name of the Eclipse Foundation, Inc. nor the
* names of its contributors may be used to endorse or promote
* products derived from this software without specific prior
* written permission.
*
* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND
* CONTRIBUTORS "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES,
* INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES
* OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
* ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR
* CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
* SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT
* NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
* LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
* CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT,
* STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
* ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF
* ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
*/
package org.eclipse.jgit.lib;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertFalse;
import static org.junit.Assert.assertNull;
import static org.junit.Assert.assertTrue;
import java.io.File;
import java.io.IOException;
import java.io.InputStream;
import java.lang.reflect.InvocationTargetException;
import java.lang.reflect.Method;
import org.eclipse.jgit.junit.LocalDiskRepositoryTestCase;
import org.eclipse.jgit.lib.GitIndex.Entry;
import org.eclipse.jgit.util.FS;
import org.junit.Before;
import org.junit.Test;
public class T0007_GitIndexTest extends LocalDiskRepositoryTestCase {
static boolean canrungitstatus;
static {
try {
canrungitstatus = system(new File("."),"git --version") == 0;
} catch (IOException e) {
System.out.println("Warning: cannot invoke native git to validate index");
} catch (InterruptedException e) {
e.printStackTrace();
}
}
private static int system(File dir, String cmd) throws IOException,
InterruptedException {
final Process process = Runtime.getRuntime().exec(cmd, null, dir);
new Thread() {
public void run() {
try {
InputStream s = process.getErrorStream();
for (int c = s.read(); c != -1; c = s.read()) {
System.err.print((char) c);
}
s.close();
} catch (IOException e1) {
// TODO Auto-generated catch block
e1.printStackTrace();
}
}
}.start();
final Thread t2 = new Thread() {
public void run() {
synchronized (this) {
try {
InputStream e = process.getInputStream();
for (int c = e.read(); c != -1; c = e.read()) {
System.out.print((char) c);
}
e.close();
} catch (IOException e1) {
// TODO Auto-generated catch block
e1.printStackTrace();
}
}
}
};
t2.start();
process.getOutputStream().close();
int ret = process.waitFor();
synchronized (t2) {
return ret;
}
}
private Repository db;
private File trash;
@Override
@Before
public void setUp() throws Exception {
super.setUp();
db = createWorkRepository();
trash = db.getWorkTree();
}
@Test
public void testCreateEmptyIndex() throws Exception {
GitIndex index = new GitIndex(db);
index.write();
// native git doesn't like an empty index
// assertEquals(0,system(trash,"git status"));
GitIndex indexr = new GitIndex(db);
indexr.read();
assertEquals(0, indexr.getMembers().length);
}
@Test
public void testReadWithNoIndex() throws Exception {
GitIndex index = new GitIndex(db);
index.read();
assertEquals(0, index.getMembers().length);
}
@Test
public void testCreateSimpleSortTestIndex() throws Exception {
GitIndex index = new GitIndex(db);
writeTrashFile("a/b", "data:a/b");
writeTrashFile("a:b", "data:a:b");
writeTrashFile("a.b", "data:a.b");
index.add(trash, new File(trash, "a/b"));
index.add(trash, new File(trash, "a:b"));
index.add(trash, new File(trash, "a.b"));
index.write();
assertEquals("a/b", index.getEntry("a/b").getName());
assertEquals("a:b", index.getEntry("a:b").getName());
assertEquals("a.b", index.getEntry("a.b").getName());
assertNull(index.getEntry("a*b"));
// Repeat test for re-read index
GitIndex indexr = new GitIndex(db);
indexr.read();
assertEquals("a/b", indexr.getEntry("a/b").getName());
assertEquals("a:b", indexr.getEntry("a:b").getName());
assertEquals("a.b", indexr.getEntry("a.b").getName());
assertNull(indexr.getEntry("a*b"));
if (canrungitstatus)
assertEquals(0, system(trash, "git status"));
}
@Test
public void testUpdateSimpleSortTestIndex() throws Exception {
GitIndex index = new GitIndex(db);
writeTrashFile("a/b", "data:a/b");
writeTrashFile("a:b", "data:a:b");
writeTrashFile("a.b", "data:a.b");
index.add(trash, new File(trash, "a/b"));
index.add(trash, new File(trash, "a:b"));
index.add(trash, new File(trash, "a.b"));
writeTrashFile("a/b", "data:a/b modified");
index.add(trash, new File(trash, "a/b"));
index.write();
if (canrungitstatus)
assertEquals(0, system(trash, "git status"));
}
@Test
public void testWriteTree() throws Exception {
GitIndex index = new GitIndex(db);
writeTrashFile("a/b", "data:a/b");
writeTrashFile("a:b", "data:a:b");
writeTrashFile("a.b", "data:a.b");
index.add(trash, new File(trash, "a/b"));
index.add(trash, new File(trash, "a:b"));
index.add(trash, new File(trash, "a.b"));
index.write();
ObjectId id = index.writeTree();
assertEquals("c696abc3ab8e091c665f49d00eb8919690b3aec3", id.name());
writeTrashFile("a/b", "data:a/b");
index.add(trash, new File(trash, "a/b"));
if (canrungitstatus)
assertEquals(0, system(trash, "git status"));
}
@Test
public void testReadTree() throws Exception {
// Prepare tree
GitIndex index = new GitIndex(db);
writeTrashFile("a/b", "data:a/b");
writeTrashFile("a:b", "data:a:b");
writeTrashFile("a.b", "data:a.b");
index.add(trash, new File(trash, "a/b"));
index.add(trash, new File(trash, "a:b"));
index.add(trash, new File(trash, "a.b"));
index.write();
ObjectId id = index.writeTree();
System.out.println("wrote id " + id);
assertEquals("c696abc3ab8e091c665f49d00eb8919690b3aec3", id.name());
GitIndex index2 = new GitIndex(db);
index2.readTree(mapTree("c696abc3ab8e091c665f49d00eb8919690b3aec3"));
Entry[] members = index2.getMembers();
assertEquals(3, members.length);
assertEquals("a.b", members[0].getName());
assertEquals("a/b", members[1].getName());
assertEquals("a:b", members[2].getName());
assertEquals(3, members.length);
GitIndex indexr = new GitIndex(db);
indexr.read();
Entry[] membersr = indexr.getMembers();
assertEquals(3, membersr.length);
assertEquals("a.b", membersr[0].getName());
assertEquals("a/b", membersr[1].getName());
assertEquals("a:b", membersr[2].getName());
assertEquals(3, membersr.length);
if (canrungitstatus)
assertEquals(0, system(trash, "git status"));
}
@Test
public void testReadTree2() throws Exception {
// Prepare a larger tree to test some odd cases in tree writing
GitIndex index = new GitIndex(db);
File f1 = writeTrashFile("a/a/a/a", "data:a/a/a/a");
File f2 = writeTrashFile("a/c/c", "data:a/c/c");
File f3 = writeTrashFile("a/b", "data:a/b");
File f4 = writeTrashFile("a:b", "data:a:b");
File f5 = writeTrashFile("a/d", "data:a/d");
File f6 = writeTrashFile("a.b", "data:a.b");
index.add(trash, f1);
index.add(trash, f2);
index.add(trash, f3);
index.add(trash, f4);
index.add(trash, f5);
index.add(trash, f6);
index.write();
ObjectId id = index.writeTree();
System.out.println("wrote id " + id);
assertEquals("ba78e065e2c261d4f7b8f42107588051e87e18e9", id.name());
GitIndex index2 = new GitIndex(db);
index2.readTree(mapTree("ba78e065e2c261d4f7b8f42107588051e87e18e9"));
Entry[] members = index2.getMembers();
assertEquals(6, members.length);
assertEquals("a.b", members[0].getName());
assertEquals("a/a/a/a", members[1].getName());
assertEquals("a/b", members[2].getName());
assertEquals("a/c/c", members[3].getName());
assertEquals("a/d", members[4].getName());
assertEquals("a:b", members[5].getName());
// reread and test
GitIndex indexr = new GitIndex(db);
indexr.read();
Entry[] membersr = indexr.getMembers();
assertEquals(6, membersr.length);
assertEquals("a.b", membersr[0].getName());
assertEquals("a/a/a/a", membersr[1].getName());
assertEquals("a/b", membersr[2].getName());
assertEquals("a/c/c", membersr[3].getName());
assertEquals("a/d", membersr[4].getName());
assertEquals("a:b", membersr[5].getName());
}
@Test
public void testDelete() throws Exception {
GitIndex index = new GitIndex(db);
writeTrashFile("a/b", "data:a/b");
writeTrashFile("a:b", "data:a:b");
writeTrashFile("a.b", "data:a.b");
index.add(trash, new File(trash, "a/b"));
index.add(trash, new File(trash, "a:b"));
index.add(trash, new File(trash, "a.b"));
index.write();
index.writeTree();
index.remove(trash, new File(trash, "a:b"));
index.write();
assertEquals("a.b", index.getMembers()[0].getName());
assertEquals("a/b", index.getMembers()[1].getName());
GitIndex indexr = new GitIndex(db);
indexr.read();
assertEquals("a.b", indexr.getMembers()[0].getName());
assertEquals("a/b", indexr.getMembers()[1].getName());
if (canrungitstatus)
assertEquals(0, system(trash, "git status"));
}
@Test
public void testCheckout() throws Exception {
// Prepare tree, remote it and checkout
GitIndex index = new GitIndex(db);
File aslashb = writeTrashFile("a/b", "data:a/b");
File acolonb = writeTrashFile("a:b", "data:a:b");
File adotb = writeTrashFile("a.b", "data:a.b");
index.add(trash, aslashb);
index.add(trash, acolonb);
index.add(trash, adotb);
index.write();
index.writeTree();
delete(aslashb);
delete(acolonb);
delete(adotb);
delete(aslashb.getParentFile());
GitIndex index2 = new GitIndex(db);
assertEquals(0, index2.getMembers().length);
index2.readTree(mapTree("c696abc3ab8e091c665f49d00eb8919690b3aec3"));
index2.checkout(trash);
assertEquals("data:a/b", read(aslashb));
assertEquals("data:a:b", read(acolonb));
assertEquals("data:a.b", read(adotb));
if (canrungitstatus)
assertEquals(0, system(trash, "git status"));
}
@Test
public void test030_executeBit_coreModeTrue() throws IllegalArgumentException, IllegalAccessException, InvocationTargetException, Error, Exception {
if (!FS.DETECTED.supportsExecute()) {
System.err.println("Test ignored since platform FS does not support the execute permission");
return;
}
try {
// coremode true is the default, typically set to false
// by git init (but not jgit!)
Method canExecute = File.class.getMethod("canExecute", (Class[])null);
Method setExecute = File.class.getMethod("setExecutable", new Class[] { Boolean.TYPE });
File execFile = writeTrashFile("exec","exec");
if (!((Boolean)setExecute.invoke(execFile, new Object[] { Boolean.TRUE })).booleanValue())
throw new Error("could not set execute bit on "+execFile.getAbsolutePath()+"for test");
File nonexecFile = writeTrashFile("nonexec","nonexec");
if (!((Boolean)setExecute.invoke(nonexecFile, new Object[] { Boolean.FALSE })).booleanValue())
throw new Error("could not clear execute bit on "+nonexecFile.getAbsolutePath()+"for test");
GitIndex index = new GitIndex(db);
index.filemode = Boolean.TRUE; // TODO: we need a way to set this using config
index.add(trash, execFile);
index.add(trash, nonexecFile);
Tree tree = mapTree(index.writeTree().name());
assertEquals(FileMode.EXECUTABLE_FILE, tree.findBlobMember(execFile.getName()).getMode());
assertEquals(FileMode.REGULAR_FILE, tree.findBlobMember(nonexecFile.getName()).getMode());
index.write();
if (!execFile.delete())
throw new Error("Problem in test, cannot delete test file "+execFile.getAbsolutePath());
if (!nonexecFile.delete())
throw new Error("Problem in test, cannot delete test file "+nonexecFile.getAbsolutePath());
GitIndex index2 = new GitIndex(db);
index2.filemode = Boolean.TRUE; // TODO: we need a way to set this using config
index2.read();
index2.checkout(trash);
assertTrue(((Boolean)canExecute.invoke(execFile,(Object[])null)).booleanValue());
assertFalse(((Boolean)canExecute.invoke(nonexecFile,(Object[])null)).booleanValue());
assertFalse(index2.getEntry(execFile.getName()).isModified(trash));
assertFalse(index2.getEntry(nonexecFile.getName()).isModified(trash));
if (!((Boolean)setExecute.invoke(execFile, new Object[] { Boolean.FALSE })).booleanValue())
throw new Error("could not clear set execute bit on "+execFile.getAbsolutePath()+"for test");
if (!((Boolean)setExecute.invoke(nonexecFile, new Object[] { Boolean.TRUE })).booleanValue())
throw new Error("could set execute bit on "+nonexecFile.getAbsolutePath()+"for test");
assertTrue(index2.getEntry(execFile.getName()).isModified(trash));
assertTrue(index2.getEntry(nonexecFile.getName()).isModified(trash));
} catch (NoSuchMethodException e) {
System.err.println("Test ignored when running under JDK < 1.6");
return;
}
}
@Test
public void test031_executeBit_coreModeFalse() throws IllegalArgumentException, IllegalAccessException, InvocationTargetException, Error, Exception {
if (!FS.DETECTED.supportsExecute()) {
System.err.println("Test ignored since platform FS does not support the execute permission");
return;
}
try {
// coremode true is the default, typically set to false
// by git init (but not jgit!)
Method canExecute = File.class.getMethod("canExecute", (Class[])null);
Method setExecute = File.class.getMethod("setExecutable", new Class[] { Boolean.TYPE });
File execFile = writeTrashFile("exec","exec");
if (!((Boolean)setExecute.invoke(execFile, new Object[] { Boolean.TRUE })).booleanValue())
throw new Error("could not set execute bit on "+execFile.getAbsolutePath()+"for test");
File nonexecFile = writeTrashFile("nonexec","nonexec");
if (!((Boolean)setExecute.invoke(nonexecFile, new Object[] { Boolean.FALSE })).booleanValue())
throw new Error("could not clear execute bit on "+nonexecFile.getAbsolutePath()+"for test");
GitIndex index = new GitIndex(db);
index.filemode = Boolean.FALSE; // TODO: we need a way to set this using config
index.add(trash, execFile);
index.add(trash, nonexecFile);
Tree tree = mapTree(index.writeTree().name());
assertEquals(FileMode.REGULAR_FILE, tree.findBlobMember(execFile.getName()).getMode());
assertEquals(FileMode.REGULAR_FILE, tree.findBlobMember(nonexecFile.getName()).getMode());
index.write();
if (!execFile.delete())
throw new Error("Problem in test, cannot delete test file "+execFile.getAbsolutePath());
if (!nonexecFile.delete())
throw new Error("Problem in test, cannot delete test file "+nonexecFile.getAbsolutePath());
GitIndex index2 = new GitIndex(db);
index2.filemode = Boolean.FALSE; // TODO: we need a way to set this using config
index2.read();
index2.checkout(trash);
assertFalse(((Boolean)canExecute.invoke(execFile,(Object[])null)).booleanValue());
assertFalse(((Boolean)canExecute.invoke(nonexecFile,(Object[])null)).booleanValue());
assertFalse(index2.getEntry(execFile.getName()).isModified(trash));
assertFalse(index2.getEntry(nonexecFile.getName()).isModified(trash));
if (!((Boolean)setExecute.invoke(execFile, new Object[] { Boolean.FALSE })).booleanValue())
throw new Error("could not clear set execute bit on "+execFile.getAbsolutePath()+"for test");
if (!((Boolean)setExecute.invoke(nonexecFile, new Object[] { Boolean.TRUE })).booleanValue())
throw new Error("could set execute bit on "+nonexecFile.getAbsolutePath()+"for test");
// no change since we ignore the execute bit
assertFalse(index2.getEntry(execFile.getName()).isModified(trash));
assertFalse(index2.getEntry(nonexecFile.getName()).isModified(trash));
} catch (NoSuchMethodException e) {
System.err.println("Test ignored when running under JDK < 1.6");
return;
}
}
private void delete(File f) throws IOException {
if (!f.delete())
throw new IOException("Failed to delete f");
}
private File writeTrashFile(String name, String body) throws IOException {
final File path = new File(trash, name);
write(path, body);
return path;
}
private Tree mapTree(String name) throws IOException {
ObjectId id = db.resolve(name + "^{tree}");
return new Tree(db, id, db.open(id).getCachedBytes());
}
}

1072
org.eclipse.jgit/src/org/eclipse/jgit/lib/GitIndex.java
View File

File diff suppressed because it is too large Load Diff

24
org.eclipse.jgit/src/org/eclipse/jgit/lib/Repository.java
Unescape View File

@ -109,8 +109,6 @@ public abstract class Repository {
/** File abstraction used to resolve paths. */
private final FS fs;
private GitIndex index;
private final ListenerList myListeners = new ListenerList();
/** If not bare, the top level directory of the working files. */
@ -808,28 +806,6 @@ public abstract class Repository {
return ret;
}
/**
* @return a representation of the index associated with this
* {@link Repository}
* @throws IOException
* if the index can not be read
* @throws NoWorkTreeException
* if this is bare, which implies it has no working directory.
* See {@link #isBare()}.
* @deprecated Use {@link #readDirCache()} instead
*/
public GitIndex getIndex() throws IOException, NoWorkTreeException {
if (isBare())
throw new NoWorkTreeException();
if (index == null) {
index = new GitIndex(this);
index.read();
} else {
index.rereadIfNecessary();
}
return index;
}
/**
* @return the index file location
* @throws NoWorkTreeException

15
org.eclipse.jgit/src/org/eclipse/jgit/lib/TreeEntry.java
Unescape View File

@ -46,7 +46,6 @@ package org.eclipse.jgit.lib;
import java.io.IOException;
import org.eclipse.jgit.lib.GitIndex.Entry;
import org.eclipse.jgit.util.RawParseUtils;
/**
@ -236,20 +235,6 @@ public abstract class TreeEntry implements Comparable {
return '/';
}
/**
* Helper for accessing tree/blob/index methods.
*
* @param i
* @return '/' for Tree entries and NUL for non-treeish objects
* @deprecated since it depends on deprecated GitIndex, and internal
*/
final public static int lastChar(Entry i) {
// FIXME, gitlink etc. Currently Trees cannot appear in the
// index so '\0' is always returned, except maybe for submodules
// which we do not support yet.
return FileMode.TREE.equals(i.getModeBits()) ? '/' : '\0';
}
/**
* @return mode (type of object)
*/

Write Preview
Loading…
Cancel
Save