<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"
"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head><meta http-equiv="content-type" content="text/html; charset=utf-8" />
<title>[150326] trunk</title>
</head>
<body>

<style type="text/css"><!--
#msg dl.meta { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }
#msg dl.meta dt { float: left; width: 6em; font-weight: bold; }
#msg dt:after { content:':';}
#msg dl, #msg dt, #msg ul, #msg li, #header, #footer, #logmsg { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt;  }
#msg dl a { font-weight: bold}
#msg dl a:link    { color:#fc3; }
#msg dl a:active  { color:#ff0; }
#msg dl a:visited { color:#cc6; }
h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }
#msg pre { overflow: auto; background: #ffc; border: 1px #fa0 solid; padding: 6px; }
#logmsg { background: #ffc; border: 1px #fa0 solid; padding: 1em 1em 0 1em; }
#logmsg p, #logmsg pre, #logmsg blockquote { margin: 0 0 1em 0; }
#logmsg p, #logmsg li, #logmsg dt, #logmsg dd { line-height: 14pt; }
#logmsg h1, #logmsg h2, #logmsg h3, #logmsg h4, #logmsg h5, #logmsg h6 { margin: .5em 0; }
#logmsg h1:first-child, #logmsg h2:first-child, #logmsg h3:first-child, #logmsg h4:first-child, #logmsg h5:first-child, #logmsg h6:first-child { margin-top: 0; }
#logmsg ul, #logmsg ol { padding: 0; list-style-position: inside; margin: 0 0 0 1em; }
#logmsg ul { text-indent: -1em; padding-left: 1em; }#logmsg ol { text-indent: -1.5em; padding-left: 1.5em; }
#logmsg > ul, #logmsg > ol { margin: 0 0 1em 0; }
#logmsg pre { background: #eee; padding: 1em; }
#logmsg blockquote { border: 1px solid #fa0; border-left-width: 10px; padding: 1em 1em 0 1em; background: white;}
#logmsg dl { margin: 0; }
#logmsg dt { font-weight: bold; }
#logmsg dd { margin: 0; padding: 0 0 0.5em 0; }
#logmsg dd:before { content:'\00bb';}
#logmsg table { border-spacing: 0px; border-collapse: collapse; border-top: 4px solid #fa0; border-bottom: 1px solid #fa0; background: #fff; }
#logmsg table th { text-align: left; font-weight: normal; padding: 0.2em 0.5em; border-top: 1px dotted #fa0; }
#logmsg table td { text-align: right; border-top: 1px dotted #fa0; padding: 0.2em 0.5em; }
#logmsg table thead th { text-align: center; border-bottom: 1px solid #fa0; }
#logmsg table th.Corner { text-align: left; }
#logmsg hr { border: none 0; border-top: 2px dashed #fa0; height: 1px; }
#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }
#patch { width: 100%; }
#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}
#patch .propset h4, #patch .binary h4 {margin:0;}
#patch pre {padding:0;line-height:1.2em;margin:0;}
#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}
#patch .propset .diff, #patch .binary .diff  {padding:10px 0;}
#patch span {display:block;padding:0 10px;}
#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}
#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}
#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}
#patch .lines, .info {color:#888;background:#fff;}
--></style>
<div id="msg">
<dl class="meta">
<dt>Revision</dt> <dd><a href="http://trac.webkit.org/projects/webkit/changeset/150326">150326</a></dd>
<dt>Author</dt> <dd>commit-queue@webkit.org</dd>
<dt>Date</dt> <dd>2013-05-18 00:19:37 -0700 (Sat, 18 May 2013)</dd>
</dl>

<h3>Log Message</h3>
<pre>[GTK] Parallel build fails if gtk-doc is enabled
https://bugs.webkit.org/show_bug.cgi?id=116227

Patch by Alberto Garcia &lt;agarcia@igalia.com&gt; on 2013-05-18
Reviewed by Martin Robinson.

.:

* GNUmakefile.am:
Define an empty noinst_DATA variable where other automake files
can add additional objects to be built.

Source/WebCore:

* GNUmakefile.am:
Don't overwrite noinst_DATA, modify its existing value instead.

Tools:

* GNUmakefile.am:
Don't use the 'all' target to generate the documentation.
The prerequisites of this target must be files that need to be
built _before_ the actual compilation of WebKit starts. Since the
documentation depends on WebKit being built first, this means in
practice that the code will be built in parallel with the
generation of its dependencies, potentially making the build fail.</pre>

<h3>Modified Paths</h3>
<ul>
<li><a href="#trunkChangeLog">trunk/ChangeLog</a></li>
<li><a href="#trunkGNUmakefileam">trunk/GNUmakefile.am</a></li>
<li><a href="#trunkSourceWebCoreChangeLog">trunk/Source/WebCore/ChangeLog</a></li>
<li><a href="#trunkSourceWebCoreGNUmakefileam">trunk/Source/WebCore/GNUmakefile.am</a></li>
<li><a href="#trunkToolsChangeLog">trunk/Tools/ChangeLog</a></li>
<li><a href="#trunkToolsGNUmakefileam">trunk/Tools/GNUmakefile.am</a></li>
</ul>

</div>
<div id="patch">
<h3>Diff</h3>
<a id="trunkChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/ChangeLog (150325 => 150326)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/ChangeLog        2013-05-18 07:12:01 UTC (rev 150325)
+++ trunk/ChangeLog        2013-05-18 07:19:37 UTC (rev 150326)
</span><span class="lines">@@ -1,3 +1,14 @@
</span><ins>+2013-05-18  Alberto Garcia  &lt;agarcia@igalia.com&gt;
+
+        [GTK] Parallel build fails if gtk-doc is enabled
+        https://bugs.webkit.org/show_bug.cgi?id=116227
+
+        Reviewed by Martin Robinson.
+
+        * GNUmakefile.am:
+        Define an empty noinst_DATA variable where other automake files
+        can add additional objects to be built.
+
</ins><span class="cx"> 2013-05-16  Martin Robinson  &lt;mrobinson@igalia.com&gt;
</span><span class="cx"> 
</span><span class="cx">         [GTK] [CMake] Disable the shadow DOM
</span></span></pre></div>
<a id="trunkGNUmakefileam"></a>
<div class="modfile"><h4>Modified: trunk/GNUmakefile.am (150325 => 150326)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/GNUmakefile.am        2013-05-18 07:12:01 UTC (rev 150325)
+++ trunk/GNUmakefile.am        2013-05-18 07:19:37 UTC (rev 150326)
</span><span class="lines">@@ -50,6 +50,7 @@
</span><span class="cx"> bin_PROGRAMS :=
</span><span class="cx"> noinst_PROGRAMS :=
</span><span class="cx"> libexec_PROGRAMS :=
</span><ins>+noinst_DATA :=
</ins><span class="cx"> noinst_HEADERS :=
</span><span class="cx"> noinst_LTLIBRARIES :=
</span><span class="cx"> lib_LTLIBRARIES :=
</span></span></pre></div>
<a id="trunkSourceWebCoreChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/ChangeLog (150325 => 150326)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/ChangeLog        2013-05-18 07:12:01 UTC (rev 150325)
+++ trunk/Source/WebCore/ChangeLog        2013-05-18 07:19:37 UTC (rev 150326)
</span><span class="lines">@@ -1,3 +1,13 @@
</span><ins>+2013-05-18  Alberto Garcia  &lt;agarcia@igalia.com&gt;
+
+        [GTK] Parallel build fails if gtk-doc is enabled
+        https://bugs.webkit.org/show_bug.cgi?id=116227
+
+        Reviewed by Martin Robinson.
+
+        * GNUmakefile.am:
+        Don't overwrite noinst_DATA, modify its existing value instead.
+
</ins><span class="cx"> 2013-05-17  Michelangelo De Simone  &lt;michelangelo@webkit.org&gt;
</span><span class="cx"> 
</span><span class="cx">         CSSParser parsing of CSSPropertyFontStretch falls into CSSPropertyGeometry
</span></span></pre></div>
<a id="trunkSourceWebCoreGNUmakefileam"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/GNUmakefile.am (150325 => 150326)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/GNUmakefile.am        2013-05-18 07:12:01 UTC (rev 150325)
+++ trunk/Source/WebCore/GNUmakefile.am        2013-05-18 07:19:37 UTC (rev 150326)
</span><span class="lines">@@ -729,7 +729,7 @@
</span><span class="cx"> 
</span><span class="cx"> # It seems that $(shell) does not expand when it is a rule dependency, so
</span><span class="cx"> # we must redefine this list of copied files with traditional dependency wildcards.
</span><del>-noinst_DATA = ${GENSOURCES_INSPECTOR}/inspector.html
</del><ins>+noinst_DATA += ${GENSOURCES_INSPECTOR}/inspector.html
</ins><span class="cx"> ${GENSOURCES_INSPECTOR}/inspector.html: $(WebCore)/inspector/front-end/*.html \
</span><span class="cx">                 $(WebCore)/inspector/front-end/*.js \
</span><span class="cx">                 $(WebCore)/inspector/front-end/UglifyJS/*.js \
</span></span></pre></div>
<a id="trunkToolsChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Tools/ChangeLog (150325 => 150326)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Tools/ChangeLog        2013-05-18 07:12:01 UTC (rev 150325)
+++ trunk/Tools/ChangeLog        2013-05-18 07:19:37 UTC (rev 150326)
</span><span class="lines">@@ -1,3 +1,18 @@
</span><ins>+2013-05-18  Alberto Garcia  &lt;agarcia@igalia.com&gt;
+
+        [GTK] Parallel build fails if gtk-doc is enabled
+        https://bugs.webkit.org/show_bug.cgi?id=116227
+
+        Reviewed by Martin Robinson.
+
+        * GNUmakefile.am:
+        Don't use the 'all' target to generate the documentation.
+        The prerequisites of this target must be files that need to be
+        built _before_ the actual compilation of WebKit starts. Since the
+        documentation depends on WebKit being built first, this means in
+        practice that the code will be built in parallel with the
+        generation of its dependencies, potentially making the build fail.
+
</ins><span class="cx"> 2013-05-17  Ryosuke Niwa  &lt;rniwa@webkit.org&gt;
</span><span class="cx"> 
</span><span class="cx">         Don't run hg.
</span></span></pre></div>
<a id="trunkToolsGNUmakefileam"></a>
<div class="modfile"><h4>Modified: trunk/Tools/GNUmakefile.am (150325 => 150326)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Tools/GNUmakefile.am        2013-05-18 07:12:01 UTC (rev 150325)
+++ trunk/Tools/GNUmakefile.am        2013-05-18 07:19:37 UTC (rev 150326)
</span><span class="lines">@@ -425,6 +425,6 @@
</span><span class="cx"> endif
</span><span class="cx"> 
</span><span class="cx"> if ENABLE_GTK_DOC
</span><del>-all: docs-build.stamp
</del><ins>+noinst_DATA += docs-build.stamp
</ins><span class="cx"> endif
</span><span class="cx"> 
</span></span></pre>
</div>
</div>

</body>
</html>