Patchwork openjade-native_1.3.2.bb: remove CONFIGUREOPTS as vardep for do_configure

login
register
mail settings
Submitter Matthew McClintock
Date Dec. 4, 2011, 8:09 p.m.
Message ID <1323029354-14952-1-git-send-email-msm@freescale.com>
Download mbox | patch
Permalink /patch/16159/
State Accepted
Commit ddb8d3de34f809b9c72eb3a2223a74f75eff7911
Headers show

Comments

Matthew McClintock - Dec. 4, 2011, 8:09 p.m.
This variable was being expanded immediately and pulling in
paths to the variable dependecies which causes it's sstate-cache
to never be reused

Signed-off-by: Matthew McClintock <msm@freescale.com>
---
 .../openjade/openjade-native_1.3.2.bb              |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)
Richard Purdie - Dec. 5, 2011, 4:08 p.m.
On Sun, 2011-12-04 at 14:09 -0600, Matthew McClintock wrote:
> This variable was being expanded immediately and pulling in
> paths to the variable dependecies which causes it's sstate-cache
> to never be reused
> 
> Signed-off-by: Matthew McClintock <msm@freescale.com>
> ---
>  .../openjade/openjade-native_1.3.2.bb              |    5 +++++
>  1 files changed, 5 insertions(+), 0 deletions(-)

Merged to master, thanks.

Richard

Patch

diff --git a/meta/recipes-devtools/openjade/openjade-native_1.3.2.bb b/meta/recipes-devtools/openjade/openjade-native_1.3.2.bb
index 6cc6bb5..72f5bb2 100644
--- a/meta/recipes-devtools/openjade/openjade-native_1.3.2.bb
+++ b/meta/recipes-devtools/openjade/openjade-native_1.3.2.bb
@@ -28,6 +28,11 @@  EXTRA_OECONF = "--enable-spincludedir=${STAGING_INCDIR}/OpenSP \
 # results in it being specified twice when configure is run.
 CONFIGUREOPTS := "${@d.getVar('CONFIGUREOPTS', True).replace('--datadir=${datadir}', '--datadir=${STAGING_DATADIR}/sgml/openjade-${PV}')}"
 
+# CONFIGUREOPTS has hard coded paths so we need to ignore it's vardeps
+# there are other bits in there too but they are picked up by other variable
+# dependencies so it all works out
+do_configure[vardepsexclude] += "CONNFIGUREOPTS"
+
 CFLAGS =+ "-I${S}/include"
 
 SSTATEPOSTINSTFUNCS += "openjade_sstate_postinst"