opendev/gerrit
Code Issues Proposed changes

Move ConsoleUI to init package

Browse Source 
It is awkward to have the init-api rule depend on exactly one class in
a non-init package. This is a backwards incompatible change to the
plugin API, but it's trivial for plugin authors to update.

Change-Id: I8a6fcfdaeb0b509f05af971f9fdbdb270fe65fb6
This commit is contained in:
Dave Borowitz
2014-07-28 11:12:34 -07:00
parent 4fbdf1e0ee
commit d2cf6d4dc6
21 changed files with 6 additions and 21 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
gerrit-pgm/src/test/java/com/google/gerrit/pgm/init/LibrariesTest.java
View File

@@ -18,10 +18,11 @@ import static org.easymock.EasyMock.createStrictMock;
import static org.easymock.EasyMock.replay;
import static org.easymock.EasyMock.verify;
import com.google.gerrit.pgm.util.ConsoleUI;
import com.google.gerrit.server.config.SitePaths;
import com.google.inject.Provider;
import org.junit.Test;
import static org.junit.Assert.assertNotNull;
import java.io.File;

1
gerrit-pgm/src/test/java/com/google/gerrit/pgm/init/UpgradeFrom2_0_xTest.java
View File

@@ -24,7 +24,6 @@ import static org.junit.Assert.assertFalse;
import static org.junit.Assert.assertNull;
import static org.junit.Assert.assertTrue;
import com.google.gerrit.pgm.util.ConsoleUI;
import com.google.gerrit.server.config.SitePaths;
import org.eclipse.jgit.errors.ConfigInvalidException;