unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Jim Porter <jporterbugs@gmail.com>
Cc: luangruo@yahoo.com, stefankangas@gmail.com, emacs-devel@gnu.org
Subject: Re: master 9ccaa09a635: ; .dir-locals.el (log-edit-mode) <fill-column>: Set to 64.
Date: Thu, 08 Feb 2024 09:23:41 +0200	[thread overview]
Message-ID: <8634u3xvuq.fsf@gnu.org> (raw)
In-Reply-To: <62ce63d3-76ce-b511-b2bf-27eb526bb711@gmail.com> (message from Jim Porter on Wed, 7 Feb 2024 22:13:01 -0800)

> Date: Wed, 7 Feb 2024 22:13:01 -0800
> Cc: Stefan Kangas <stefankangas@gmail.com>, emacs-devel@gnu.org, eliz@gnu.org
> From: Jim Porter <jporterbugs@gmail.com>
> 
> On 2/7/2024 7:34 PM, Po Lu wrote:
> > Jim Porter <jporterbugs@gmail.com> writes:
> > 
> >> I don't have a terribly strong opinion here of what number to use, but
> >> we should at least resolve the inconsistencies in CONTRIBUTE,
> >> .dir-locals.el, and build-aux/git-hooks/commit-msg.
> > 
> > commit-msg is fine as-is, but CONTRIBUTE should be fixed.

I'm not sure I understand what needs to be fixed in CONTRIBUTE.  If
that's fallout of the recent change of fill-column, then I'd expect
the author of that change to adjust CONTRIBUTE as needed.

> I think I understand now. I find the CONTRIBUTE explanation rather 
> confusing to be honest, since it introduces ChangeLog files, mentions 
> ChangeLog entries in that context (which I interpret as "one commit 
> message -> one ChangeLog entry"), and then later talks about ChangeLog 
> entries again, but in the context of the commit message (where I 
> interpret it to mean "one ChangeLog entry = one file/function changed"). 
> Only after reading this a few times in detail did I notice this distinction.

Please try to tell in more detail what confused you.  I don't want to
make any significant change in CONTRIBUTE, since the current text is
the result of many people reading it and commenting on its text.  So
I'd prefer to make the minimal required changes, not rewrite whole
passages anew.

AFAICS, the current text clearly explains what it means by "ChangeLog
entries" that are part of the commit log messages.  All of this text
is under the "Commit messages" heading, so it is all about commit log
messages, not about ChangeLog files (which we no longer maintain by
hand as part of documenting changes we install).  So I don't quite see
why you got confused by the text there; please elaborate.

> No worries if this doesn't get merged, or if people change my wording. I 
> just thought it would be helpful to show an example of what I'd find 
> clearer.

I'd prefer if you explained what is not clear in the current text.

Thanks.



  reply	other threads:[~2024-02-08  7:23 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <170727415485.32408.11264518274307262467@vcs2.savannah.gnu.org>
     [not found] ` <20240207024915.38686C0EFEC@vcs2.savannah.gnu.org>
2024-02-07 20:01   ` master 9ccaa09a635: ; .dir-locals.el (log-edit-mode) <fill-column>: Set to 64 Stefan Kangas
2024-02-08  1:36     ` Po Lu
2024-02-08  3:10       ` Jim Porter
2024-02-08  3:34         ` Po Lu
2024-02-08  6:13           ` Jim Porter
2024-02-08  7:23             ` Eli Zaretskii [this message]
2024-02-08 17:49               ` Jim Porter
2024-02-08 19:30                 ` Eli Zaretskii
2024-02-08 20:21                   ` Jim Porter
2024-02-08 20:30                     ` Eli Zaretskii
2024-02-18  2:00           ` Stefan Kangas
2024-02-18  3:16             ` Po Lu
2024-02-08  7:09         ` Eli Zaretskii

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=8634u3xvuq.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=jporterbugs@gmail.com \
    --cc=luangruo@yahoo.com \
    --cc=stefankangas@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).