public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Peter Tyser <ptyser@xes-inc.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 2/2] remove main CHANGELOG file
Date: Wed, 05 May 2010 16:43:28 -0500	[thread overview]
Message-ID: <1273095808.2451.4674.camel@localhost.localdomain> (raw)
In-Reply-To: <20100505205802.97211B0FF81@gemini.denx.de>

> > I'll quit whining, just wanted to give my +1 for removing the changelog.
> 
> I don't consider you whining. I am listening to the arguments.
> I am not convinced yet, though.

Well in that case, I'll chime in again:)

> > I still don't grasp what the common use of looking at U-Boot's entire
> > changelog is.  What are some common tasks that require grepping U-Boot's
> > entire changelog?   Are these tasks performed frequently enough that the
> > extra <1 second 'git log' requires is bothersome?
> 
> I'm trying to maintain some awareness of open patches. For example,
> when I receive a pull request I mark all included patches as
> processed; sometimes this includes tracking down older versions of
> the patches, or commit messages that have been changed on the way.
> And yes, the <1 second delay is bothersome when doing this frequently
> enough.

I see, makes sense.  This seems like a problem that is unique to you, as
well as potentially a few other maintainers.  I imagine the vast
majority of developers rarely use the CHANGELOG file though.  Would
having maintainers create their own CHANGELOG files and/or use something
like patchwork be acceptable?

> > > 	-> wget -O u-boot.tgz 'http://git.denx.de/?p=u-boot.git;a=snapshot;sf=tgz'
> > 
> > For snapshots, the CHANGELOG file is going to be out of date though.
> > It'll only list the changes that occurred up to the previous release.
> > This seems worse than not having a CHANGELOG at all as its actively
> > misleading people as to what changes their source code has.
> 
> I disagree here. I find myself quite frequently in the situation that
> I have to identify the exact version some source tree has been
> derived from. Companies who maintain out-of-tree ports usually don't
> include any SCM information either. To me, the CHANGELOG is an
> extremely useful resource in such cases.

The CHANGELOG in a company's source tree won't provide any more
information than the VERSION, PATCHLEVEL, and EXTRAVERSION in the
Makefile though, right?  Ie the CHANGELOG is only updated when the
VERSION, PATCHLEVEL, and EXTRAVERSION are updated.  Either way, it seems
like an inexact method to determine the revision of the source code, and
there should be a better solution so that you don't have to be a
detective every time you get another vendor's source tree.

It seems bad to have an inaccurate CHANGELOG.  For example if someone
got a snapshot tarball a week before the last release, their CHANGELOG
would be missing over 500 changes.  In this case the CHANGELOG clearly
isn't serving its intended purpose and could actually mislead someone,
eg "I know commit XYZ introduces a bug, but the changelog says I don't
have commit XYZ in my tree, so I'm bug free!".  In reality there have
been 500 commits the user is unaware of, including potentially commit
XYZ they were trying to avoid.

OK, its a stretch, but I don't get the purpose of an inaccurate
CHANGELOG in any case.  Other than when the 1 commit that correlates to
a tagged release is the the top-of-tree, the CHANGELOG is *never* up to
date.  It can only be believed if someone downloads an officially
released tarball - otherwise it is most likely wrong.

Best,
Peter

  reply	other threads:[~2010-05-05 21:43 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-04-14  2:16 [U-Boot] [PATCH 2/2] remove main CHANGELOG file Kim Phillips
2010-05-04 21:41 ` Wolfgang Denk
2010-05-05  0:25   ` Kim Phillips
2010-05-05  6:54     ` Wolfgang Denk
2010-05-05 13:51       ` Marek Vasut
2010-05-05 14:17         ` Wolfgang Denk
2010-05-05 14:56           ` Timur Tabi
2010-05-05 15:07             ` Wolfgang Denk
2010-05-05 16:03               ` Peter Tyser
2010-05-05 19:05                 ` Wolfgang Denk
2010-05-05 19:45                   ` Scott Wood
2010-05-05 20:36                     ` Wolfgang Denk
2010-05-05 20:37                   ` Peter Tyser
2010-05-05 20:58                     ` Wolfgang Denk
2010-05-05 21:43                       ` Peter Tyser [this message]
2010-05-05 21:51                         ` Wolfgang Denk
2010-07-16 18:44                           ` Kim Phillips
2010-05-05 21:06       ` Kim Phillips
2010-05-05 21:07         ` Wolfgang Denk
2010-05-05 21:09         ` Kim Phillips

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=1273095808.2451.4674.camel@localhost.localdomain \
    --to=ptyser@xes-inc.com \
    --cc=u-boot@lists.denx.de \
    /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