Use defface, not make-face, for creating font lock faces

[Osku-salerma]

The normal way of customizing font lock faces does not
work with python-mode since it does not use the defface
function. The attached patch against python-mode 4.7
fixes this, and one can now use customize-faces to
customize python-mode's faces.

This is tested on emacs 21.4; I have no idea if xemacs
support this or not.

[http://sourceforge.net/tracker/index.php?func=detail&aid=1466941&group_id=86916&atid=581351]

[Montanaro (montanaro)]

XEmacs has defface. Can you compare this documentation with what's in
GNU Emacs? I don't have a recent GNU Emacs handy. /usr/bin/emacs on my
Mac says it's version 21.2.1. Apropos in that environment does not turn
up
a defface macro. Is it a recent addition?

`defface' (buffer: *Hyper Apropos*, mode: Hyper-Apropos)

Compiled Lisp macro,
(loaded from "/Users/skip/src/xemacs-21.5/lisp/custom.elc"):

  arguments: (face spec doc &rest args)

  Declare FACE as a customizable face that defaults to SPEC.
  FACE does not need to be quoted.

  Third argument DOC is the face documentation.

  If FACE has been set with `custom-set-face', set the face attributes
  as specified by that function, otherwise set the face attributes
  according to SPEC.

  The remaining arguments should have the form

     [KEYWORD VALUE]...

  The following KEYWORDs are defined:

  :group VALUE should be a customization group.
   Add FACE to that group.

  SPEC should be an alist of the form ((DISPLAY ATTS)...).

  ATTS is a list of face attributes and their values. The possible
  attributes are defined in the variable `custom-face-attributes'.

  The ATTS of the first entry in SPEC where the DISPLAY matches the
  frame should take effect in that frame. DISPLAY can either be the
  symbol t, which will match all frames, or an alist of the form
  ((REQ ITEM...)...)

  For the DISPLAY to match a FRAME, the REQ property of the frame must
  match one of the ITEM. The following REQ are defined:

  `type' (the value of `window-system')
    Should be one of `x', `mswindows', or `tty'.

  `class' (the frame's color support)
    Should be one of `color', `grayscale', or `mono'.

  `background' (what color is used for the background text)
    Should be one of `light' or `dark'.

  Read the section about customization in the Emacs Lisp manual for more
  information.

[Osku-salerma (osku-salerma)]

The documentation for defface in emacs 21.4 is basically
identical to the one you posted. It it not a recent
addition; according to
http://cvs.savannah.gnu.org/viewcvs/emacs/lisp/custom.el?rev=1.1&root=emacs&view=markup
it has been in existence at least since April 1997, the date
that the file custom.el was created.

[Osku-salerma (osku-salerma)]

So...are you going to apply this patch or is there something
else that needs to be done before that?

[Andreas Roehler (a-roehler)]

Hi,

looks like applying this patch went lost somwhere in cvs-svn troubles.

Would take another try...

Cheers

Andreas