GNU bug report logs - #45068
[PATCH] 28.0.50; Update Modus themes 1.0.2 (backward-incompatible)

Previous Next

Full log

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

From: Eli Zaretskii <eliz <at> gnu.org>
To: Protesilaos Stavrou <info <at> protesilaos.com>
Cc: contovob <at> tcd.ie, 45068 <at> debbugs.gnu.org, maurooaranda <at> gmail.com
Subject: Re: bug#45068: [PATCH] 28.0.50; Update Modus themes 1.0.2
 (backward-incompatible)
Date: Tue, 02 Mar 2021 07:51:31 +0200

> From: Protesilaos Stavrou <info <at> protesilaos.com>
> Cc: Mauro Aranda <maurooaranda <at> gmail.com>,  Eli Zaretskii <eliz <at> gnu.org>,
>   45068 <at> debbugs.gnu.org
> Date: Tue, 02 Mar 2021 07:47:58 +0200
> 
> >   (defun require-theme (feature &optional noerror)
> >     (let ((load-path (custom-theme--load-path)))
> >       (require feature nil noerror)))
> 
> No the themes do not need that dual behaviour.  And yes, your version
> works and I am happy with it: all I need is a 'require' that reads from
> etc/themes.

Won't the above version cause problems if the loaded theme needs to
require some Emacs package?  Binding load-path would get in the way of
that, no?

Previous Next

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