Bug 69350

Summary: [WebKit2][gtk] Report gtk-doc warnings for every WebKit2-GTK build.
Product: WebKit Reporter: Nayan Kumar K <nayankk>
Component: WebKit wxAssignee: Nobody <webkit-unassigned>
Status: RESOLVED INVALID    
Severity: Normal CC: mrobinson
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Doc generation logic added none

Nayan Kumar K
Reported 2011-10-04 10:36:42 PDT
As per the discussion in https://bugs.webkit.org/show_bug.cgi?id=69325, it would be nice to have build-webkit script show warnings (if any) while generating documentation using gtk-doc. I would help to achieve 0 warning goal in generating documentation.
Attachments
Doc generation logic added (1.56 KB, patch)
2011-10-07 09:33 PDT, Nayan Kumar K 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Nayan Kumar K
Comment 1 2011-10-04 10:56:02 PDT
Immediate question which comes onto my mind, a). This would mean that 'enable-gtk-doc' will be enabled by default, and it might increase the build time. b). Should this be supported for both WebKit1-GTK and WebKit2-GTK? WebKit1 seems to be having lots of warnings already, I guess for now we should have it only for WebKit2-GTK?
Martin Robinson
Comment 2 2011-10-04 10:58:48 PDT
(In reply to comment #1) > Immediate question which comes onto my mind, > a). This would mean that 'enable-gtk-doc' will be enabled by default, and it might increase the build time. > b). Should this be supported for both WebKit1-GTK and WebKit2-GTK? WebKit1 seems to be having lots of warnings already, I guess for now we should have it only for WebKit2-GTK? a. Yes. I think so, but perhaps we should figure out a way to not actually generate the documentation, simply report any warnings. We should take a look at what effect this has on build speed. b. Just WebKit2 for now. The WebKit1 documentaiton is full of warnings.
Nayan Kumar K
Comment 3 2011-10-07 09:33:57 PDT
Created attachment 110160 [details] Doc generation logic added
Note You need to log in before you can comment on or make changes to this bug.