If we enable introspection always we can sniff out issues sooner and turn the EWS bubble red when API is introduced without proper annotations.
Created attachment 199107 [details] Patch
Comment on attachment 199107 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=199107&action=review > Source/WebKit2/ChangeLog:8 > + * GNUmakefile.am: Make warnings cause the build to fail. Eliminate Be good to make it obvious you're talking about the scanner build here. > Source/WebKit/gtk/GNUmakefile.am:300 > - $(AM_V_GEN)$(G_IR_SCANNER) -v --warn-all \ > + $(AM_V_GEN) \ > + CFLAGS="$(CFLAGS) -Wno-deprecated-declarations" \ > + $(G_IR_SCANNER) \ > + --quiet \ Is girscanner still verbose enough to help debug issues shown by the EWS after replacing -v with --quiet here? Other than that it's a great idea, and the patch looks OK to me.
Committed r148916: <http://trac.webkit.org/changeset/148916>
(In reply to comment #2) > (From update of attachment 199107 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=199107&action=review > > > Source/WebKit2/ChangeLog:8 > > + * GNUmakefile.am: Make warnings cause the build to fail. Eliminate > > Be good to make it obvious you're talking about the scanner build here. Sound good. > > Source/WebKit/gtk/GNUmakefile.am:300 > > - $(AM_V_GEN)$(G_IR_SCANNER) -v --warn-all \ > > + $(AM_V_GEN) \ > > + CFLAGS="$(CFLAGS) -Wno-deprecated-declarations" \ > > + $(G_IR_SCANNER) \ > > + --quiet \ > > Is girscanner still verbose enough to help debug issues shown by the EWS after replacing -v with --quiet here? Other than that it's a great idea, and the patch looks OK to me. When not in quiet mode, g-ir-scanner prints the command-line it uses to compile the scanning program, which, for us, is many lines long. On the other hand it does print out the details of any problems it finds when --quiet is active.