public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
From: Yao Qi <qiyaoltc@gmail.com>
To: "Maciej W. Rozycki" <macro@mips.com>
Cc: GDB <gdb@sourceware.org>,  GDB Patches <gdb-patches@sourceware.org>
Subject: Re: Stop updating ChangeLog?
Date: Wed, 07 Feb 2018 09:11:00 -0000	[thread overview]
Message-ID: <867erpjhau.fsf@gmail.com> (raw)
In-Reply-To: <alpine.DEB.2.00.1802062040590.3553@tp.orcam.me.uk> (Maciej	W. Rozycki's message of "Tue, 6 Feb 2018 20:58:34 +0000")

"Maciej W. Rozycki" <macro@mips.com> writes:

> 1. It is often cheaper in terms of computation or I/O to run `grep' on
>    local ChangeLog files than on output from `git', which in turn means
>    you get results sooner.  I know it's a weak argument, but still.
>

ChangeLog is a summary of 'git log', it is fast to do search in it, but
the benefit-cost ratio is quite low, IMO.  We have 1979 ChangeLog
entries in 2017, and we must spend some efforts to create these
entries.  Some of them have more than 40 lines, and all of them are not
useful, AFAICS. Most of these long entries just describe some mechanical
changes by "Likewise".

> 2. More importantly, many times I found writing a ChangeLog entry the last 
>    reflection point when I realised the solution wasn't actually the right 
>    one.  This is because at that point you need to give individual changes
>    another thought, and then actually look at them from a different angle, 
>    to have them properly expressed in ChangeLog-speak.  I expect that at 
>    least some people have had a similar experience and I'd like to keep 
>    that opportunity.

Yes, I had the similar experience too.  What we need here is to jump out
of the code/patch, and think about the big picture again.  I did that
by means of writing commit log, and it happens several times that I find
some flaws when I write commit log.  We can still describe the design
and change of patch in commit log, which is more valuable and less
mechanical than ChangeLog.

-- 
Yao (齐尧)

      reply	other threads:[~2018-02-07  9:11 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-02-05 16:00 Yao Qi
2018-02-05 16:30 ` Simon Marchi
2018-02-05 17:23 ` Dmitry Samersoff
2018-02-05 19:04 ` Joseph Myers
2018-02-05 21:56   ` Yao Qi
2018-02-05 22:32     ` Joseph Myers
2018-02-06  7:34       ` Joel Brobecker
2018-02-05 19:08 ` John Baldwin
2018-02-05 19:44   ` Andrew Burgess
2018-02-06 21:02 ` Maciej W. Rozycki
2018-02-07  9:11   ` Yao Qi [this message]

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=867erpjhau.fsf@gmail.com \
    --to=qiyaoltc@gmail.com \
    --cc=gdb-patches@sourceware.org \
    --cc=gdb@sourceware.org \
    --cc=macro@mips.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.
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).