Subject: bug#30716: closed (Re: bug#30716: guix pull: error: Git error: unable to parse OID - contains invalid characters when COMMIT-ID is shortened)

From: [email-masked] (GNU bug Tracking System)
Date: Sat, 17 Mar 2018 23:06:02 +0000

Your bug report

#30716: guix pull: error: Git error: unable to parse OID - contains invalid characters when COMMIT-ID is shortened

which was filed against the guix package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to [email-masked].

-- 
30716: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=30716
GNU Bug Tracking System
Contact [email-masked] with problems

Subject: Re: bug#30716: guix pull: error: Git error: unable to parse OID - contains invalid characters when COMMIT-ID is shortened
From: [email-masked] (Ludovic Courtès)
To: Björn Höfling <[email-masked]>
Cc: [email-masked]
Date: Sun, 18 Mar 2018 00:04:58 +0100
List-Id: <debbugs-submit.debbugs.gnu.org>

Hi Björn,

Björn Höfling <[email-masked]> skribis:

> $>guix pull --commit=d20647722ad36024a3a0b6e1e75c1ba49b9f61dz
> Updating from Git repository at 'https://git.savannah.gnu.org/git/guix.git'...
> guix pull: error: Git error: unable to parse OID - contains invalid characters

This was actually and out-of-bounds read in Guile-Git (!), now fixed:

  https://gitlab.com/guile-git/guile-git/commit/d37a40de69d727529fe2aebb6afcf362bf8e9ba7

Furthermore, ‘guix pull’ now supports short IDs provided a recent-enough
Guile-Git is available:

  https://git.savannah.gnu.org/cgit/guix.git/commit/?id=95bd9f65a8ee64d17707a76aebc8720bbd961b68

Thanks,
Ludo’.

Subject: guix pull: error: Git error: unable to parse OID - contains invalid characters when COMMIT-ID is shortened
From: Björn Höfling <[email-masked]>
To: <[email-masked]>
Date: Mon, 5 Mar 2018 21:29:12 +0100
List-Id: <debbugs-submit.debbugs.gnu.org>

Guix pull has the wrong error message when the commit-id for the
"--commit" parameter is too short:

$>guix pull --commit=d20647722ad36024a3a0b6e1e75c1ba49b9f61d
Updating from Git repository at
'https://git.savannah.gnu.org/git/guix.git'... guix pull: error: Git
error: unable to parse OID - contains invalid characters

Instead of that error-message it should complain that the OID is too
short.

Even better, it should just accept a shortened OID and use it, as long
as it is unique.

Note: When there really is a wrong character used, this error message
is also correctly shown:

$>guix pull --commit=d20647722ad36024a3a0b6e1e75c1ba49b9f61dz
Updating from Git repository at 'https://git.savannah.gnu.org/git/guix.git'...
guix pull: error: Git error: unable to parse OID - contains invalid characters

Note: If the OID is long enough and well-formed, but just non-existent,
the error message is adequately correct:

$> guix pull --commit=d20647722ad36024a3a0b6e1e75c1ba49b9f61dd 
Updating from Git repository at 
'https://git.savannah.gnu.org/git/guix.git'... guix pull: error: Git
error: object not found - no match for id
(d20647722ad36024a3a0b6e1e75c1ba49b9f61dd)

Björn