[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
02/13: git: 'update-cached-checkout' returns the commit ID when given a
From: |
guix-commits |
Subject: |
02/13: git: 'update-cached-checkout' returns the commit ID when given a tag. |
Date: |
Mon, 17 Oct 2022 04:45:52 -0400 (EDT) |
civodul pushed a commit to branch master
in repository guix.
commit 46f7011591601b4783b2dc2c7252c41564ca0140
Author: Ludovic Courtès <ludo@gnu.org>
AuthorDate: Tue Oct 11 11:16:23 2022 +0200
git: 'update-cached-checkout' returns the commit ID when given a tag.
Previously, starting with commit
efa578ecaece67366b4b0e2266de7c2faaa4ae54, 'update-cached-checkout' would
return the OID of the annotated tag the tag points to. With this change
it returns the OID of the commit object in all cases.
* guix/git.scm (resolve-reference): In the 'tag' case, call
'tag-target-id' and 'tag-lookup' when OID designates an annotated tag.
* tests/git.scm ("update-cached-checkout, tag"): New test.
---
guix/git.scm | 15 +++++++++------
tests/git.scm | 32 +++++++++++++++++++++++++++++++-
2 files changed, 40 insertions(+), 7 deletions(-)
diff --git a/guix/git.scm b/guix/git.scm
index 10e6dcaf23..95630a5e69 100644
--- a/guix/git.scm
+++ b/guix/git.scm
@@ -272,12 +272,15 @@ corresponding Git object."
;; There's no such tag, so it must be a commit ID.
(resolve `(commit . ,str)))))))
(('tag . tag)
- (let ((oid (reference-name->oid repository
- (string-append "refs/tags/" tag))))
- ;; OID may point to a "tag" object, but it can also point directly
- ;; to a "commit" object, as surprising as it may seem. Return that
- ;; object, whatever that is.
- (object-lookup repository oid))))))
+ (let* ((oid (reference-name->oid repository
+ (string-append "refs/tags/" tag)))
+ (obj (object-lookup repository oid)))
+ ;; OID may designate an "annotated tag" object or a "commit" object.
+ ;; Return the commit object in both cases.
+ (if (= OBJ-TAG (object-type obj))
+ (object-lookup repository
+ (tag-target-id (tag-lookup repository oid)))
+ obj))))))
(define (switch-to-ref repository ref)
"Switch to REPOSITORY's branch, commit or tag specified by REF. Return the
diff --git a/tests/git.scm b/tests/git.scm
index ca59d2a33e..9c944d65b1 100644
--- a/tests/git.scm
+++ b/tests/git.scm
@@ -22,8 +22,12 @@
#:use-module (guix git)
#:use-module (guix tests git)
#:use-module (guix build utils)
+ #:use-module ((guix utils) #:select (call-with-temporary-directory))
#:use-module (srfi srfi-1)
- #:use-module (srfi srfi-64))
+ #:use-module (srfi srfi-64)
+ #:use-module (srfi srfi-71)
+ #:use-module (ice-9 popen)
+ #:use-module (ice-9 textual-ports))
;; Test the (guix git) tools.
@@ -239,4 +243,30 @@
(tag "v1.1" "Release 1.1"))
(remote-refs directory #:tags? #t)))
+(unless (which (git-command)) (test-skip 1))
+(test-assert "update-cached-checkout, tag"
+ (call-with-temporary-directory
+ (lambda (cache)
+ (with-temporary-git-repository directory
+ '((add "a.txt" "A")
+ (commit "First commit")
+ (tag "v1.0" "release-1.0")
+ (branch "develop")
+ (checkout "develop")
+ (add "b.txt" "B")
+ (commit "Second commit")
+ (tag "v1.1" "release-1.1"))
+ (let ((directory commit relation
+ (update-cached-checkout directory
+ #:ref '(tag . "v1.1")
+ #:cache-directory cache))
+ (head (let* ((pipe (open-pipe* OPEN_READ (git-command)
+ "-C" directory
+ "rev-parse" "HEAD"))
+ (str (get-string-all pipe)))
+ (close-pipe pipe)
+ (string-trim-right str))))
+ ;; COMMIT should be the ID of the commit object, not that of the tag.
+ (string=? commit head))))))
+
(test-end "git")
- branch master updated (094b357783 -> c2cb116032), guix-commits, 2022/10/17
- 11/13: gnu: git-annex: Promote git to propagated input, guix-commits, 2022/10/17
- 01/13: Revert "nix: Guard against removing temporary roots of living processes.", guix-commits, 2022/10/17
- 04/13: gnu: Add openfst-for-vosk., guix-commits, 2022/10/17
- 03/13: channels: Interpret the 'commit' field of channel as a tag or commit., guix-commits, 2022/10/17
- 05/13: gnu: Add kaldi-for-vosk., guix-commits, 2022/10/17
- 08/13: gnu: Add nerd-dictation., guix-commits, 2022/10/17
- 06/13: gnu: Add vosk-api., guix-commits, 2022/10/17
- 02/13: git: 'update-cached-checkout' returns the commit ID when given a tag.,
guix-commits <=
- 07/13: gnu: Add python-vosk., guix-commits, 2022/10/17
- 09/13: gnu: Add nerd-dictation/wayland., guix-commits, 2022/10/17
- 10/13: read-print: Recognize 'define-record-type' and 'define-record-type*'., guix-commits, 2022/10/17
- 12/13: gnu: nheko: Do not build api docs., guix-commits, 2022/10/17
- 13/13: gnu: Add hunspell-dict-hu., guix-commits, 2022/10/17