From be63d100680242d05e2a852dc1bd6c40aa91e82a Mon Sep 17 00:00:00 2001 From: "Shawn O. Pearce" Date: Wed, 22 Oct 2008 16:26:19 -0700 Subject: [PATCH] Remove unused commit field from SyncProjectRequest Gerrit hasn't required the commit field per-branch in a long time, since before the code tree was open sourced. It was an accident that we didn't remove this sooner, as it only complicates the project initialization process. Signed-off-by: Shawn O. Pearce --- mgrapp/src/com/google/codereview/manager/ProjectSync.java | 1 - proto/internal/sync_project.proto | 4 +--- 2 files changed, 1 insertion(+), 4 deletions(-) diff --git a/mgrapp/src/com/google/codereview/manager/ProjectSync.java b/mgrapp/src/com/google/codereview/manager/ProjectSync.java index b3d2d9eb2a..1e910c34b2 100644 --- a/mgrapp/src/com/google/codereview/manager/ProjectSync.java +++ b/mgrapp/src/com/google/codereview/manager/ProjectSync.java @@ -113,7 +113,6 @@ public class ProjectSync { private BranchSync toBranch(final Ref r, final RevCommit c) { final BranchSync.Builder bs = BranchSync.newBuilder(); bs.setBranchName(r.getName()); - bs.setCommit(GitMetaUtil.toGitCommit(c)); return bs.build(); } diff --git a/proto/internal/sync_project.proto b/proto/internal/sync_project.proto index f661da71da..9d5b534e57 100644 --- a/proto/internal/sync_project.proto +++ b/proto/internal/sync_project.proto @@ -15,11 +15,9 @@ package codereview.internal; option java_package = "com.google.codereview.internal"; -import "git_meta.proto"; - message BranchSync { required string branch_name = 1; - required GitCommit commit = 2; + // field 2 used to be "commit" } message SyncProjectRequest {