Bug 117198

Summary: [GTK] Forwarding headers not generated for incremental build
Product: WebKit Reporter: Wojciech Bielawski <w.bielawski>
Component: Tools / TestsAssignee: Wojciech Bielawski <w.bielawski>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, gustavo, mrobinson, zan
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 98410    
Attachments:
Description Flags
Generate forwarding headers if needed even for incremental build
none
Now with changelog
cgarcia: review-
changelog generated with script (and --bug argument) none

Description Wojciech Bielawski 2013-06-04 07:43:42 PDT
When adding some includes in source files in Tools/ (like WebKitTestRunner/EventSendingController.cpp) a forwarding headers will not be generated for incremental build.
Comment 1 Wojciech Bielawski 2013-06-04 07:52:16 PDT
Created attachment 203696 [details]
Generate forwarding headers if needed even for incremental build
Comment 2 Wojciech Bielawski 2013-06-04 07:53:12 PDT
Thanks for Zan Dobersek.
Comment 3 Wojciech Bielawski 2013-06-04 08:08:29 PDT
Created attachment 203699 [details]
Now with changelog
Comment 4 Carlos Garcia Campos 2013-06-05 02:17:41 PDT
Comment on attachment 203699 [details]
Now with changelog

View in context: https://bugs.webkit.org/attachment.cgi?id=203699&action=review

Patch looks good to me, could you please update the changelog entry to match the bug? We have scripts that parse the changelog entries, so it's important to make sure they are generated correctly. Use the prepare-ChangeLog script to make sure.

> Tools/ChangeLog:3
> +        [GTK] Generate forwarding headers if needed even for incremental build.

This doesn't match the bug title, you should use the Tools/Scripts/prepare-ChangeLog to generate the changelog entry.

> Tools/ChangeLog:4
> +        https://bugs.webkit.org/show_bug.cgi?id=117198.

And this doesn't finish with a period
Comment 5 Wojciech Bielawski 2013-06-05 03:29:21 PDT
Created attachment 203786 [details]
changelog generated with script (and --bug argument)

Now prepare-ChangeLog had generated Title without period on the end. Is it correct?
Comment 6 WebKit Commit Bot 2013-06-05 05:26:57 PDT
Comment on attachment 203786 [details]
changelog generated with script (and --bug argument)

Clearing flags on attachment: 203786

Committed r151215: <http://trac.webkit.org/changeset/151215>
Comment 7 WebKit Commit Bot 2013-06-05 05:26:59 PDT
All reviewed patches have been landed.  Closing bug.