Patchwork [3/3] texinfo.bbclass: native/cross uses dummy texinfo; target uses host's Texinfo.

login
register
mail settings
Submitter Max Eliaser
Date June 10, 2014, 8:24 p.m.
Message ID <4b7de930ed0850b747248daf99495d5779d7156d.1402431641.git.max.eliaser@intel.com>
Download mbox | patch
Permalink /patch/73589/
State New
Headers show

Comments

Max Eliaser - June 10, 2014, 8:24 p.m.
To unpack that to more than a single line: -native and -cross recipes are made
to use the dummy Texinfo utilities provided by texinfo-dummy-native if they
invoke those utilities at build time. The target-architecture (cross-compiled)
recipes still use the genuine Texinfo utilites. Right now, they still use
the host system's Texinfo utilities, but could be made to use the
texinfo-native recipe we already ship with some config file changes.

Signed-off-by: Max Eliaser <max.eliaser@intel.com>
---
 meta/classes/texinfo.bbclass | 15 ++++++++++++++-
 meta/conf/bitbake.conf       |  1 +
 2 files changed, 15 insertions(+), 1 deletion(-)
Richard Purdie - June 13, 2014, 11:32 a.m.
On Tue, 2014-06-10 at 13:24 -0700, Max Eliaser wrote:
> To unpack that to more than a single line: -native and -cross recipes are made
> to use the dummy Texinfo utilities provided by texinfo-dummy-native if they
> invoke those utilities at build time. The target-architecture (cross-compiled)
> recipes still use the genuine Texinfo utilites. Right now, they still use
> the host system's Texinfo utilities, but could be made to use the
> texinfo-native recipe we already ship with some config file changes.
> 
> Signed-off-by: Max Eliaser <max.eliaser@intel.com>
> ---
>  meta/classes/texinfo.bbclass | 15 ++++++++++++++-
>  meta/conf/bitbake.conf       |  1 +
>  2 files changed, 15 insertions(+), 1 deletion(-)
> 
> diff --git a/meta/classes/texinfo.bbclass b/meta/classes/texinfo.bbclass
> index 60cba0f..92efbcc 100644
> --- a/meta/classes/texinfo.bbclass
> +++ b/meta/classes/texinfo.bbclass
> @@ -1,2 +1,15 @@
>  # This class is inherited by recipes whose upstream packages invoke the
> -# texinfo utilities at build-time. This class currently does nothing.
> +# texinfo utilities at build-time. Native and cross recipes are made to use the
> +# dummy scripts provided by texinfo-dummy-native, for improved performance. 
> +# Target architecture recipes use the genuine Texinfo utilities. By default, 
> +# they use the Texinfo utilities on the host system. If you want to use the
> +# Texinfo recipe shipped with yoco, you can remove texinfo-native from 
> +# ASSUME_PROVIDED and makeinfo from SANITY_REQUIRED_UTILITIES.
> +
> +TEXDEP = "texinfo-native"
> +TEXDEP_class-native = "texinfo-dummy-native"
> +TEXDEP_class-cross = "texinfo-dummy-native"
> +

I took this patch but I noticed we should have a class-crosssdk in there
too. Its not a big issue, just a completeness thing.

Cheers,

Richard

Patch

diff --git a/meta/classes/texinfo.bbclass b/meta/classes/texinfo.bbclass
index 60cba0f..92efbcc 100644
--- a/meta/classes/texinfo.bbclass
+++ b/meta/classes/texinfo.bbclass
@@ -1,2 +1,15 @@ 
 # This class is inherited by recipes whose upstream packages invoke the
-# texinfo utilities at build-time. This class currently does nothing.
+# texinfo utilities at build-time. Native and cross recipes are made to use the
+# dummy scripts provided by texinfo-dummy-native, for improved performance. 
+# Target architecture recipes use the genuine Texinfo utilities. By default, 
+# they use the Texinfo utilities on the host system. If you want to use the
+# Texinfo recipe shipped with yoco, you can remove texinfo-native from 
+# ASSUME_PROVIDED and makeinfo from SANITY_REQUIRED_UTILITIES.
+
+TEXDEP = "texinfo-native"
+TEXDEP_class-native = "texinfo-dummy-native"
+TEXDEP_class-cross = "texinfo-dummy-native"
+DEPENDS_append = " ${TEXDEP}"
+PATH_prepend_class-native = "${STAGING_BINDIR_NATIVE}/texinfo-dummy-native:"
+PATH_prepend_class-cross = "${STAGING_BINDIR_NATIVE}/texinfo-dummy-native:"
+
diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
index 1d70d3c..cd6f940 100644
--- a/meta/conf/bitbake.conf
+++ b/meta/conf/bitbake.conf
@@ -181,6 +181,7 @@  ASSUME_PROVIDED = "\
     python-native-runtime \
     tar-native \
     virtual/libintl-native \
+    texinfo-native\
     "
 # gzip-native should be listed above?