[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Emacs-diffs] /srv/bzr/emacs/trunk r106712: * progmodes/cfengine.el:
From: |
Stefan Monnier |
Subject: |
Re: [Emacs-diffs] /srv/bzr/emacs/trunk r106712: * progmodes/cfengine.el: Add CFEngine 3.x syntax highlighting and support. |
Date: |
Thu, 22 Dec 2011 19:27:06 -0500 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.0.92 (gnu/linux) |
Ted, please be careful with your commit messages:
1- the bzr commit message should be the same as the one is the ChangeLog
(so the ChangeLog will ultimately be produced from the Bzr commit
messages).
2- the style of your ChangeLog entries need to follow the Gnu
coding standard. See the megabytes of examples in the Emacs source
tree, and http://www.gnu.org/prep/standards/html_node/Change-Logs.html.
Details follow.
Stefan
> * progmodes/cfengine.el: Add CFEngine 3.x syntax highlighting and support.
Not a fair enough summary since it doesn't talk about the change of
cfengine-mode (and because there already was cfengine-3 font-lock
support, so this support is definitely not added by the commit, it's
just slightly improved). And you need to add a copy of the ChangeLog
text (use C-c C-a from the *VC-Log* buffer to do that for you).
> + * progmodes/cfengine.el: Add Version.
> +
> + Add CFEngine 3.x syntax highlighting and support with
> + `cfengine3-mode', and rename the old `cfengine-mode' to
> + `cfengine2-mode'. Make `cfengine-mode' an automatic content-based
> + switcher between `cfengine3-mode' and `cfengine2-mode' by aliasing
> + it to `cfengine-auto-mode'. Rename variables specific to
> + `cfengine2-mode' from cfengine-* to cfengine2-*. Change the
> + modeline indicator to "CFE2" and "CFE3" depending on the mode, to
> + save space. Mark `cfengine-mode-abbrevs' as obsolete.
Wrong format. For example, the last sentence should be replaced by:
(cfengine-mode-abbrevs): Mark obsolete.
Also the empty line between "* progmodes/cfengine.el: Add Version." and
the rest makes it seem like the two are separate and hence makes it
unclear which file the second part refers to.
Stefan