]> code.ossystems Code Review - openembedded-core.git/commitdiff
gtk-doc: don't regenerate gtk-doc in do_install
authorRoss Burton <ross.burton@intel.com>
Wed, 21 Mar 2018 14:47:29 +0000 (14:47 +0000)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Sun, 25 Mar 2018 08:33:36 +0000 (09:33 +0100)
In out-of-tree builds gtk-doc's setup-build target copies all the content from
$srcdir to $builddir. However, if some of this content is regenerated at
configure time this can happen:

1) configure writes new build/version.xml
2) make compile copies content, including the tarball's src/version.xml
   to build/version.xml, and generates gtk-doc.
3) make install notices build/version.xml is older than configure.status,
   so regenerates gtk-doc.

gtk-doc generation is a slow process at the best of times, so doing it twice
isn't good.

Signed-off-by: Ross Burton <ross.burton@intel.com>
meta/recipes-gnome/gtk-doc/files/no-clobber.patch [new file with mode: 0644]
meta/recipes-gnome/gtk-doc/gtk-doc_1.27.bb

diff --git a/meta/recipes-gnome/gtk-doc/files/no-clobber.patch b/meta/recipes-gnome/gtk-doc/files/no-clobber.patch
new file mode 100644 (file)
index 0000000..0fa22b2
--- /dev/null
@@ -0,0 +1,27 @@
+In out-of-tree builds gtk-doc's setup-build target copies all the content from $srcdir to $builddir.
+
+However, if some of this content is regenerated at configure time this can happen:
+
+1) configure writes new build/version.xml
+2) make compile copies content, including the tarball's src/version.xml
+   to build/version.xml, and generates gtk-doc.
+3) make install notices build/version.xml is older than configure.status,
+   so regenerates gtk-doc.
+
+gtk-doc generation is a slow process at the best of times, so doing it twice isn't good.
+
+Solve this by changing cp --force to cp --no-clobber, so setup-build only copies
+files which don't already exist.
+
+Upstream-Status: Submitted [https://bugzilla.gnome.org/show_bug.cgi?id=794571]
+Signed-off-by: Ross Burton <ross.burton@intel.com>
+
+diff --git a/gtk-doc.make b/gtk-doc.make
+index f87eaab..246f3c0 100644
+--- a/gtk-doc.make
++++ b/gtk-doc.make
+@@ -113,3 +113,3 @@ setup-build.stamp:
+             test -f $(abs_srcdir)/$$file && \
+-              cp -pf $(abs_srcdir)/$$file $(abs_builddir)/$$file || true; \
++              cp -pn $(abs_srcdir)/$$file $(abs_builddir)/$$file || true; \
+           done; \
index b907bae147808cac4c27fc5e2b93251953381a12..45528111c1341f0840161fa1fdae7de2ae171d86 100644 (file)
@@ -23,6 +23,7 @@ SRC_URI[archive.sha256sum] = "e26bd3f7080c749b1cb66c46c6bf8239e2f320a949964fb9c6
 SRC_URI += "file://0001-Do-not-hardocode-paths-to-perl-python-in-scripts.patch \
            file://0001-Do-not-error-out-if-xsltproc-is-not-found.patch \
            file://conditionaltests.patch \
+           file://no-clobber.patch \
            "
 SRC_URI_append_class-native = " file://pkg-config-native.patch"