Re: On autogenerated ChangeLog
- From: Stefan Kost <ensonic hora-obscura de>
- To: Tristan Van Berkom <tvb gnome org>
- Cc: Cody Russell <bratsche gnome org>, gnome-infrastructure <gnome-infrastructure gnome org>, desktop-devel-list <desktop-devel-list gnome org>
- Subject: Re: On autogenerated ChangeLog
- Date: Thu, 23 Apr 2009 09:57:03 +0300
Tristan Van Berkom schrieb:
> On Mon, Apr 20, 2009 at 10:20 PM, Cody Russell <bratsche gnome org> wrote:
> [....]
>
>> Yeah, but the thing that sucks about versioned ChangeLogs is
>> merging/rebasing your code. Typically you always leave writing a
>> ChangeLog last for this reason, but it just makes so much more sense to
>> be able to write your entry when you do your commit.
>>
>> If you're posting a branch for review or something, people can read your
>> commit logs as well as the code.. but if you post patches for review,
>> you probably don't post the ChangeLog with it because it'll get
>> clobbered when you have to merge it into the tree.
>>
>>
>
> You always post ChangeLogs diffs with large patches, large patches
> generally come to the maintainer in the form of a patch, with a single
> changelog entry, the maintainer reviewing a branch doesnt want to
> see the revision history of what happened on the branch, or why
> you reverted that peice of code thats not actually in the patch
> (and never made it into the baseline/trunk).
>
A git patch has metadata. That is if you use git format-patch then the
commit messages go into the patch and if you use git apply they will be
applied along with the patch.
Stefan
> Now if I can demand that a patch submitter provide the base minimum:
> - A patch that applies to trunk
> - A rich ChangeLog entry that describes what happens in the change
>
> Then why would I waste my time flipping through individual commit
> logs ?
>
> Cheers,
> -Tristan
> _______________________________________________
> desktop-devel-list mailing list
> desktop-devel-list gnome org
> http://mail.gnome.org/mailman/listinfo/desktop-devel-list
>
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]