GNU bug report logs - #55152
[PATCH] gnu: Add emacs-soothe-theme.

Previous Next

Package: guix-patches;

Reported by: Adam Kandur <kefironpremise <at> gmail.com>

Date: Wed, 27 Apr 2022 14:32:02 UTC

Severity: normal

Tags: patch

Done: Nicolas Goaziou <mail <at> nicolasgoaziou.fr>

Bug is archived. No further changes may be made.

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 55152 in the body.
You can then email your comments to 55152 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to guix-patches <at> gnu.org:
bug#55152; Package guix-patches. (Wed, 27 Apr 2022 14:32:03 GMT) Full text and rfc822 format available.

Acknowledgement sent to Adam Kandur <kefironpremise <at> gmail.com>:
New bug report received and forwarded. Copy sent to guix-patches <at> gnu.org. (Wed, 27 Apr 2022 14:32:03 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Adam Kandur <kefironpremise <at> gmail.com>
To: guix-patches <at> gnu.org
Cc: Adam Kandur <kefironpremise <at> gmail.com>
Subject: [PATCH] gnu: Add emacs-soothe-theme.
Date: Wed, 27 Apr 2022 16:31:37 +0300
* gnu/packages/emacs-xyz.scm (emacs-soothe-theme): New variable.
---
 gnu/packages/emacs-xyz.scm | 24 ++++++++++++++++++++++++
 1 file changed, 24 insertions(+)

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index db1663c..34658e4 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -1482,6 +1482,30 @@ (define-public emacs-solidity
 a constant work in progress as the language itself also progresses.")
       (license license:gpl3+))))
 
+(define-public emacs-soothe-theme
+  (let ((commit "0786fe70c6c1b4ddcfb932fdc6862b9611cfc09b")
+        (revision "0"))
+    (package
+      (name "emacs-soothe-theme")
+      (version (git-version "0.1" revision commit))
+      (source
+       (origin
+	 (method git-fetch)
+	 (uri (git-reference
+	       (url "https://github.com/emacsfodder/emacs-soothe-theme.git")
+	       (commit commit)))
+	 (file-name (git-file-name name version))
+	 (sha256
+	  (base32 "10gh1hvxq9gm29r6qzlnva7vjidd7n4kih4z2ihyvbvy9za20xqw"))))
+      (build-system emacs-build-system)
+      (home-page "https://github.com/emacsfodder/emacs-soothe-theme.git")
+      (synopsis "A colorful, but muted theme for Emacs 24, dark background with light text")
+      (description
+       "An amalgam of muted color tones and highlighted backgrounds.  Builtin support 
+for rainbow-delimiters, org-mode, whitespace-mode, ECB, flyspell, ido, linum, highlight
+indentation, show-paren-mode, further mode support to come.")
+      (license license:gpl3+))))
+
 (define-public emacs-unpackaged-el
   (let ((commit "746801a677ada6cd6fa076e423aa0953779f3fad")
         (revision "3"))
-- 
2.35.1





Information forwarded to guix-patches <at> gnu.org:
bug#55152; Package guix-patches. (Mon, 27 Jun 2022 10:28:01 GMT) Full text and rfc822 format available.

Message #8 received at 55152 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Goaziou <mail <at> nicolasgoaziou.fr>
To: Adam Kandur <kefironpremise <at> gmail.com>
Cc: 55152 <at> debbugs.gnu.org
Subject: Re: [bug#55152] [PATCH] gnu: Add emacs-soothe-theme.
Date: Mon, 27 Jun 2022 12:27:29 +0200
Hello,

Adam Kandur <kefironpremise <at> gmail.com> writes:

> * gnu/packages/emacs-xyz.scm (emacs-soothe-theme): New variable.

Thank you. Sorry for the delay. Some comments follow.

> +(define-public emacs-soothe-theme
> +  (let ((commit "0786fe70c6c1b4ddcfb932fdc6862b9611cfc09b")
> +        (revision "0"))
> +    (package
> +      (name "emacs-soothe-theme")
> +      (version (git-version "0.1" revision commit))

Where does this "0.1" come from? According to "soothe-theme.el" and
Changelog, version is 20141027.2233.

Also, I suggest to stick to commit hash
0568a61eeec0b074d8911886359a6c5da13d14cb, which is the penultimate
commit, but matches version bump. You don't need to use `git-version'
then.

> +      (source
> +       (origin
> +	 (method git-fetch)
> +	 (uri (git-reference
> +	       (url "https://github.com/emacsfodder/emacs-soothe-theme.git")

You can drop the ".git" suffix from the URL.

> +	       (commit commit)))
> +	 (file-name (git-file-name name version))
> +	 (sha256
> +	  (base32 "10gh1hvxq9gm29r6qzlnva7vjidd7n4kih4z2ihyvbvy9za20xqw"))))
> +      (build-system emacs-build-system)
> +      (home-page "https://github.com/emacsfodder/emacs-soothe-theme.git")
> +      (synopsis "A colorful, but muted theme for Emacs 24, dark background with light text")

Synopsis must not start with an article. I would also remove reference
to Emacs 24, which is unimportant in Guix.

> +      (description
> +       "An amalgam of muted color tones and highlighted backgrounds.  Builtin support 
> +for rainbow-delimiters, org-mode, whitespace-mode, ECB, flyspell, ido, linum, highlight
> +indentation, show-paren-mode, further mode support to come.")

Description must contain complete sentence only. So it should be "Soothe
theme is an amalgam…".

Also (nitpick) major mode are usually capitalized and un-hyphenized at
least in the Emacs manual. So: Rainbom delimiters, Org mode, Whitespace
mode, Flyspell, Ido, Linum…

Since the project hasn't seen any commit in the last 7 years, maybe
"further mode support to come" should be removed.

> +      (license license:gpl3+))))

License seems to be gpl3, not gpl3+.

Could you send an updated patch?

Regards,
-- 
Nicolas Goaziou




Reply sent to Nicolas Goaziou <mail <at> nicolasgoaziou.fr>:
You have taken responsibility. (Wed, 20 Jul 2022 13:03:01 GMT) Full text and rfc822 format available.

Notification sent to Adam Kandur <kefironpremise <at> gmail.com>:
bug acknowledged by developer. (Wed, 20 Jul 2022 13:03:02 GMT) Full text and rfc822 format available.

Message #13 received at 55152-done <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Goaziou <mail <at> nicolasgoaziou.fr>
To: Adam Kandur <kefironpremise <at> gmail.com>
Cc: 55152-done <at> debbugs.gnu.org
Subject: Re: [bug#55152] [PATCH] gnu: Add emacs-soothe-theme.
Date: Wed, 20 Jul 2022 15:01:51 +0200
Hello,

You sent me the updated patch in private. I applied it. Thank you.

Closing,
-- 
Nicolas Goaziou




bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Thu, 18 Aug 2022 11:24:04 GMT) Full text and rfc822 format available.

This bug report was last modified 1 year and 224 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.