[PATCH] gnu: emacs-modus-vivendi-theme: Update to 0.13.0.

  • Done
  • quality assurance status badge
Details
5 participants
  • Morgan.J.Smith
  • André Alexandre Gomes
  • Protesilaos Stavrou
  • Ludovic Courtès
  • Nicolas Goaziou
Owner
unassigned
Submitted by
Morgan.J.Smith
Severity
normal

Debbugs page

Morgan.J.Smith wrote 4 years ago
(address . guix-patches@gnu.org)(name . Morgan Smith)(address . Morgan.J.Smith@outlook.com)
DM5PR1001MB210520D021E476DE3ACB101BC5090@DM5PR1001MB2105.namprd10.prod.outlook.com
From: Morgan Smith <Morgan.J.Smith@outlook.com>

* gnu/packages/emacs-xyz.scm (emacs-modus-vivendi-theme): Update to 0.13.0.
[source]: Use git instead of elpa, as elpa is out of date
[arguments]: Add phase to remove modus-operandi-theme.el
---
gnu/packages/emacs-xyz.scm | 20 +++++++++++++++-----
1 file changed, 15 insertions(+), 5 deletions(-)

Toggle diff (37 lines)
diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 44aaaa0541..f1ebdd90ff 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -22685,15 +22685,25 @@ standard. This is the highest standard of its kind.")
(define-public emacs-modus-vivendi-theme
(package
(name "emacs-modus-vivendi-theme")
- (version "0.12.0")
+ (version "0.13.0")
(source
(origin
- (method url-fetch)
- (uri (string-append "https://elpa.gnu.org/packages/"
- "modus-vivendi-theme-" version ".el"))
+ (method git-fetch)
+ (uri (git-reference
+ (url "https://gitlab.com/protesilaos/modus-themes")
+ (commit version)))
+ (file-name (git-file-name name version))
(sha256
- (base32 "01f6z5xjnmki1k9m83jwva42lxidb31pdpwm4wpxjzxqmb96picn"))))
+ (base32
+ "0i6n3pzrmlgabsijy6z0hhs71q7g99yagwr2m33lr30skqc9rvdn"))))
(build-system emacs-build-system)
+ (arguments
+ `(#:phases
+ (modify-phases %standard-phases
+ (add-after 'unpack 'remove-other-theme
+ (lambda _
+ (delete-file "modus-operandi-theme.el")
+ #t)))))
(home-page "https://gitlab.com/protesilaos/modus-themes")
(synopsis "Accessible dark theme (WCAG AAA)")
(description
--
2.28.0
Nicolas Goaziou wrote 4 years ago
(address . Morgan.J.Smith@outlook.com)(address . 43910@debbugs.gnu.org)
87pn5o7mrt.fsf@nicolasgoaziou.fr
Hello,

Morgan.J.Smith@outlook.com writes:

Toggle quote (4 lines)
> * gnu/packages/emacs-xyz.scm (emacs-modus-vivendi-theme): Update to 0.13.0.
> [source]: Use git instead of elpa, as elpa is out of date
> [arguments]: Add phase to remove modus-operandi-theme.el

Thank you.

However I suggest we stick to GNU ELPA repository, since it makes the
package much easier to update. Also, GNU ELPA will eventually catch up
with the new version.

WDYT?

Regards,
--
Nicolas Goaziou
Morgan Smith wrote 4 years ago
(name . Nicolas Goaziou)(address . mail@nicolasgoaziou.fr)(address . 43910@debbugs.gnu.org)
DM5PR1001MB2105E48EE0CCAACFA05C7902C5060@DM5PR1001MB2105.namprd10.prod.outlook.com
I was thinking the author likely stopped updating elpa once his package
made it into mainline emacs (but not in time for 27.1). I guess I'll
reach out and let him know he should keep updating it for a bit.

Also I remembered someone telling me to not use elpa. Upon checking the
history, you warned me about melpa, not elpa.

Cheers!
André Alexandre Gomes wrote 4 years ago
(no subject)
(address . 43910@debbugs.gnu.org)
87o8kjtw97.fsf@gmail.com
Hi,

You may find the following discussion useful:


So it seems that Guix will wait for GNU ELPA to release the latest
version, correct?

--
André Alexandre Gomes
"Free Thought, Free World"
Protesilaos Stavrou wrote 4 years ago
Re: [PATCH] gnu: emacs-modus-vivendi-theme: Update to 0.13.0.
(address . 43910@debbugs.gnu.org)
877dr7m19t.fsf@protesilaos.com
Hello everyone,

Just to note that I have not abandoned GNU ELPA: it remains an important
target.

The delay for this version is due to administrative changes that need to
happen. The themes are now part of GNU Emacs, so GNU ELPA must be
patched to get them from emacs.git (together with their new Info
manual).

I do not have push access to GNU ELPA and must thus coordinate with the
maintainer. Once this is done, future updates will not go through any
delays.

Thank you for your understanding!

Best regards,
Protesilaos

--
Protesilaos Stavrou
protesilaos.com
Ludovic Courtès wrote 4 years ago
Re: [bug#43910] [PATCH] gnu: emacs-modus-vivendi-theme: Update to 0.13.0.
(name . Morgan Smith)(address . Morgan.J.Smith@outlook.com)
87wnz3lvg2.fsf@gnu.org
Hi,

Nicolas, Morgan: what’s the status of this patch?

TIA! :-)

Ludo’.

Morgan Smith <Morgan.J.Smith@outlook.com> skribis:

Toggle quote (8 lines)
> I was thinking the author likely stopped updating elpa once his package
> made it into mainline emacs (but not in time for 27.1). I guess I'll
> reach out and let him know he should keep updating it for a bit.
>
> Also I remembered someone telling me to not use elpa. Upon checking the
> history, you warned me about melpa, not elpa.
>
> Cheers!
Morgan Smith wrote 4 years ago
(name . Ludovic Courtès)(address . ludo@gnu.org)
DM5PR1001MB21052C65378C0A8A80F4A3FCC5100@DM5PR1001MB2105.namprd10.prod.outlook.com
Hi Ludo,

So I really want it to be applied as is so I can use package
transformations and build this package from master (Currently if you try
to use package transformations you will obtain an extra unwanted file).
However, for some reason (I still don't know why) we like to use elpa.

If we do really want to keep using elpa (someone please explain to me
the benefits of doing this) than the good news is the author is working
on it and we just have to be patient. The moment this link says 0.13
instead of 0.12, we can do a version bump:

Morgan

On 11/2/20 8:50 AM, Ludovic Courtès wrote:
Toggle quote (18 lines)
> Hi,
>
> Nicolas, Morgan: what’s the status of this patch?
>
> TIA! :-)
>
> Ludo’.
>
> Morgan Smith <Morgan.J.Smith@outlook.com> skribis:
>
>> I was thinking the author likely stopped updating elpa once his package
>> made it into mainline emacs (but not in time for 27.1). I guess I'll
>> reach out and let him know he should keep updating it for a bit.
>>
>> Also I remembered someone telling me to not use elpa. Upon checking the
>> history, you warned me about melpa, not elpa.
>>
>> Cheers!
Nicolas Goaziou wrote 4 years ago
(name . Morgan Smith)(address . Morgan.J.Smith@outlook.com)
87h7q7mwvw.fsf@nicolasgoaziou.fr
Hello,

Morgan Smith <Morgan.J.Smith@outlook.com> writes:

Toggle quote (6 lines)
> If we do really want to keep using elpa (someone please explain to me
> the benefits of doing this) than the good news is the author is working
> on it and we just have to be patient. The moment this link says 0.13
> instead of 0.12, we can do a version bump:
> https://elpa.gnu.org/packages/modus-operandi-theme.html

GNU ELPA is preferred because updating the package is much easier. This
is important when you have lot of packages you want to keep up-to-date.
It is also easier to get notified about updates, but that's not specific
to Guix.

Usually, GNU ELPA catches up releases quickly, but it seems more
complicated in this case. So, I think it is reasonable to update this
package using GitHub source and switch back to ELPA when the situation
is sorted out.

Regards,

--
Nicolas Goaziou
Nicolas Goaziou wrote 4 years ago
(address . Morgan.J.Smith@outlook.com)(address . 43910-done@debbugs.gnu.org)
87v9emln5w.fsf@nicolasgoaziou.fr
Hello,

Morgan.J.Smith@outlook.com writes:

Toggle quote (4 lines)
> * gnu/packages/emacs-xyz.scm (emacs-modus-vivendi-theme): Update to 0.13.0.
> [source]: Use git instead of elpa, as elpa is out of date
> [arguments]: Add phase to remove modus-operandi-theme.el

Applied. Thank you.

Regards,
--
Nicolas Goaziou
Closed
?
Your comment

This issue is archived.

To comment on this conversation send an email to 43910@debbugs.gnu.org

To respond to this issue using the mumi CLI, first switch to it
mumi current 43910
Then, you may apply the latest patchset in this issue (with sign off)
mumi am -- -s
Or, compose a reply to this issue
mumi compose
Or, send patches to this issue
mumi send-email *.patch
You may also tag this issue. See list of standard tags. For example, to set the confirmed and easy tags
mumi command -t +confirmed -t +easy
Or, remove the moreinfo tag and set the help tag
mumi command -t -moreinfo -t +help