[Webkit-unassigned] [Bug 74358] prepare-Changelog should support updating the list of changed files

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=74358





--- Comment #3 from Ojan Vafai <ojan at chromium.org>  2011-12-12 16:05:06 PST ---
(In reply to comment #2)
> (In reply to comment #0)
> > 1. Update the one-line description if the bug description has changed.
> 
> This is new territory for prepare-ChangeLog, since currently it does not do anything with the description.

Yes. I'm also OK with it being a different script or a special command-line flag, or even just being part of "webkit-patch upload".

> > 2. Update the list of changed files.
> 
> I presume you mean both he changed files and functions.

Yes.

> If it did this, it would need to highlight the newly added items, for people like me who add comments to each function in the list. Otherwise, it would be easy to miss something.

Oh, that's a great idea. Not sure how to do that though. It would be important to keep old comments I think. Some ideas:
1. Add a * at the beginning of the line.
2. Add an [OOPS new changelog line] to the line
3. Instead of interleaving, when the list changes, just add both sets in their entirety (one above the other) and the let the developer resolve it.

1 and 2 seem awful. 3 actually doesn't seem that bad to me.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.