GNU bug report logs - #77928
[PATCH] use-package :custom-face is meant to behave like custom-set-face

Previous Next

Package: emacs;

Reported by: Michael Shields <shields <at> msrl.com>

Date: Sat, 19 Apr 2025 20:42:02 UTC

Severity: normal

Tags: patch

Done: Eli Zaretskii <eliz <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: John Wiegley <johnw <at> gnu.org>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 77928 <at> debbugs.gnu.org, Michael Shields <shields <at> msrl.com>, steven <at> stebalien.com
Subject: bug#77928: 31.0.50; ebcde0f90f6 (bug#77928) breaks :custom-face for themed faces
Date: Mon, 26 May 2025 16:03:51 -0700
>>>>> Eli Zaretskii <eliz <at> gnu.org> writes:

>> From: Michael Shields <shields <at> msrl.com>
>> Date: Mon, 19 May 2025 22:51:29 -0700
>> Cc: steven <at> stebalien.com, 77928 <at> debbugs.gnu.org
>> 
>> I think I understand the issue, but not its solution. There are more or less three standard ways to change
>> faces now: Customize, themes, and use-package :custom-face. But there is no clear way to resolve the
>> ordering between them, or whether they replace or inherit at each level. The behavior I was surprised by,
>> and that has been reported previously, is that there was no way to specify a replacement for face settings.
>> But Steven is surprised by the fact that things now happen at a level below theme application, and the theme
>> replaces instead of inherits the defface spec.

> John, could you perhaps help us out here?  Bug#77928 was about
> :custom-face, which is a use-package feature.  I don't really
> understand how this feature is supposed to work, given that
> use-package produces macros whose effect happens when a package is
> loaded, and given that other methods of face definition are available.

All that use-package is doing here is expanding the ‘:custom-face‘ declaration
into this call:

  (apply #'face-spec-set '(font-lock-keyword-face ((t :italic t))))

Should it be calling something other than `face-spec-set' now?

-- 
John Wiegley                  GPG fingerprint = 4710 CF98 AF9B 327B B80F
http://newartisans.com                          60E1 46C4 BD1A 7AC1 4BA2




This bug report was last modified 42 days ago.

Previous Next


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