Ensure a new commit exists before triggering hook Change-Id: Ie055c0edc825da0be0c8386b6c0f1fa815db0839
diff --git a/src/main/java/com/amd/gerrit/plugins/manifestsubscription/Utilities.java b/src/main/java/com/amd/gerrit/plugins/manifestsubscription/Utilities.java index db8eaf9..caaafc0 100644 --- a/src/main/java/com/amd/gerrit/plugins/manifestsubscription/Utilities.java +++ b/src/main/java/com/amd/gerrit/plugins/manifestsubscription/Utilities.java
@@ -126,14 +126,17 @@ // https://code.google.com/p/gerrit/issues/detail?id=2564 // https://gerrit-review.googlesource.com/55540 if (commit != null) { - ObjectId parent = ObjectId.zeroId(); + if (!commit.equals(commitId)) { + ObjectId parent = ObjectId.zeroId(); - if (commit.getParents().length > 0) { - parent = commit.getParent(0).getId(); + if (commit.getParents().length > 0) { + parent = commit.getParent(0).getId(); + } + changeHooks.doRefUpdatedHook(new Branch.NameKey(p, refName), + parent, + commit.getId(), null); + } - changeHooks.doRefUpdatedHook(new Branch.NameKey(p, refName), - parent, - commit.getId(), null); return commit.getId(); } else { log.warn("Failing to commit manifest subscription update:"+