GNU bug report logs -
#42736
[PATCH] gnu: emacs-doom-themes: Update to 2.1.6-5.
Previous Next
Reported by: Jack Hill <jackhill <at> jackhill.us>
Date: Fri, 7 Aug 2020 03:19:01 UTC
Severity: normal
Tags: patch
Done: Brett Gilio <brettg <at> gnu.org>
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 42736 in the body.
You can then email your comments to 42736 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#42736
; Package
guix-patches
.
(Fri, 07 Aug 2020 03:19:02 GMT)
Full text and
rfc822 format available.
Acknowledgement sent
to
Jack Hill <jackhill <at> jackhill.us>
:
New bug report received and forwarded. Copy sent to
guix-patches <at> gnu.org
.
(Fri, 07 Aug 2020 03:19:02 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-doom-themes): Update to 2.1.6-5. This
commit revision add support for Emacs 27.
---
gnu/packages/emacs-xyz.scm | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index f857b594eb..7dc8e42f81 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -22053,8 +22053,8 @@ contrast and few colors.")
(license license:gpl3+))))
(define-public emacs-doom-themes
- (let ((commit "54039c5171e3f8c9cef1f82122549b66cd8c8f7b")
- (revision "4")
+ (let ((commit "e803fc4ac8cf7118e2d1544d8241b848b5e79e9f")
+ (revision "5")
(version "2.1.6"))
(package
(name "emacs-doom-themes")
@@ -22066,7 +22066,7 @@ contrast and few colors.")
(commit commit)))
(file-name (git-file-name name version))
(sha256
- (base32 "1iwdjq4q2gkhi6jwas3ywgmdz5dg14sfb3fzhqd7wih6j3i2l3cr"))))
+ (base32 "128hdmf0jkzr12fv2r6z349qiwba6q97hsb6b1n2qlhi0v5v3mfh"))))
(build-system emacs-build-system)
(native-inputs
`(("emacs-ert-runner" ,emacs-ert-runner)))
--
2.28.0
Information forwarded
to
guix-patches <at> gnu.org
:
bug#42736
; Package
guix-patches
.
(Fri, 07 Aug 2020 03:38:02 GMT)
Full text and
rfc822 format available.
Message #8 received at 42736 <at> debbugs.gnu.org (full text, mbox):
Jack Hill <jackhill <at> jackhill.us> writes:
> * gnu/packages/emacs-xyz.scm (emacs-doom-themes): Update to 2.1.6-5. This
> commit revision add support for Emacs 27.
> ---
> gnu/packages/emacs-xyz.scm | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
> index f857b594eb..7dc8e42f81 100644
> --- a/gnu/packages/emacs-xyz.scm
> +++ b/gnu/packages/emacs-xyz.scm
> @@ -22053,8 +22053,8 @@ contrast and few colors.")
> (license license:gpl3+))))
>
> (define-public emacs-doom-themes
> - (let ((commit "54039c5171e3f8c9cef1f82122549b66cd8c8f7b")
> - (revision "4")
> + (let ((commit "e803fc4ac8cf7118e2d1544d8241b848b5e79e9f")
> + (revision "5")
> (version "2.1.6"))
> (package
> (name "emacs-doom-themes")
> @@ -22066,7 +22066,7 @@ contrast and few colors.")
> (commit commit)))
> (file-name (git-file-name name version))
> (sha256
> - (base32 "1iwdjq4q2gkhi6jwas3ywgmdz5dg14sfb3fzhqd7wih6j3i2l3cr"))))
> + (base32 "128hdmf0jkzr12fv2r6z349qiwba6q97hsb6b1n2qlhi0v5v3mfh"))))
> (build-system emacs-build-system)
> (native-inputs
> `(("emacs-ert-runner" ,emacs-ert-runner)))
Hey Jack,
Thanks for taking time to revise this package. When I originally wrote
it I made notice to the fact that some elisp bytecompilations were
failing or not behaving appropriately. Since then I am pretty sure
hlissner has disabled the bytecompilation completely? Could you review
this for me, and if true please revise the appropriate arguments. If you
aren't sure what I am talking about, please let me know.
Thanks,
Brett Gilio
Information forwarded
to
guix-patches <at> gnu.org
:
bug#42736
; Package
guix-patches
.
(Fri, 07 Aug 2020 04:30:02 GMT)
Full text and
rfc822 format available.
Message #11 received at 42736 <at> debbugs.gnu.org (full text, mbox):
On Thu, 6 Aug 2020, Brett Gilio wrote:
>
> Hey Jack,
>
> Thanks for taking time to revise this package. When I originally wrote
> it I made notice to the fact that some elisp bytecompilations were
> failing or not behaving appropriately. Since then I am pretty sure
> hlissner has disabled the bytecompilation completely? Could you review
> this for me, and if true please revise the appropriate arguments. If you
> aren't sure what I am talking about, please let me know.
Brett,
I saw your lovely comment, but in my excitement that the update solved the
problem I was having with Emacs 27 compatibility, I didn't think too hard
about it.
I believe that I understand what you're asking. I'll take a look tomorrow
to see if we should change the package definition in light of upstream
changes. If I get stuck, I'll be sure to let you know.
Best,
Jack
Information forwarded
to
guix-patches <at> gnu.org
:
bug#42736
; Package
guix-patches
.
(Fri, 07 Aug 2020 18:11:02 GMT)
Full text and
rfc822 format available.
Message #14 received at 42736 <at> debbugs.gnu.org (full text, mbox):
Jack Hill <jackhill <at> jackhill.us> writes:
> On Thu, 6 Aug 2020, Brett Gilio wrote:
>
>>
>> Hey Jack,
>>
>> Thanks for taking time to revise this package. When I originally wrote
>> it I made notice to the fact that some elisp bytecompilations were
>> failing or not behaving appropriately. Since then I am pretty sure
>> hlissner has disabled the bytecompilation completely? Could you review
>> this for me, and if true please revise the appropriate arguments. If you
>> aren't sure what I am talking about, please let me know.
>
> Brett,
>
> I saw your lovely comment, but in my excitement that the update solved
> the problem I was having with Emacs 27 compatibility, I didn't think
> too hard about it.
>
> I believe that I understand what you're asking. I'll take a look
> tomorrow to see if we should change the package definition in light of
> upstream changes. If I get stuck, I'll be sure to let you know.
>
> Best,
> Jack
Sounds great! Thanks.
Information forwarded
to
guix-patches <at> gnu.org
:
bug#42736
; Package
guix-patches
.
(Sat, 08 Aug 2020 02:10:02 GMT)
Full text and
rfc822 format available.
Message #17 received at 42736 <at> debbugs.gnu.org (full text, mbox):
>> On Thu, 6 Aug 2020, Brett Gilio wrote:
>>
>>>
>>> Hey Jack,
>>>
>>> Thanks for taking time to revise this package. When I originally wrote
>>> it I made notice to the fact that some elisp bytecompilations were
>>> failing or not behaving appropriately. Since then I am pretty sure
>>> hlissner has disabled the bytecompilation completely? Could you review
>>> this for me, and if true please revise the appropriate arguments. If you
>>> aren't sure what I am talking about, please let me know.
Brett,
I think the way forward is to follow upstream's choices and not enable or
disable byte compilation in Guix.
After upstream introduced commit 9cd6872 [0], our trick to selectively
leave batch compilation enabled for some files didn't work because they
already had `-*- no-byte-compile: t; -*-` at the top of the file. In my
testing, I added a phase to substitute this away. Indeed, this allowed our
trick to work again. However, the material, snazzy, and tomorrow-day
themes now have problems with byte compilation.
Therefore, I removed the disable-breaking-compilation phase entirely. With
it removed, doom-themes-autoloads.el, doom-themes-base.el, doom-themes.el,
doom-themes-ext-org.el, and doom-themes-ext-visual-bell.el do get byte
compiled. From this evidence I concluded that upstream is aware of this
issue and is only disabling byte compilation where necessary.
I'll send a version 2 of the patch with the phase removed shortly.
[0] https://github.com/hlissner/emacs-doom-themes/commit/9cd6872b1af88165834230abd45743036861f925
Best,
Jack
Information forwarded
to
guix-patches <at> gnu.org
:
bug#42736
; Package
guix-patches
.
(Sat, 08 Aug 2020 02:12:01 GMT)
Full text and
rfc822 format available.
Message #20 received at 42736 <at> debbugs.gnu.org (full text, mbox):
* gnu/packages/emacs-xyz.scm (emacs-doom-themes): Update to 2.1.6-5. This
commit revision add support for Emacs 27.
[arguments]: Remove disable-breaking-compilation phase.
---
gnu/packages/emacs-xyz.scm | 31 +++----------------------------
1 file changed, 3 insertions(+), 28 deletions(-)
diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index f857b594eb..0b66e9507b 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -22053,8 +22053,8 @@ contrast and few colors.")
(license license:gpl3+))))
(define-public emacs-doom-themes
- (let ((commit "54039c5171e3f8c9cef1f82122549b66cd8c8f7b")
- (revision "4")
+ (let ((commit "e803fc4ac8cf7118e2d1544d8241b848b5e79e9f")
+ (revision "5")
(version "2.1.6"))
(package
(name "emacs-doom-themes")
@@ -22066,7 +22066,7 @@ contrast and few colors.")
(commit commit)))
(file-name (git-file-name name version))
(sha256
- (base32 "1iwdjq4q2gkhi6jwas3ywgmdz5dg14sfb3fzhqd7wih6j3i2l3cr"))))
+ (base32 "128hdmf0jkzr12fv2r6z349qiwba6q97hsb6b1n2qlhi0v5v3mfh"))))
(build-system emacs-build-system)
(native-inputs
`(("emacs-ert-runner" ,emacs-ert-runner)))
@@ -22086,31 +22086,6 @@ contrast and few colors.")
(for-each (lambda (f)
(rename-file f (basename f)))
(find-files "./themes" ".*\\.el$"))
- #t))
- ;; There is a byte-code overflow issue in the latest checkout
- ;; which affects byte-compilation for several (read `most') theme
- ;; files. In order to cope with this issue, we disable
- ;; byte-compilation until this issue is resolved.
- ;; <https://github.com/hlissner/emacs-doom-themes/issues/314>
- ;;
- ;; NOTE: Byte-comp has been disabled in/after commit 9cd6872.
- ;; However our method of selective disabling is preferential to
- ;; just widely disabling byte-compilation.
- (add-after 'move-themes 'disable-breaking-compilation
- (lambda _
- (for-each (lambda (file)
- (chmod file #o600) ; needed to write changes.
- (emacs-batch-disable-compilation file))
- (cons "doom-themes-ext-neotree.el"
- ;; NOTE: When updating this package, determine
- ;; whether changed theme files can byte-compile.
- ;; If they can successfully byte-compile, add them
- ;; to this list of exceptions.
- (lset-difference string-contains
- (find-files "." ".*-theme.el")
- '("material"
- "snazzy"
- "tomorrow-day"))))
#t)))))
(synopsis "Wide collection of color themes for Emacs")
(description "Emacs-doom-themes contains numerous popular color themes for
--
2.28.0
Information forwarded
to
guix-patches <at> gnu.org
:
bug#42736
; Package
guix-patches
.
(Sat, 08 Aug 2020 02:28:02 GMT)
Full text and
rfc822 format available.
Message #23 received at 42736 <at> debbugs.gnu.org (full text, mbox):
Jack Hill <jackhill <at> jackhill.us> writes:
>>> On Thu, 6 Aug 2020, Brett Gilio wrote:
>>>
>>>>
>>>> Hey Jack,
>>>>
>>>> Thanks for taking time to revise this package. When I originally wrote
>>>> it I made notice to the fact that some elisp bytecompilations were
>>>> failing or not behaving appropriately. Since then I am pretty sure
>>>> hlissner has disabled the bytecompilation completely? Could you review
>>>> this for me, and if true please revise the appropriate arguments. If you
>>>> aren't sure what I am talking about, please let me know.
>
> Brett,
>
> I think the way forward is to follow upstream's choices and not enable
> or disable byte compilation in Guix.
>
> After upstream introduced commit 9cd6872 [0], our trick to selectively
> leave batch compilation enabled for some files didn't work because
> they already had `-*- no-byte-compile: t; -*-` at the top of the
> file. In my testing, I added a phase to substitute this away. Indeed,
> this allowed our trick to work again. However, the material, snazzy,
> and tomorrow-day themes now have problems with byte compilation.
>
> Therefore, I removed the disable-breaking-compilation phase
> entirely. With it removed, doom-themes-autoloads.el,
> doom-themes-base.el, doom-themes.el, doom-themes-ext-org.el, and
> doom-themes-ext-visual-bell.el do get byte compiled. From this
> evidence I concluded that upstream is aware of this issue and is only
> disabling byte compilation where necessary.
>
> I'll send a version 2 of the patch with the phase removed shortly.
>
> [0] https://github.com/hlissner/emacs-doom-themes/commit/9cd6872b1af88165834230abd45743036861f925
>
> Best,
> Jack
Jack,
Thank you for investigating this issue. I agree with your solution, and
I will apply your patch removing the phase.
Brett
Reply sent
to
Brett Gilio <brettg <at> gnu.org>
:
You have taken responsibility.
(Sat, 08 Aug 2020 02:30:01 GMT)
Full text and
rfc822 format available.
Notification sent
to
Jack Hill <jackhill <at> jackhill.us>
:
bug acknowledged by developer.
(Sat, 08 Aug 2020 02:30:02 GMT)
Full text and
rfc822 format available.
Message #28 received at 42736-close <at> debbugs.gnu.org (full text, mbox):
Applied with c9eb5ad6b0443fafe1cd1a18aae507c1ee5fea1f. Closing.
Thanks!
Information forwarded
to
guix-patches <at> gnu.org
:
bug#42736
; Package
guix-patches
.
(Sat, 08 Aug 2020 02:41:02 GMT)
Full text and
rfc822 format available.
Message #31 received at 42736 <at> debbugs.gnu.org (full text, mbox):
On Fri, 7 Aug 2020, Brett Gilio wrote:
> Jack,
>
> Thank you for investigating this issue. I agree with your solution, and
> I will apply your patch removing the phase.
>
> Brett
Thanks for the review!
Best,
Jack
bug archived.
Request was from
Debbugs Internal Request <help-debbugs <at> gnu.org>
to
internal_control <at> debbugs.gnu.org
.
(Sat, 05 Sep 2020 11:24:04 GMT)
Full text and
rfc822 format available.
This bug report was last modified 3 years and 205 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.