GNU bug report logs -
#55152
[PATCH] gnu: Add emacs-soothe-theme.
Previous Next
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.
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):
* 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):
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):
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.