public inbox for gdb@sourceware.org
 help / color / mirror / Atom feed
From: Simon Marchi <simon.marchi@polymtl.ca>
To: Eli Zaretskii <eliz@gnu.org>, Luis Machado <luis.machado@linaro.org>
Cc: gdb@sourceware.org
Subject: Re: sim: replacing ChangeLog files with online git logs
Date: Wed, 17 Mar 2021 11:52:49 -0400	[thread overview]
Message-ID: <ff3552e2-684d-813e-1902-14fba35484cd@polymtl.ca> (raw)
In-Reply-To: <83mtv1dbzr.fsf@gnu.org>

We're getting into this again, I know you've said the horse has been
beaten to death about ChangeLogs, but it keeps coming up because it's a
real daily pain for many people.

On 2021-03-17 11:40 a.m., Eli Zaretskii via Gdb wrote:
> The list of files touched by a changeset can be easily generated, but
> the functions/macros/classes not so easily, and not necessarily
> accurately enough, even if we will use the more elaborate script that
> is used by glibc.  So some information loss will be present.
> 
> OTOH, the ChangeLog-format Git log messages don't require one to have
> the ChangeLog file, they just require to use the same format when
> describing the changeset.

It's not the ChangeLog file per-se that's annoying, it's writing the
entry.  It's tedious and contains mostly uninteresting information.  So
putting the ChangeLog entry in the git log instead of a file doesn't
change anything.

In the GDB project, we write what I believe are excellent git commit
messages that go in depth describing the observed problem and the
solution provided by the patch.  That provides valuable information for
anyone wanting to understand why the code is the way it is.  A list of
modified symbols doesn't help.  So I am completely fine if the "list of
modified symbols" part of the information is lost.

> If you use Emacs to make the changes
> themselves and commit the changes into Git, you just need to use the
> Emacs commands to format the Git log messages, those commands will
> produce the same information as the manual ChangeLog files
> automatically.

I don't really understand how that helps, you still have to spell out
the symbols.  But please don't assume everyone uses Emacs.  Making a
decision based on "Emacs makes it easy" and ignoring the rest of users
wouldn't be fair.

> If that is not an option that this community wants to adopt, then
> gitlog-to-changelog is probably not the adequate solution, and we
> should try using vcs-to-changelog.py (also in Gnulib).

I tried vcs-to-changelog, it gave horrible/useless results with our
codebase.  This is not an option.

Simon

  reply	other threads:[~2021-03-17 15:53 UTC|newest]

Thread overview: 45+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-03-13  2:53 Mike Frysinger
2021-03-13  7:30 ` Eli Zaretskii
2021-03-13 15:12   ` Mike Frysinger
2021-03-13 16:02     ` Eli Zaretskii
2021-03-13 16:26       ` Mike Frysinger
2021-03-13 16:38         ` Eli Zaretskii
2021-03-13 18:21           ` Mike Frysinger
2021-03-13 18:32             ` Eli Zaretskii
2021-03-17  6:07               ` Mike Frysinger
2021-03-17 13:14                 ` Eli Zaretskii
2021-03-17 14:31                   ` Luis Machado
2021-03-17 14:47                     ` Eli Zaretskii
2021-03-17 14:52                       ` Luis Machado
2021-03-17 15:09                         ` Eli Zaretskii
2021-03-17 15:22                           ` Luis Machado
2021-03-17 15:40                             ` Eli Zaretskii
2021-03-17 15:52                               ` Simon Marchi [this message]
2021-03-17 16:09                                 ` Eli Zaretskii
2021-03-17 17:24                                   ` Luis Machado
2021-03-17 17:53                                     ` Eli Zaretskii
2021-03-17 20:53                                       ` Luis Machado
2021-03-18  9:31                                       ` Andrew Burgess
2021-03-18  9:48                                         ` Eli Zaretskii
2021-03-22  2:19                                           ` Maciej W. Rozycki
2021-03-22  3:38                                             ` Eli Zaretskii
2021-03-23  4:57                                               ` Mike Frysinger
2021-03-23  5:05                   ` Mike Frysinger
2021-03-23 11:06                     ` Andrew Burgess
2021-03-24  3:45                       ` Mike Frysinger
2021-03-24 16:46                         ` Eli Zaretskii
2021-03-24 16:59                           ` Andrew Burgess
2021-03-24 17:39                             ` Eli Zaretskii
2021-03-24 18:44                               ` Mike Frysinger
2021-03-31  2:27                                 ` Mike Frysinger
2021-03-31  3:27                                   ` Joel Brobecker
2021-03-31 11:35                                     ` Luis Machado
2021-03-31 15:29                                       ` Joel Brobecker
2021-04-01  3:41                                     ` Mike Frysinger
2021-04-19  0:52                                       ` Mike Frysinger
2021-04-19  2:32                                         ` Joel Brobecker
2021-04-19  4:31                                           ` Mike Frysinger
2021-04-19  5:00                                             ` Joel Brobecker
2021-05-23  3:26                                               ` Mike Frysinger
2021-06-10 12:11                                                 ` Luis Machado
2021-03-13 18:34             ` Paul Smith

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

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

  git send-email \
    --in-reply-to=ff3552e2-684d-813e-1902-14fba35484cd@polymtl.ca \
    --to=simon.marchi@polymtl.ca \
    --cc=eliz@gnu.org \
    --cc=gdb@sourceware.org \
    --cc=luis.machado@linaro.org \
    /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.
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).