[PATCH] Improve error message: not a valid branch name

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



(create_branch): Extend die message from 'is not a valid branch name'
to '...(see git-check-ref-format)'.

Signed-off-by: Jari Aalto <jari.aalto@xxxxxxxxx>
---
 builtin-branch.c |    9 ++++++---
 1 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/builtin-branch.c b/builtin-branch.c
index 7408285..1006a85 100644
--- a/builtin-branch.c
+++ b/builtin-branch.c
@@ -431,7 +431,8 @@ static void create_branch(const char *name, const char *start_name,
 
 	snprintf(ref, sizeof ref, "refs/heads/%s", name);
 	if (check_ref_format(ref))
-		die("'%s' is not a valid branch name.", name);
+		die("'%s' is not a valid branch name "
+                    "(see git-check-ref-format)", name);
 
 	if (resolve_ref(ref, sha1, 1, NULL)) {
 		if (!force)
@@ -502,13 +503,15 @@ static void rename_branch(const char *oldname, const char *newname, int force)
 		die("Old branchname too long");
 
 	if (check_ref_format(oldref))
-		die("Invalid branch name: %s", oldref);
+		die("Invalid branch name: %s "
+                    "(see git-check-ref-format)", oldref);
 
 	if (snprintf(newref, sizeof(newref), "refs/heads/%s", newname) > sizeof(newref))
 		die("New branchname too long");
 
 	if (check_ref_format(newref))
-		die("Invalid branch name: %s", newref);
+		die("Invalid branch name: %s "
+                    "(see git-check-ref-format)", newref);
 
 	if (resolve_ref(newref, sha1, 1, NULL) && !force)
 		die("A branch named '%s' already exists.", newname);
-- 
1.5.3.rc5


-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux