Summary: | [GTK] Streamline wk1 documentation build | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Gustavo Noronha (kov) <gustavo> | ||||||
Component: | New Bugs | Assignee: | Gustavo Noronha (kov) <gustavo> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | ||||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Gustavo Noronha (kov)
2011-10-19 15:52:10 PDT
Created attachment 111686 [details]
Patch
Comment on attachment 111686 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=111686&action=review This looks good to me. Would it make sense to just include the contents of gtk-doc.make in the GNUmakefile.am? > Source/WebKit/gtk/gtk-doc.make:5 > +#################################### > +# Everything below here is generic # > +#################################### I think this comment is inaccurate now? > Source/WebKit/gtk/gtk-doc.make:15 > +if GTK_DOC_USE_LIBTOOL > +GTKDOC_CC = $(LIBTOOL) --tag=CC --mode=compile $(CC) $(INCLUDES) $(GTKDOC_DEPS_CFLAGS) $(AM_CPPFLAGS) $(CPPFLAGS) $(AM_CFLAGS) $(CFLAGS) > +GTKDOC_LD = $(LIBTOOL) --tag=CC --mode=link $(CC) $(GTKDOC_DEPS_LIBS) $(AM_CFLAGS) $(CFLAGS) $(AM_LDFLAGS) $(LDFLAGS) > +GTKDOC_RUN = $(LIBTOOL) --mode=execute > +else > +GTKDOC_CC = $(CC) $(INCLUDES) $(GTKDOC_DEPS_CFLAGS) $(AM_CPPFLAGS) $(CPPFLAGS) $(AM_CFLAGS) $(CFLAGS) > +GTKDOC_LD = $(CC) $(GTKDOC_DEPS_LIBS) $(AM_CFLAGS) $(CFLAGS) $(AM_LDFLAGS) $(LDFLAGS) > +GTKDOC_RUN = > +endif Can we remove this check? > Source/WebKit/gtk/gtk-doc.make:32 > +EXTRA_DIST += \ This aberration from the norm tickles my obsessive side. :) > autogen.sh:16 > -gtkdocize --copy > /dev/null 2>&1 || echo "Warning: not running gtk-docize." > +# We do not call gtkdocize becase we use a custom version of the > +# gtk-doc.make include I think you can just omit this. We should let our newfound awesomeness stand alone. (In reply to comment #2) > This looks good to me. Would it make sense to just include the contents of gtk-doc.make in the GNUmakefile.am? I thought of doing it that way but then realized we might want to adopt this for wk2. I prefer to leave it as it is right now, and we can move the contents into the GNUmakefile.am if we decided we really don't want wk2 docs to be buildable through the build system. I'll apply the other comments =) Created attachment 111701 [details]
Patch I'll land
Committed r97901: <http://trac.webkit.org/changeset/97901> |