* [bug#62488] [PATCH] Update emacs-lemon
2023-03-27 17:23 ` Nicolas Goaziou
@ 2023-03-29 0:06 ` Baptiste Strazzulla
2023-03-30 10:37 ` bug#62488: " Nicolas Goaziou
0 siblings, 1 reply; 4+ messages in thread
From: Baptiste Strazzulla @ 2023-03-29 0:06 UTC (permalink / raw)
To: Nicolas Goaziou; +Cc: 62488@debbugs.gnu.org
[-- Attachment #1.1: Type: text/plain, Size: 1159 bytes --]
Hello,
Here is two requested updates to the patch
Kind regards,
Baptiste
________________________________
From: Nicolas Goaziou <mail@nicolasgoaziou.fr>
Sent: Monday, March 27, 2023 5:23 PM
To: Baptiste Strazzulla <bstrazzull@hotmail.fr>
Cc: 62488@debbugs.gnu.org <62488@debbugs.gnu.org>
Subject: Re: [bug#62488] [PATCH] Update emacs-lemon
Hello,
Baptiste Strazzulla <bstrazzull@hotmail.fr> writes:
> * gnu/packages/emacs-xyz.scm (emacs-lemon-version): Update to 2.1.0
Thank you.
> (define-public emacs-lemon
> - (let ((commit "37a6e6d6ef0900ca19c820a2dbc122c7fe6d86cf")
> + (let ((commit "b10e992a25757a91723c7d554a5fa8e14291e702")
> (revision "0"))
> (package
> (name "emacs-lemon")
> - (version (git-version "2.0.0" revision commit))
> + (version (git-version "2.1.0" revision commit))
Since the commit matches the version bump, you can drop the `revision'
binding and `git-version' call. Version is simply "2.1.0". Could you
send an updated patch?
I noticed the package contains some tests. Would you try to run them, in
a subsequent patch?
Regards,
--
Nicolas Goaziou
[-- Attachment #1.2: Type: text/html, Size: 3280 bytes --]
[-- Attachment #2: 0001-gnu-emacs-lemon-Update-to-2.1.0.patch --]
[-- Type: application/octet-stream, Size: 8829 bytes --]
From 1be9d8560983a3dea2b0814a52b48557a3ed6f6e Mon Sep 17 00:00:00 2001
From: Zzull <bstrazzull@hotmail.fr>
Date: Mon, 27 Mar 2023 18:28:01 +0200
Subject: [PATCH 1/2] gnu: emacs-lemon: Update to 2.1.0.
* gnu/packages/emacs-xyz.scm (emacs-lemon-version): Update to 2.1.0
---
gnu/packages/emacs-xyz.scm | 112 ++++++++++++++++++-------------------
1 file changed, 55 insertions(+), 57 deletions(-)
diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index da2856abc8..0fb31f74f7 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -49,7 +49,7 @@
;;; Copyright © 2019, 2020, 2021 Joseph LaFreniere <joseph@lafreniere.xyz>
;;; Copyright © 2019 Todor Kondić <tk.code@protonmail.com>15669
;;; Copyright © 2019 Amar Singh <nly@disroot.org>
-;;; Copyright © 2019, 2022 Baptiste Strazzulla <bstrazzull@hotmail.fr>
+;;; Copyright © 2019, 2022, 2023 Baptiste Strazzulla <bstrazzull@hotmail.fr>
;;; Copyright © 2019 Giacomo Leidi <goodoldpaul@autistici.org>
;;; Copyright © 2019 Jens Mølgaard <jens@zete.tk>
;;; Copyright © 2019, 2020 Amin Bandali <bandali@gnu.org>
@@ -5363,33 +5363,31 @@ (define-public emacs-symon
(license license:gpl2+)))
(define-public emacs-lemon
- (let ((commit "37a6e6d6ef0900ca19c820a2dbc122c7fe6d86cf")
- (revision "0"))
- (package
- (name "emacs-lemon")
- (version (git-version "2.0.0" revision commit))
- (source
- (origin
- (method git-fetch)
- (uri (git-reference
- (url "https://codeberg.org/emacs-weirdware/lemon")
- (commit commit)))
- (file-name (git-file-name name version))
- (sha256
- (base32 "0bc77vzi4p6mlzmhgybbldlpcsiiv4xqrd5lnc7wzvmxv8byhqpm"))))
- (build-system emacs-build-system)
- (native-inputs
- (list emacs-blight emacs-emms))
- (propagated-inputs
- (list emacs-s))
- (home-page "https://codeberg.org/emacs-weirdware/lemon")
- (synopsis "System monitors in the echo area")
- (description
- "Lemon is a tiny system monitor which displays system information in
+ (package
+ (name "emacs-lemon")
+ (version "2.1.0")
+ (source
+ (origin
+ (method git-fetch)
+ (uri (git-reference
+ (url "https://codeberg.org/emacs-weirdware/lemon")
+ (commit "b10e992a25757a91723c7d554a5fa8e14291e702")))
+ (file-name (git-file-name name version))
+ (sha256
+ (base32 "18yclk2zzqcahzhz2kq8g9gy1xnnxiy6rxs2lwhskj475kvwy2f9"))))
+ (build-system emacs-build-system)
+ (native-inputs
+ (list emacs-blight emacs-emms))
+ (propagated-inputs
+ (list emacs-s))
+ (home-page "https://codeberg.org/emacs-weirdware/lemon")
+ (synopsis "System monitors in the echo area")
+ (description
+ "Lemon is a tiny system monitor which displays system information in
the echo area when Emacs is has been idle for a few seconds. This is a fork
of zk_phi’s Symon, which has been largely rewritten. It works nicely with
EXWM.")
- (license license:gpl3+))))
+ (license license:gpl3+)))
(define-public emacs-sx
(let ((version "20191229")
@@ -7144,7 +7142,7 @@ (define-public emacs-mode-line-idle
;; Package has no release. Version is extracted from "Version:" keyword in
;; main file.
(let ((commit "02b1da6278e43cc9cc0356110cc6bfbb37eb8241")
- (revision "1"))
+ (revision "1"))
(package
(name "emacs-mode-line-idle")
(version (git-version "0.1" revision commit))
@@ -7228,7 +7226,7 @@ (define-public emacs-smart-mode-line
(define-public emacs-sqlite
;; XXX: There is no tagged commit.
(let ((commit "dad42b8bbca4994be1871343dd18fd6528ee5797")
- (revision "0"))
+ (revision "0"))
(package
(name "emacs-sqlite")
(build-system emacs-build-system)
@@ -7237,8 +7235,8 @@ (define-public emacs-sqlite
(origin
(method git-fetch)
(uri (git-reference
- (url "https://gitlab.com/cnngimenez/sqlite.el")
- (commit commit)))
+ (url "https://gitlab.com/cnngimenez/sqlite.el")
+ (commit commit)))
(file-name (git-file-name name version))
(sha256
(base32 "06ln4vijl8kii3nzc5cscgsadx1fqgxksflijd3ain83bn8g4wrd"))))
@@ -9023,7 +9021,7 @@ (define-public emacs-scel
(define-public emacs-soothe-theme
;; There is no named branch.
(let ((commit "0786fe70c6c1b4ddcfb932fdc6862b9611cfc09b")
- (revision "0"))
+ (revision "0"))
(package
(name "emacs-soothe-theme")
(version (git-version "20141027.2233" revision commit))
@@ -9031,11 +9029,11 @@ (define-public emacs-soothe-theme
(origin
(method git-fetch)
(uri (git-reference
- (url "https://github.com/emacsfodder/emacs-soothe-theme")
- (commit commit)))
+ (url "https://github.com/emacsfodder/emacs-soothe-theme")
+ (commit commit)))
(file-name (git-file-name name version))
(sha256
- (base32 "10gh1hvxq9gm29r6qzlnva7vjidd7n4kih4z2ihyvbvy9za20xqw"))))
+ (base32 "10gh1hvxq9gm29r6qzlnva7vjidd7n4kih4z2ihyvbvy9za20xqw"))))
(build-system emacs-build-system)
(home-page "https://github.com/emacsfodder/emacs-soothe-theme")
(synopsis "Colorful, but muted theme for Emacs, dark background with light text")
@@ -18141,12 +18139,12 @@ (define-public emacs-ert-runner
(origin
(method git-fetch)
(uri (git-reference
- (url "https://github.com/rejeep/ert-runner.el")
- (commit (string-append "v" version))))
+ (url "https://github.com/rejeep/ert-runner.el")
+ (commit (string-append "v" version))))
(file-name (git-file-name name version))
(sha256
- (base32
- "08gygn9fjank5gpi4v6ynrkn0jbknxbwsn7md4p9ndygdbmnkf98"))))
+ (base32
+ "08gygn9fjank5gpi4v6ynrkn0jbknxbwsn7md4p9ndygdbmnkf98"))))
(build-system emacs-build-system)
(inputs
(list emacs-ansi
@@ -18158,22 +18156,22 @@ (define-public emacs-ert-runner
(arguments
`(#:phases
(modify-phases %standard-phases
- (add-after 'install 'install-executable
- (lambda* (#:key inputs outputs #:allow-other-keys)
- (let ((out (assoc-ref outputs "out"))
- (source-directory (string-append
- (getenv "TMPDIR") "/source")))
- (substitute* "bin/ert-runner"
- (("ERT_RUNNER=\"\\$\\(dirname \\$\\(dirname \\$0\\)\\)")
- (string-append "ERT_RUNNER=\"" (elpa-directory out))))
- (install-file "bin/ert-runner" (string-append out "/bin"))
- (wrap-program (string-append out "/bin/ert-runner")
- (list "EMACSLOADPATH" ":" 'prefix
- ;; Do not capture the transient source directory in
- ;; the wrapper.
- (delete source-directory
- (string-split (getenv "EMACSLOADPATH") #\:))))
- #t))))
+ (add-after 'install 'install-executable
+ (lambda* (#:key inputs outputs #:allow-other-keys)
+ (let ((out (assoc-ref outputs "out"))
+ (source-directory (string-append
+ (getenv "TMPDIR") "/source")))
+ (substitute* "bin/ert-runner"
+ (("ERT_RUNNER=\"\\$\\(dirname \\$\\(dirname \\$0\\)\\)")
+ (string-append "ERT_RUNNER=\"" (elpa-directory out))))
+ (install-file "bin/ert-runner" (string-append out "/bin"))
+ (wrap-program (string-append out "/bin/ert-runner")
+ (list "EMACSLOADPATH" ":" 'prefix
+ ;; Do not capture the transient source directory in
+ ;; the wrapper.
+ (delete source-directory
+ (string-split (getenv "EMACSLOADPATH") #\:))))
+ #t))))
#:include (cons* "^reporters/.*\\.el$" %default-include)))
(home-page "https://github.com/rejeep/ert-runner.el")
(synopsis "Opinionated Ert testing workflow")
@@ -19966,8 +19964,8 @@ (define-public emacs-powershell
(origin
(method git-fetch)
(uri (git-reference
- (url "https://github.com/jschaf/powershell.el")
- (commit commit)))
+ (url "https://github.com/jschaf/powershell.el")
+ (commit commit)))
(file-name (git-file-name name version))
(sha256
(base32 "1cxhzaaig88zhylyycvb3849r85j1ijqklnh9zbqsfl2zhpb0g5c"))))
@@ -33210,8 +33208,8 @@ (define-public emacs-helm-xref
(define-public emacs-metal-mercury-mode
(let ((commit "99e2d8fb7177cae3bfa2dec2910fc28216d5f5a8")
- (revision "1")
- (version "0.0.0"))
+ (revision "1")
+ (version "0.0.0"))
(package
(name "emacs-metal-mercury-mode")
(version (git-version version revision commit))
--
2.39.2
[-- Attachment #3: 0002-gnu-emacs-lemon-Add-tests.patch --]
[-- Type: application/octet-stream, Size: 1195 bytes --]
From 6ade7ae9d0029cd64e65047134242ac8898d75c3 Mon Sep 17 00:00:00 2001
From: Zzull <bstrazzull@hotmail.fr>
Date: Wed, 29 Mar 2023 02:02:24 +0200
Subject: [PATCH 2/2] gnu: emacs-lemon: Add tests
---
gnu/packages/emacs-xyz.scm | 11 +++++++++++
1 file changed, 11 insertions(+)
diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 0fb31f74f7..c6cd65e13c 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -5376,6 +5376,17 @@ (define-public emacs-lemon
(sha256
(base32 "18yclk2zzqcahzhz2kq8g9gy1xnnxiy6rxs2lwhskj475kvwy2f9"))))
(build-system emacs-build-system)
+ (arguments
+ (list
+ #:tests? #true
+
+ ;; Only one test out of the four passes
+ #:test-command #~(list "emacs" "-Q" "--batch" "-L" "."
+ ;; "-l" "lemon--test.el"
+ ;; "-l" "lemon-monitor--test.el"
+ ;; "-l" "lemon-sparkline--test.el"
+ "-l" "lemon-time--test.el"
+ "-f" "ert-run-tests-batch-and-exit")))
(native-inputs
(list emacs-blight emacs-emms))
(propagated-inputs
--
2.39.2
^ permalink raw reply related [flat|nested] 4+ messages in thread