Skip to contentSkip to conversation

cg-object-id: comment the alternate id shortcut parsing

PATCH In review Jonas Fonseca 2 participants 4 messages Jun 15, 2016

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 cg-object-id: comment the alternate id shortcut parsing

Add comment about why Cogito's own 4 digit id shortcut parser is needed.

---
commit 3c14cded46e110396127fc5b5e65883eb5cd60b9
tree 593da711de14cfc3f8dc1fd5773990902568b42e
parent 36c0d9fae209ab7fa889333e8eebef22e7f0f55c
author Jonas Fonseca <[email protected]> Wed, 07 Dec 2005 08:17:04 +0100 committer Jonas Fonseca <[email protected]> Wed, 07 Dec 2005 08:17:04 +0100

 cg-object-id |    4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)
cg-object-id +3 -1
index 56d9f91..ea04dca 100755
@@ -52,7 +52,9 @@ normalize_id()
 		id="$revid"
 		valid=1
 
-	# Short id's must be lower case and at least 4 digits.
+	# Short id's must be lower case and at least 4 digits. git-rev-parse(1)
+	# allows only down to 5 digits and we want to show the list of matched
+	# ids.
 	elif [[ "$id" == [0-9a-f][0-9a-f][0-9a-f][0-9a-f]* ]]; then
 		idpref=${id:0:2}
 		idpost=${id:2}
Signature · 1 line
Jonas Fonseca
#1 lore raw

Dear diary, on Wed, Dec 07, 2005 at 10:39:43PM CET, I got a letter where Jonas Fonseca <[email protected]> said that...

cg-object-id normalize_id() · line 57

Quoted 4 lines
cg-object-id +3 -1
-	# Short id's must be lower case and at least 4 digits.
+	# Short id's must be lower case and at least 4 digits. git-rev-parse(1)
+	# allows only down to 5 digits and we want to show the list of matched
+	# ids.

Huh. The code in sha1_name.c speaks differently and it works as well:

	$ git-rev-parse b1fc
	b1fcca45069d4f6b00a9ac7136be4268d09fd6b9
Signature · 4 lines
				Petr "Pasky" Baudis
Stuff: http://pasky.or.cz/
VI has two modes: the one in which it beeps and the one in which
it doesn't.
#2 lore raw

Petr Baudis <[email protected]> wrote Wed, Dec 07, 2005:

Quoted 11 lines

Dear diary, on Wed, Dec 07, 2005 at 10:39:43PM CET, I got a letter where Jonas Fonseca <[email protected]> said that...

cg-object-id normalize_id() · line 57

Quoted 4 lines
cg-object-id +3 -1
-	# Short id's must be lower case and at least 4 digits.
+	# Short id's must be lower case and at least 4 digits. git-rev-parse(1)
+	# allows only down to 5 digits and we want to show the list of matched
+	# ids.

Huh. The code in sha1_name.c speaks differently and it works as well:

	$ git-rev-parse b1fc
	b1fcca45069d4f6b00a9ac7136be4268d09fd6b9

I had this one testcase:

	~/src/elinks/0.11 > cg-object-id --verify e6b6
	cg-object-id: unrecognized option `--verify'
	~/src/elinks/0.11 > cg-object-id e6b6
	e6b65b543108bfeb95ea24978112b46e34cfde87

Could it have something to do with the local packing?

Signature · 1 line
Jonas Fonseca
#3 lore raw

Jonas Fonseca <[email protected]> wrote Wed, Dec 07, 2005:

Quoted 10 lines

Petr Baudis <[email protected]> wrote Wed, Dec 07, 2005:

Quoted 9 lines

Dear diary, on Wed, Dec 07, 2005 at 10:39:43PM CET, I got a letter where Jonas Fonseca <[email protected]> said that...

cg-object-id normalize_id() · line 57

Quoted 4 lines
cg-object-id +3 -1
-	# Short id's must be lower case and at least 4 digits.
+	# Short id's must be lower case and at least 4 digits. git-rev-parse(1)
+	# allows only down to 5 digits and we want to show the list of matched
+	# ids.

Huh. The code in sha1_name.c speaks differently and it works as well:

Quoted 5 lines
Quoted 2 lines
	$ git-rev-parse b1fc
	b1fcca45069d4f6b00a9ac7136be4268d09fd6b9

I had this one testcase:

~/src/elinks/0.11 > git-rev-parse --verify e6b6
error: short SHA1 e6b6 is ambiguous.
fatal: Needed a single revision

Quoted 4 lines
	~/src/elinks/0.11 > cg-object-id e6b6
	e6b65b543108bfeb95ea24978112b46e34cfde87

Could it have something to do with the local packing?

Signature · 1 line
Jonas Fonseca