Summary: | [bzt] Automate the process of calling prepare-ChangeLog | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Adam Barth <abarth> | ||||||
Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | eric, mjs, webkit.review.bot | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Attachments: |
|
Description
Adam Barth
2009-12-21 00:36:11 PST
Created attachment 45309 [details]
Patch
Attachment 45309 [details] did not pass style-queue:
Failed to run "WebKitTools/Scripts/check-webkit-style" exit_code: 1
WebKitTools/ChangeLog:17: Line contains tab character. [whitespace/tab] [5]
Total errors found: 1
Created attachment 45310 [details]
Patch
style-queue ran check-webkit-style on attachment 45310 [details] without any errors.
Comment on attachment 45310 [details]
Patch
I think the -o approach is strange, and probably wrong. $EDITOR or some sort of user prompting is what we want in the end.
Also --non-interactive makes little sense for PrepareChangeLog, but ok.
The addition of SubmitPatch at this time doesn't seem much value add. But the rest of the changes seem great! (I'm OK keeping it around, just seems kinda vacuous atm.)
Clearly PostCommits needs to be converted too. :(
LGTM.
Comment on attachment 45310 [details]
Patch
tahnks
Committed r52480: <http://trac.webkit.org/changeset/52480> |