Bug 70447

Summary: [GTK] Streamline wk1 documentation build
Product: WebKit Reporter: Gustavo Noronha (kov) <gustavo>
Component: New BugsAssignee: Gustavo Noronha (kov) <gustavo>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch I'll land none

Description Gustavo Noronha (kov) 2011-10-19 15:52:10 PDT
[GTK] Streamline wk1 documentation build
Comment 1 Gustavo Noronha (kov) 2011-10-19 15:58:55 PDT
Created attachment 111686 [details]
Patch
Comment 2 Martin Robinson 2011-10-19 16:29:36 PDT
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.
Comment 3 Gustavo Noronha (kov) 2011-10-19 16:40:21 PDT
(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 =)
Comment 4 Gustavo Noronha (kov) 2011-10-19 17:32:42 PDT
Created attachment 111701 [details]
Patch I'll land
Comment 5 Gustavo Noronha (kov) 2011-10-19 17:37:09 PDT
Committed r97901: <http://trac.webkit.org/changeset/97901>