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

[Eli Zaretskii]

> From: Protesilaos Stavrou <info@protesilaos.com>
> Cc: Mauro Aranda <maurooaranda@gmail.com>,  Eli Zaretskii <eliz@gnu.org>,
>   45068@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?