From mboxrd@z Thu Jan 1 00:00:00 1970 From: "J. Bruce Fields" Subject: [PATCH] errors: "strict subset" -> "ancestor" Date: Fri, 2 Nov 2007 22:39:44 -0400 Message-ID: <20071103023944.GA15379@fieldses.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: git@vger.kernel.org To: Junio C Hamano X-From: git-owner@vger.kernel.org Sat Nov 03 03:40:09 2007 Return-path: Envelope-to: gcvg-git-2@gmane.org Received: from vger.kernel.org ([209.132.176.167]) by lo.gmane.org with esmtp (Exim 4.50) id 1Io8ve-00045M-22 for gcvg-git-2@gmane.org; Sat, 03 Nov 2007 03:40:06 +0100 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753656AbXKCCjw (ORCPT ); Fri, 2 Nov 2007 22:39:52 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754081AbXKCCjw (ORCPT ); Fri, 2 Nov 2007 22:39:52 -0400 Received: from mail.fieldses.org ([66.93.2.214]:56025 "EHLO fieldses.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752949AbXKCCjv (ORCPT ); Fri, 2 Nov 2007 22:39:51 -0400 Received: from bfields by fieldses.org with local (Exim 4.68) (envelope-from ) id 1Io8vI-0004w3-VC; Fri, 02 Nov 2007 22:39:44 -0400 Content-Disposition: inline User-Agent: Mutt/1.5.16 (2007-06-11) Sender: git-owner@vger.kernel.org Precedence: bulk X-Mailing-List: git@vger.kernel.org Archived-At: From: J. Bruce Fields The term "ancestor" is a bit more intuitive (and more consistent with the documentation) than the term "strict subset". Also, remove superfluous "ref", capitalize, and add some carriage returns, changing: error: remote 'refs/heads/master' is not a strict subset of local ref 'refs/heads/master'. maybe you are not up-to-date and need to pull first? error: failed to push to 'ssh://linux-nfs.org/~bfields/exports/git.git' to: error: remote 'refs/heads/master' is not an ancestor of local 'refs/heads/master'. Maybe you are not up-to-date and need to pull first? error: failed to push to 'ssh://linux-nfs.org/~bfields/exports/git.git' Signed-off-by: J. Bruce Fields --- builtin-branch.c | 2 +- http-push.c | 8 ++++---- send-pack.c | 6 +++--- 3 files changed, 8 insertions(+), 8 deletions(-) diff --git a/builtin-branch.c b/builtin-branch.c index 3da8b55..e8de27e 100644 --- a/builtin-branch.c +++ b/builtin-branch.c @@ -142,7 +142,7 @@ static int delete_branches(int argc, const char **argv, int force, int kinds) if (!force && !in_merge_bases(rev, &head_rev, 1)) { - error("The branch '%s' is not a strict subset of " + error("The branch '%s' is not an ancestor of " "your current HEAD.\n" "If you are sure you want to delete it, " "run 'git branch -D %s'.", argv[i], argv[i]); diff --git a/http-push.c b/http-push.c index c02a3af..5960d7c 100644 --- a/http-push.c +++ b/http-push.c @@ -2241,7 +2241,7 @@ static int delete_remote_branch(char *pattern, int force) /* Remote branch must be an ancestor of remote HEAD */ if (!verify_merge_base(head_sha1, remote_ref->old_sha1)) { - return error("The branch '%s' is not a strict subset of your current HEAD.\nIf you are sure you want to delete it, run:\n\t'git http-push -D %s %s'", remote_ref->name, remote->url, pattern); + return error("The branch '%s' is not an ancestor of your current HEAD.\nIf you are sure you want to delete it, run:\n\t'git http-push -D %s %s'", remote_ref->name, remote->url, pattern); } } @@ -2424,9 +2424,9 @@ int main(int argc, char **argv) * commits at the remote end and likely * we were not up to date to begin with. */ - error("remote '%s' is not a strict " - "subset of local ref '%s'. " - "maybe you are not up-to-date and " + error("remote '%s' is not an ancestor of\n" + " local '%s'.\n" + " Maybe you are not up-to-date and " "need to pull first?", ref->name, ref->peer_ref->name); diff --git a/send-pack.c b/send-pack.c index 5e127a1..b74fd45 100644 --- a/send-pack.c +++ b/send-pack.c @@ -297,9 +297,9 @@ static int send_pack(int in, int out, struct remote *remote, int nr_refspec, cha * commits at the remote end and likely * we were not up to date to begin with. */ - error("remote '%s' is not a strict " - "subset of local ref '%s'. " - "maybe you are not up-to-date and " + error("remote '%s' is not an ancestor of\n" + " local '%s'.\n" + " Maybe you are not up-to-date and " "need to pull first?", ref->name, ref->peer_ref->name); -- 1.5.3.5.475.g477d-dirty