progfolio / elpaca

An elisp package manager
GNU General Public License v3.0
634 stars 31 forks source link

[Bug]: elpaca--date-version does not ignore user's git config #256

Closed 0x08088405 closed 8 months ago

0x08088405 commented 8 months ago

Confirmation

Elpaca Version

Elpaca 325060c HEAD -> master, origin/master, origin/HEAD
installer:      0.6
emacs-version:  GNU Emacs 29.1 (build 1, x86_64-pc-linux-gnu, cairo version 1.18.0)
 of 2024-02-08
git --version:  git version 2.43.0

Operating System

Gentoo GNU/Linux amd64

Description

Garbage issue title, not sure what to call this, sorry. Trying out elpaca again with a blank user-init-file. I try to install magit, ran into the seq issue, did the pinned workaround. These errors remain:

magit                          failed               Failed dependencies: (magit-section)                                             00.546592
magit-section                  failed               Requires dash minimum version: 20221013                                          00.550316
git-commit                     failed               Requires transient minimum version: 20231204                                     00.619223

According to elpaca-info, I have dash 2.19.1 13f3fcd and transient 0.5.3 85ecbc6 installed. Now I admit I don't yet write elisp packages and don't know how the development versioning works, but those commits are ahead of the calendar dates it's indicating, so I'm not sure what the complaint actually is.

progfolio commented 8 months ago

Thanks for taking the time to fill out a ticket. Please share the full text of the init file you are using.

0x08088405 commented 8 months ago

Sure, I've stripped it of personalisations to make it less confusing, and tested that it still happens.

early-init-file ```elisp ;; -*- lexical-binding: t; no-byte-compile: t; -*- (setq package-enable-at-startup nil) ```
user-init-file ```elisp ;; -*- lexical-binding: t; no-byte-compile: t; -*- (defvar elpaca-installer-version 0.6) (defvar elpaca-directory (expand-file-name "elpaca/" user-emacs-directory)) (defvar elpaca-builds-directory (expand-file-name "builds/" elpaca-directory)) (defvar elpaca-repos-directory (expand-file-name "repos/" elpaca-directory)) (defvar elpaca-order '(elpaca :repo "https://github.com/progfolio/elpaca.git" :ref nil :files (:defaults "elpaca-test.el" (:exclude "extensions")) :build (:not elpaca--activate-package))) (let* ((repo (expand-file-name "elpaca/" elpaca-repos-directory)) (build (expand-file-name "elpaca/" elpaca-builds-directory)) (order (cdr elpaca-order)) (default-directory repo)) (add-to-list 'load-path (if (file-exists-p build) build repo)) (unless (file-exists-p repo) (make-directory repo t) (when (< emacs-major-version 28) (require 'subr-x)) (condition-case-unless-debug err (if-let ((buffer (pop-to-buffer-same-window "*elpaca-bootstrap*")) ((zerop (call-process "git" nil buffer t "clone" (plist-get order :repo) repo))) ((zerop (call-process "git" nil buffer t "checkout" (or (plist-get order :ref) "--")))) (emacs (concat invocation-directory invocation-name)) ((zerop (call-process emacs nil buffer nil "-Q" "-L" "." "--batch" "--eval" "(byte-recompile-directory \".\" 0 'force)"))) ((require 'elpaca)) ((elpaca-generate-autoloads "elpaca" repo))) (progn (message "%s" (buffer-string)) (kill-buffer buffer)) (error "%s" (with-current-buffer buffer (buffer-string)))) ((error) (warn "%s" err) (delete-directory repo 'recursive)))) (unless (require 'elpaca-autoloads nil t) (require 'elpaca) (elpaca-generate-autoloads "elpaca" repo) (load "./elpaca-autoloads"))) (add-hook 'after-init-hook #'elpaca-process-queues) (elpaca `(,@elpaca-order)) (defun +elpaca-unload-seq (e) (and (featurep 'seq) (unload-feature 'seq t)) (elpaca--continue-build e)) (defun +elpaca-seq-build-steps () (append (butlast (if (file-exists-p (expand-file-name "seq" elpaca-builds-directory)) elpaca--pre-built-steps elpaca-build-steps)) (list '+elpaca-unload-seq 'elpaca--activate-package))) (elpaca elpaca-use-package (elpaca-use-package-mode)) (elpaca-wait) (elpaca `(seq :build ,(+elpaca-seq-build-steps))) (use-package magit :ensure t) ```

Additionally, these, if they may be useful:

(elpaca-info 'dash) ``` dash [MELPA|GNU-devel ELPA] A modern list library for Emacs source: MELPA url: https://github.com/magnars/dash.el menu item recipe: ( :package "dash" :fetcher github :repo "magnars/dash.el" :files ("dash.el" "dash.texi") :source "MELPA") full recipe: ( :package "dash" ;; Inherited from elpaca-order-functions. :depth 1 :inherit t :protocol https ;; Inherited from elpaca-menu-item. :source "MELPA" :files ("dash.el" "dash.texi") :repo "magnars/dash.el" :fetcher github) dependencies: emacs >= 24 dependents: magit magit-section installed version: 2.19.1 13f9fcd statuses: (finished activation info byte-compilation autoloads linking unblocked ref-checked-out cloning continued-dep queued) files: $REPOS/dash/dash.texi → $BUILDS/dash/dash.texi $REPOS/dash/dash.el → $BUILDS/dash/dash.el log: [2024-02-11 13:19:01] Package queued [2024-02-11 13:19:02] Continued by: elpaca--continue-dependency [2024-02-11 13:19:02] Cloning [2024-02-11 13:19:02] $git clone --depth 1 https://github.com/magnars/dash.el.git /home/viri/.emacs.d/elpaca/repos/dash/ [2024-02-11 13:19:02] Cloning into '/home/viri/.emacs.d/elpaca/repos/dash'... [2024-02-11 13:19:03] Resolving deltas: 100% (1/1), done. [2024-02-11 13:19:03] Continued by: elpaca--clone-process-sentinel [2024-02-11 13:19:03] Continued by: elpaca--configure-remotes [2024-02-11 13:19:03] Continued by: elpaca--checkout-ref [2024-02-11 13:19:03] Continued by: elpaca--dispatch-build-commands [2024-02-11 13:19:03] Continued by: elpaca--queue-dependencies [2024-02-11 13:19:03] No external dependencies [2024-02-11 13:19:03] Checking dependency versions [2024-02-11 13:19:03] Continued by: elpaca--check-version [2024-02-11 13:19:03] Linking build files [2024-02-11 13:19:03] Continued by: elpaca--link-build-files [2024-02-11 13:19:03] Build files linked [2024-02-11 13:19:03] Generating autoloads: /home/viri/.emacs.d/elpaca/builds/dash [2024-02-11 13:19:03] $/usr/bin/emacs -Q -L /home/viri/.emacs.d/elpaca/repos/elpaca/ -l /home/viri/.emacs.d/elpaca/repos/elpaca/elpaca.el --batch --eval (progn (setq gc-cons-percentage 1.0) (elpaca-generate-autoloads "dash" "/home/viri/.emacs.d/elpaca/builds/dash")) [2024-02-11 13:19:03] INFO Scraping files for loaddefs... [2024-02-11 13:19:03] INFO Scraping files for loaddefs...done [2024-02-11 13:19:03] GEN dash-autoloads.el [2024-02-11 13:19:03] Continued by: #[128 "\302\301\303\300\"\"\207" [("Autoloads Generated" nil) elpaca--process-sentinel apply append] 6 " (fn &rest ARGS2)"] [2024-02-11 13:19:03] Autoloads Generated [2024-02-11 13:19:03] Byte compiling [2024-02-11 13:19:03] $/usr/bin/emacs -q --batch --eval (let ((gc-cons-percentage 1.0) (native-comp-eln-load-path '("/home/viri/.emacs.d/eln-cache/" "/usr/lib64/emacs/29.1/native-lisp/"))) (dolist (dir '("/home/viri/.emacs.d/elpaca/builds/dash")) (let ((default-directory dir)) (add-to-list 'load-path dir) (normal-top-level-add-subdirs-to-load-path))) (byte-recompile-directory "/home/viri/.emacs.d/elpaca/builds/dash" 0 'force)) [2024-02-11 13:19:03] Checking /home/viri/.emacs.d/elpaca/builds/dash... [2024-02-11 13:19:03] Compiling /home/viri/.emacs.d/elpaca/builds/dash/dash-autoloads.el... [2024-02-11 13:19:03] Compiling /home/viri/.emacs.d/elpaca/builds/dash/dash.el... [2024-02-11 13:19:04] Done (Total of 1 file compiled, 1 skipped) [2024-02-11 13:19:04] Continued by: #[128 "\302\301\303\300\"\"\207" [("Byte compilation complete" nil) elpaca--process-sentinel apply append] 6 " (fn &rest ARGS2)"] [2024-02-11 13:19:04] Byte compilation complete [2024-02-11 13:19:04] Compiling Info files [2024-02-11 13:19:11] Continued by: elpaca--compile-info-process-sentinel [2024-02-11 13:19:11] Info compiled [2024-02-11 13:19:11] Installing Info files [2024-02-11 13:19:11] Continued by: elpaca--install-info [2024-02-11 13:19:11] /home/viri/.emacs.d/elpaca/builds/dash/dash.info [2024-02-11 13:19:11] Continued by: elpaca--install-info-process-sentinel [2024-02-11 13:19:11] Info installed [2024-02-11 13:19:11] Adding Info path [2024-02-11 13:19:11] Continued by: elpaca--add-info-path [2024-02-11 13:19:11] Continued by: elpaca--dispatch-build-commands [2024-02-11 13:19:11] Activating package [2024-02-11 13:19:11] Package build dir added to load-path [2024-02-11 13:19:11] Caching autoloads [2024-02-11 13:19:11] Autoloads cached [2024-02-11 13:19:11] Continued by: elpaca--activate-package [2024-02-11 13:19:11] ✓ 9.409 secs ```
(elpaca-info 'transient) ``` transient [MELPA|GNU-devel ELPA] Transient commands source: MELPA url: https://github.com/magit/transient menu item recipe: ( :package "transient" :fetcher github :repo "magit/transient" :files ("*.el" "*.el.in" "dir" "*.info" "*.texi" "*.texinfo" "doc/dir" "doc/*.info" "doc/*.texi" "doc/*.texinfo" "lisp/*.el" (:exclude ".dir-locals.el" "test.el" "tests.el" "*-test.el" "*-tests.el" "LICENSE" "README*" "*-pkg.el")) :source "MELPA") full recipe: ( :package "transient" ;; Inherited from elpaca-order-functions. :depth 1 :inherit t :protocol https ;; Inherited from elpaca-menu-item. :source "MELPA" :files (:defaults) :repo "magit/transient" :fetcher github) dependencies: emacs >= 26.1 compat >= 29.1.4.4 seq >= 2.24 dependents: git-commit magit installed version: 0.5.3 85ecbc6 statuses: (finished activation info byte-compilation autoloads linking unblocked blocked ref-checked-out cloning continued-dep queued) files: $REPOS/transient/lisp/transient.el → $BUILDS/transient/transient.el log: [2024-02-11 13:19:02] Package queued [2024-02-11 13:19:02] Continued by: elpaca--continue-dependency [2024-02-11 13:19:02] Cloning [2024-02-11 13:19:02] $git clone --depth 1 https://github.com/magit/transient.git /home/viri/.emacs.d/elpaca/repos/transient/ [2024-02-11 13:19:02] Cloning into '/home/viri/.emacs.d/elpaca/repos/transient'... [2024-02-11 13:19:02] Resolving deltas: 100% (3/3), done. [2024-02-11 13:19:03] Continued by: elpaca--clone-process-sentinel [2024-02-11 13:19:03] Continued by: elpaca--configure-remotes [2024-02-11 13:19:03] Continued by: elpaca--checkout-ref [2024-02-11 13:19:03] Continued by: elpaca--dispatch-build-commands [2024-02-11 13:19:03] Queueing Dependencies [2024-02-11 13:19:03] Blocked by: (seq compat) [2024-02-11 13:19:04] Blocked by: (compat) [2024-02-11 13:19:10] Continued by: elpaca--check-status [2024-02-11 13:19:10] Unblocked by: compat [2024-02-11 13:19:10] Checking dependency versions [2024-02-11 13:19:10] Continued by: elpaca--check-version [2024-02-11 13:19:10] Linking build files [2024-02-11 13:19:10] Continued by: elpaca--link-build-files [2024-02-11 13:19:10] Build files linked [2024-02-11 13:19:10] Generating autoloads: /home/viri/.emacs.d/elpaca/builds/transient [2024-02-11 13:19:11] $/usr/bin/emacs -Q -L /home/viri/.emacs.d/elpaca/repos/elpaca/ -l /home/viri/.emacs.d/elpaca/repos/elpaca/elpaca.el --batch --eval (progn (setq gc-cons-percentage 1.0) (elpaca-generate-autoloads "transient" "/home/viri/.emacs.d/elpaca/builds/transient")) [2024-02-11 13:19:11] INFO Scraping files for loaddefs... [2024-02-11 13:19:11] INFO Scraping files for loaddefs...done [2024-02-11 13:19:11] GEN transient-autoloads.el [2024-02-11 13:19:11] Continued by: #[128 "\302\301\303\300\"\"\207" [("Autoloads Generated" nil) elpaca--process-sentinel apply append] 6 " (fn &rest ARGS2)"] [2024-02-11 13:19:11] Autoloads Generated [2024-02-11 13:19:11] Byte compiling [2024-02-11 13:19:11] $/usr/bin/emacs -q --batch --eval (let ((gc-cons-percentage 1.0) (native-comp-eln-load-path '("/home/viri/.emacs.d/eln-cache/" "/usr/lib64/emacs/29.1/native-lisp/"))) (dolist (dir '("/home/viri/.emacs.d/elpaca/builds/transient" "/home/viri/.emacs.d/elpaca/builds/compat" "/home/viri/.emacs.d/elpaca/builds/seq")) (let ((default-directory dir)) (add-to-list 'load-path dir) (normal-top-level-add-subdirs-to-load-path))) (byte-recompile-directory "/home/viri/.emacs.d/elpaca/builds/transient" 0 'force)) [2024-02-11 13:19:11] Checking /home/viri/.emacs.d/elpaca/builds/transient... [2024-02-11 13:19:11] Compiling /home/viri/.emacs.d/elpaca/builds/transient/transient-autoloads.el... [2024-02-11 13:19:11] Compiling /home/viri/.emacs.d/elpaca/builds/transient/transient.el... [2024-02-11 13:19:12] Done (Total of 1 file compiled, 1 skipped) [2024-02-11 13:19:12] Continued by: #[128 "\302\301\303\300\"\"\207" [("Byte compilation complete" nil) elpaca--process-sentinel apply append] 6 " (fn &rest ARGS2)"] [2024-02-11 13:19:12] Byte compilation complete [2024-02-11 13:19:12] Compiling Info files [2024-02-11 13:19:12] Continued by: elpaca--compile-info [2024-02-11 13:19:12] Info source files not found [2024-02-11 13:19:12] Continued by: elpaca--dispatch-build-commands [2024-02-11 13:19:12] Activating package [2024-02-11 13:19:12] Package build dir added to load-path [2024-02-11 13:19:12] Caching autoloads [2024-02-11 13:19:12] Autoloads cached [2024-02-11 13:19:12] Continued by: elpaca--activate-package [2024-02-11 13:19:12] ✓ 10.154 secs ```
progfolio commented 8 months ago

I'm unable to reproduce the issue given the test you provided on Emacs 29.1 or Emacs 30.

I've stripped it of personalisations to make it less confusing

The issue may be due to something in here, but it's hard to say without seeing any of it.

0x08088405 commented 8 months ago

I think you misunderstand, when I said

I've stripped it of personalisations to make it less confusing, and tested that it still happens

I meant that I wiped my user-emacs-directory, put in that early-init-file and user-init-file that I also posted here, and still hit the same issue. I was trying to reduce it to not be related to something else, and I posted a version where I still encounter this problem. Just tried again to be sure with those exact contents and nothing else in my user-emacs-directory and I still get the exact same output.

progfolio commented 8 months ago

Understood. Unfortunately I'm still not able to reproduce on my end. I'll see if there's another test case I can provide to get more info.

progfolio commented 8 months ago

My hunch is that the heuristic for getting the date versioning for the packages is failing somehow. I've pushed a change to include the version being compared against in the error message. Would you mind testing from scratch again and sharing the failure messages from the elpaca-log buffer?

ryunix commented 8 months ago

How about adding (elpaca-wait) after (elpaca `(seq :build ,(+elpaca-seq-build-steps)))?

progfolio commented 8 months ago

How about adding (elpaca-wait) after (elpaca `(seq :build ,(+elpaca-seq-build-steps)))?

That shouldn't effect the versions of the dependencies which are failing.

ryunix commented 8 months ago

That shouldn't effect the versions of the dependencies which are failing.

I'm sorry for the incorrect comment. :bow:

I'm reporting that I couldn't reproduce the issue in my environment either. I'm using Emacs 29.1 on Gentoo too. The versions matched, except for the build date.

0x08088405 commented 8 months ago

Installed version nil is lower than YYYYMMDD, for both of them.

magit                          failed               Failed dependencies: (git-commit)                                                25.622426
git-commit                     failed               transient installed version nil lower than min required 20231204                 25.628268
magit-section                  failed               dash installed version nil lower than min required 20221013                      25.990285
progfolio commented 8 months ago

Installed version nil is lower than YYYYMMDD, for both of them.

Thanks. That confirms my hunch. We need to figure out why elpaca--date-version is not returning anything for those packages. A good first step would be to bypass the function altogether to see if the git command used is working. Let's use dash as our example. Can you please:

  1. Navigate to dash's repository directory (should be at $ELPACA-DIR/repos/dash/) in a terminal.
  2. Execute git log -n 1 --format=%cd --date=format:%Y%m%d.%s

What output do you get from that command?

0x08088405 commented 8 months ago

Okay, that was a really good guess. I immediately knew why this was broken for me, and not you. Check this out:

gpg: Signature made Tue 23 Jan 2024 06:33:43 CST
gpg:                using RSA key AB0DD41003567A55A0B3B70F205AB54A5D5D8CFF
gpg: Can't check signature: No public key
20240123.1706013217

This is because of log.showSignature=true in my ~/.gitconfig. I temporarily removed that and the rest of magit compiles and loads and everything is happy. One could force that flag like git -c log.showSignature=false log ..., dunno if that's the best way, usually I see people use libgit2 (like magit does).

progfolio commented 8 months ago

Thanks for the details. That makes sense. I've pushed a branch with a potential fix so you won't have to workaround this. Would you mind executing the following test case on your machine and sharing the results?

Test Case [How to run this test?](https://github.com/progfolio/elpaca/wiki/Troubleshooting#the-elpaca-test-macro) ```emacs-lisp (elpaca-test :ref "fix/date-version" :init (defun +elpaca-unload-seq (e) (and (featurep 'seq) (unload-feature 'seq t)) (elpaca--continue-build e)) (defun +elpaca-seq-build-steps () (append (butlast (if (file-exists-p (expand-file-name "seq" elpaca-builds-directory)) elpaca--pre-built-steps elpaca-build-steps)) (list '+elpaca-unload-seq 'elpaca--activate-package))) (elpaca (` (seq :build (, (+elpaca-seq-build-steps))))) (elpaca magit) (elpaca-wait) (elpaca-test-log "#unique")) ```
Host Env
elpaca09a94ca HEAD -> fix/date-version, origin/fix/date-version
installer0.6
emacsGNU Emacs 30.0.50 (build 1, x86_64-pc-linux-gnu, GTK+ Version 3.24.41, cairo version 1.18.0) of 2024-02-03
gitgit version 2.43.0
Output ```emacs-lisp Elpaca menu item cache discarded due to version change. INFO Scraping files for loaddefs... INFO Scraping files for loaddefs...done GEN ../elpaca-autoloads.el Cloning into '/tmp/elpaca.BaUysT/elpaca/repos/elpaca'... Switched to a new branch 'fix/date-version' branch 'fix/date-version' set up to track 'origin/fix/date-version'. Checking /tmp/elpaca.BaUysT/elpaca/repos/elpaca... Compiling /tmp/elpaca.BaUysT/elpaca/repos/elpaca/elpaca-info.el... Compiling /tmp/elpaca.BaUysT/elpaca/repos/elpaca/elpaca-log.el... Compiling /tmp/elpaca.BaUysT/elpaca/repos/elpaca/elpaca-manager.el... Compiling /tmp/elpaca.BaUysT/elpaca/repos/elpaca/elpaca-menu-elpa.el... Compiling /tmp/elpaca.BaUysT/elpaca/repos/elpaca/elpaca-menu-melpa.el... Compiling /tmp/elpaca.BaUysT/elpaca/repos/elpaca/elpaca-menu-org.el... Compiling /tmp/elpaca.BaUysT/elpaca/repos/elpaca/elpaca-process.el... Compiling /tmp/elpaca.BaUysT/elpaca/repos/elpaca/elpaca-test.el... Compiling /tmp/elpaca.BaUysT/elpaca/repos/elpaca/elpaca-ui.el... Compiling /tmp/elpaca.BaUysT/elpaca/repos/elpaca/elpaca.el... Checking /tmp/elpaca.BaUysT/elpaca/repos/elpaca/doc... Compiling /tmp/elpaca.BaUysT/elpaca/repos/elpaca/doc/early-init.el... Compiling /tmp/elpaca.BaUysT/elpaca/repos/elpaca/doc/init.el... Checking /tmp/elpaca.BaUysT/elpaca/repos/elpaca/extensions... Compiling /tmp/elpaca.BaUysT/elpaca/repos/elpaca/extensions/elpaca-use-package.el... Checking /tmp/elpaca.BaUysT/elpaca/repos/elpaca/images... Checking /tmp/elpaca.BaUysT/elpaca/repos/elpaca/test... Compiling /tmp/elpaca.BaUysT/elpaca/repos/elpaca/test/elpaca-test.el... Done (Total of 11 files compiled, 3 skipped in 4 directories) Downloading MELPA recipes... Downloading MELPA recipes...100% Downloading NonGNU-devel ELPA... Downloading GNU-devel ELPA... Downloading NonGNU ELPA... Downloading GNU ELPA... elpaca-log: "#unique" " elpaca finished ✓ 0.565 secs 06.881992 seq finished ✓ 0.686 secs 07.003568 compat finished ✓ 1.192 secs 08.006836 dash finished ✓ 1.363 secs 08.177767 with-editor finished ✓ 1.406 secs 08.224217 git-commit finished ✓ 1.730 secs 08.539966 magit-section finished ✓ 1.768 secs 08.589694 magit finished ✓ 4.581 secs 10.897632 " Test Env Elpaca 09a94ca HEAD -> fix/date-version, origin/fix/date-version installer: 0.6 emacs-version: GNU Emacs 30.0.50 (build 1, x86_64-pc-linux-gnu, GTK+ Version 3.24.41, cairo version 1.18.0) of 2024-02-03 git --version: git version 2.43.0 ```

usually I see people use libgit2 (like magit does).

I'm trying to minimize elisp dependencies for this package, but I'll note that down to weigh out the tradeoffs. Thanks for the suggestion.

0x08088405 commented 8 months ago

(elpaca (` (seq :build (, (+elpaca-seq-build-steps)))))

I imagine this part of the test is meant to be (elpaca `(seq :build ,(+elpaca-seq-build-steps))) like in my above user-init-file, otherwise that elisp gets a classic "wrong argument type: symbolp" et cetera, so I edited that part of the expression. Here are the results, and I did check to put log.showSignature=true in my ~/.gitconfig beforehand;

Test Case [How to run this test?](https://github.com/progfolio/elpaca/wiki/Troubleshooting#the-elpaca-test-macro) ```emacs-lisp (elpaca-test :ref "fix/date-version" :init (defun +elpaca-unload-seq (e) (and (featurep 'seq) (unload-feature 'seq t)) (elpaca--continue-build e)) (defun +elpaca-seq-build-steps () (append (butlast (if (file-exists-p (expand-file-name "seq" elpaca-builds-directory)) elpaca--pre-built-steps elpaca-build-steps)) (list '+elpaca-unload-seq 'elpaca--activate-package))) (elpaca (` (seq :build (, (+elpaca-seq-build-steps))))) (elpaca magit) (elpaca-wait) (elpaca-test-log "#unique")) ```
Host Env
elpaca00ce5bb HEAD -> master, origin/master, origin/HEAD
installer0.6
emacsGNU Emacs 29.1 (build 1, x86_64-pc-linux-gnu, cairo version 1.18.0) of 2024-02-08
gitgit version 2.43.0
Output ```emacs-lisp Elpaca menu item cache discarded due to version change. INFO Scraping files for loaddefs... INFO Scraping files for loaddefs...done GEN ../elpaca-autoloads.el Cloning into '/tmp/elpaca.rtJtUg/elpaca/repos/elpaca'... Switched to a new branch 'fix/date-version' branch 'fix/date-version' set up to track 'origin/fix/date-version'. Checking /tmp/elpaca.rtJtUg/elpaca/repos/elpaca... Compiling /tmp/elpaca.rtJtUg/elpaca/repos/elpaca/elpaca-info.el... Compiling /tmp/elpaca.rtJtUg/elpaca/repos/elpaca/elpaca-log.el... Compiling /tmp/elpaca.rtJtUg/elpaca/repos/elpaca/elpaca-manager.el... Compiling /tmp/elpaca.rtJtUg/elpaca/repos/elpaca/elpaca-menu-elpa.el... Compiling /tmp/elpaca.rtJtUg/elpaca/repos/elpaca/elpaca-menu-melpa.el... Compiling /tmp/elpaca.rtJtUg/elpaca/repos/elpaca/elpaca-menu-org.el... Compiling /tmp/elpaca.rtJtUg/elpaca/repos/elpaca/elpaca-process.el... Compiling /tmp/elpaca.rtJtUg/elpaca/repos/elpaca/elpaca-test.el... Compiling /tmp/elpaca.rtJtUg/elpaca/repos/elpaca/elpaca-ui.el... Compiling /tmp/elpaca.rtJtUg/elpaca/repos/elpaca/elpaca.el... Checking /tmp/elpaca.rtJtUg/elpaca/repos/elpaca/doc... Compiling /tmp/elpaca.rtJtUg/elpaca/repos/elpaca/doc/early-init.el... Compiling /tmp/elpaca.rtJtUg/elpaca/repos/elpaca/doc/init.el... Checking /tmp/elpaca.rtJtUg/elpaca/repos/elpaca/extensions... Compiling /tmp/elpaca.rtJtUg/elpaca/repos/elpaca/extensions/elpaca-use-package.el... Checking /tmp/elpaca.rtJtUg/elpaca/repos/elpaca/images... Checking /tmp/elpaca.rtJtUg/elpaca/repos/elpaca/test... Compiling /tmp/elpaca.rtJtUg/elpaca/repos/elpaca/test/elpaca-test.el... Done (Total of 11 files compiled, 3 skipped in 4 directories) Downloading MELPA recipes... Downloading MELPA recipes...100% Downloading NonGNU-devel ELPA... Downloading GNU-devel ELPA... Downloading NonGNU ELPA... Downloading GNU ELPA... elpaca-log: "#unique" " seq finished ✓ 2.684 secs 13.568629 elpaca finished ✓ 2.995 secs 13.876013 compat finished ✓ 5.071 secs 17.447952 with-editor finished ✓ 6.044 secs 18.308732 transient finished ✓ 6.613 secs 19.018036 dash finished ✓ 6.975 secs 19.279611 git-commit finished ✓ 8.087 secs 20.342451 magit-section finished ✓ 8.613 secs 20.923314 magit finished ✓ 32.343 secs 43.224765 " Test Env Elpaca 09a94ca HEAD -> fix/date-version, origin/fix/date-version installer: 0.6 emacs-version: GNU Emacs 29.1 (build 1, x86_64-pc-linux-gnu, cairo version 1.18.0) of 2024-02-08 git --version: git version 2.43.0 ```
progfolio commented 8 months ago

viri @.***> writes:

(elpaca (` (seq :build (, (+elpaca-seq-build-steps)))))

I imagine this part of the test is meant to be (elpaca `(seq :build , (+elpaca-seq-build-steps))) like in my above user-init-file, otherwise that elisp gets a classic "wrong argument type: symbolp" et cetera, so I edited that part of the expression.

Yes. That looks like it's an elisp pretty printer bug or limitation. Thanks for pointing that out. I'll look into it.

Here are the results, and I did check to put log.showSignature=true in my ~/.gitconfig beforehand; elpaca-log: "#unique" " seq finished ✓ 2.684 secs 13.568629 elpaca finished ✓ 2.995 secs 13.876013 compat finished ✓ 5.071 secs 17.447952 with-editor finished ✓ 6.044 secs 18.308732 transient finished ✓ 6.613 secs 19.018036 dash finished ✓ 6.975 secs 19.279611 git-commit finished ✓ 8.087 secs 20.342451 magit-section finished ✓ 8.613 secs 20.923314 magit finished ✓ 32.343 secs 43.224765 "

Test Env

Elpaca 09a94ca HEAD -> fix/date-version, origin/fix/date-version installer: 0.6 emacs-version: GNU Emacs 29.1 (build 1, x86_64-pc-linux-gnu, cairo version 1.18.0) of 2024-02-08 git --version: git version 2.43.0

Looks like everything is working as expected. I've merged the change into the master branch. Thanks again for testing.