Skip to contentSkip to conversation

Fixes for doc-related build failures.

PATCH In review Scott Garman 1 participant 5 messages Apr 4, 2011

lore
Share & tools
lore

Scott Garman

author
#0 lore raw

cover Fixes for doc-related build failures.

From: Scott Garman <[email protected]>

Hi Saul,

Paul identified the root cause of the build failures occurring with the documentation related packages. The problem is that functions appended to SSTATEPOSTINSTFUNCS can now be run for *all* tasks. These commits ensure the sysroot-related operations only happen during the do_populate_sysroot tasks.

This fixes [YOCTO #948] (Thank you Dexuan for the bug report).

Pull URL: git://git.pokylinux.org/poky-contrib.git
Branch: sgarman/docbook-fix
Browse: http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=sgarman/docbook-fix

Thanks,
Scott Garman <[email protected]>
---

Scott Garman (3):
docbook-sgml-dtd-native.inc: run install-catalog only during
do_populate_sysroot
openjade-native: run install-catalog only during do_populate_sysroot docbook-dsssl-stylesheets-native: run install-catalog only during
do_populate_sysroot

 .../docbook-dsssl-stylesheets-native_1.79.bb       |   12 +++++++-----
 .../docbook-sgml-dtd/docbook-sgml-dtd-native.inc   |   12 +++++++-----
 .../openjade/openjade-native_1.3.2.bb              |   12 +++++++-----
 3 files changed, 21 insertions(+), 15 deletions(-)
author
#1 lore raw

PATCH 1/3 docbook-sgml-dtd-native.inc: run install-catalog only during do_populate_sysroot

From: Scott Garman <[email protected]>

Signed-off-by Scott Garman <[email protected]>
---
 .../docbook-sgml-dtd/docbook-sgml-dtd-native.inc   |   12 +++++++-----
 1 files changed, 7 insertions(+), 5 deletions(-)
meta/recipes-devtools/docbook-sgml-dtd/docbook-sgml-dtd-native.inc +7 -5
index 83cedbc..5df6e6e 100644
@@ -39,9 +39,11 @@ do_install () {
 }
 
 docbook_sgml_dtd_sstate_postinst () {
-	# Ensure that the catalog file sgml-docbook.cat is properly
-	# updated when the package is installed from sstate cache.
-	install-catalog \
-		--add ${sysconfdir}/sgml/sgml-docbook.cat \
-		${sysconfdir}/sgml/sgml-docbook-dtd-${DTD_VERSION}.cat
+	if [ "${BB_CURRENTTASK}" = "populate_sysroot" ]; then
+		# Ensure that the catalog file sgml-docbook.cat is properly
+		# updated when the package is installed from sstate cache.
+		install-catalog \
+			--add ${sysconfdir}/sgml/sgml-docbook.cat \
+			${sysconfdir}/sgml/sgml-docbook-dtd-${DTD_VERSION}.cat
+	fi
 }
Signature · 1 line
1.7.1
author
#2 lore raw

PATCH 2/3 openjade-native: run install-catalog only during do_populate_sysroot

In reply to Scott Garman

From: Scott Garman <[email protected]>

Signed-off-by Scott Garman <[email protected]>
---
 .../openjade/openjade-native_1.3.2.bb              |   12 +++++++-----
 1 files changed, 7 insertions(+), 5 deletions(-)
meta/recipes-devtools/openjade/openjade-native_1.3.2.bb +7 -5
index 5c1037a..f98eea2 100644
@@ -62,9 +62,11 @@ do_install() {
 }
 
 openjade_sstate_postinst() {
-	# Ensure that the catalog file sgml-docbook.cat is properly
-	# updated when the package is installed from sstate cache.
-	install-catalog \
-		--add ${sysconfdir}/sgml/sgml-docbook.cat \
-		${sysconfdir}/sgml/openjade-${PV}.cat
+	if [ "${BB_CURRENTTASK}" = "populate_sysroot" ]; then
+		# Ensure that the catalog file sgml-docbook.cat is properly
+		# updated when the package is installed from sstate cache.
+		install-catalog \
+			--add ${sysconfdir}/sgml/sgml-docbook.cat \
+			${sysconfdir}/sgml/openjade-${PV}.cat
+	fi
 }
Signature · 1 line
1.7.1
author
#3 lore raw

PATCH 3/3 docbook-dsssl-stylesheets-native: run install-catalog only during do_populate_sysroot

In reply to Scott Garman

From: Scott Garman <[email protected]>

Signed-off-by Scott Garman <[email protected]>
---
 .../docbook-dsssl-stylesheets-native_1.79.bb       |   12 +++++++-----
 1 files changed, 7 insertions(+), 5 deletions(-)
meta/recipes-devtools/docbook-dsssl-stylesheets/docbook-dsssl-stylesheets-native_1.79.bb +7 -5
index fc4d266..b680384 100644
@@ -38,9 +38,11 @@ do_install () {
 }
 
 docbook_dsssl_stylesheets_sstate_postinst () {
-	# Ensure that the catalog file sgml-docbook.cat is properly
-	# updated when the package is installed from sstate cache.
-	install-catalog \
-		--add ${sysconfdir}/sgml/sgml-docbook.cat \
-		${sysconfdir}/sgml/dsssl-docbook-stylesheets.cat
+	if [ "${BB_CURRENTTASK}" = "populate_sysroot" ]; then
+		# Ensure that the catalog file sgml-docbook.cat is properly
+		# updated when the package is installed from sstate cache.
+		install-catalog \
+			--add ${sysconfdir}/sgml/sgml-docbook.cat \
+			${sysconfdir}/sgml/dsssl-docbook-stylesheets.cat
+	fi
 }
Signature · 1 line
1.7.1
author
#4 lore raw
In reply to Scott Garman
Quoted 11 lines · Scott Garman

From: Scott Garman<[email protected]>

Hi Saul,

Paul identified the root cause of the build failures occurring with the documentation related packages. The problem is that functions appended to SSTATEPOSTINSTFUNCS can now be run for *all* tasks. These commits ensure the sysroot-related operations only happen during the do_populate_sysroot tasks.

This fixes [YOCTO #948] (Thank you Dexuan for the bug report).

Richard just alerted me to a problem with this pull request. Going to re-spin shortly.

Scott

Signature · 3 lines
Scott Garman
Embedded Linux Engineer - Yocto Project
Intel Open Source Technology Center