opendev/gerrit
Code Issues Proposed changes

Merge "Rely on @ImplementedBy for binding default repomgr"

Browse Source
This commit is contained in:
David Pursehouse
2016-11-15 21:48:48 +00:00
committed by Gerrit Code Review
parent 7b409444dc 8e5418db0a
commit 16f2eb42f0
2 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
gerrit-server/src/main/java/com/google/gerrit/server/git/GitRepositoryManager.java
View File

@@ -15,6 +15,7 @@
package com.google.gerrit.server.git; package com.google.gerrit.server.git;
import com.google.gerrit.reviewdb.client.Project; import com.google.gerrit.reviewdb.client.Project;
import com.google.inject.ImplementedBy;
import com.google.inject.Singleton; import com.google.inject.Singleton;
import org.eclipse.jgit.errors.RepositoryNotFoundException; import org.eclipse.jgit.errors.RepositoryNotFoundException;
@@ -30,6 +31,7 @@ import java.util.SortedSet;
* registered in Guice so they are globally available within the server * registered in Guice so they are globally available within the server
* environment. * environment.
*/ */
@ImplementedBy(value = LocalDiskRepositoryManager.class)
public interface GitRepositoryManager { public interface GitRepositoryManager {
/** /**
* Get (or open) a repository by name. * Get (or open) a repository by name.

1
gerrit-server/src/main/java/com/google/gerrit/server/git/LocalDiskRepositoryManager.java
View File

@@ -61,7 +61,6 @@ public class LocalDiskRepositoryManager implements GitRepositoryManager {
public static class Module extends LifecycleModule { public static class Module extends LifecycleModule {
@Override @Override
protected void configure() { protected void configure() {
bind(GitRepositoryManager.class).to(LocalDiskRepositoryManager.class);
listener().to(LocalDiskRepositoryManager.Lifecycle.class); listener().to(LocalDiskRepositoryManager.Lifecycle.class);
} }
} }