<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Mon, Feb 15, 2016 at 9:58 AM Richard Purdie <<a href="mailto:richard.purdie@linuxfoundation.org">richard.purdie@linuxfoundation.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Mon, 2016-02-15 at 16:47 +0000, Christopher Larson wrote:<br>
><br>
><br>
> On Mon, Feb 15, 2016 at 8:45 AM Richard Purdie <<br>
> <a href="mailto:richard.purdie@linuxfoundation.org" target="_blank">richard.purdie@linuxfoundation.org</a>> wrote:<br>
> > Also clean up use of immediate expansion as there are cleaner<br>
> > mechanisms now.<br>
> ><br>
> > Signed-off-by: Richard Purdie <<a href="mailto:richard.purdie@linuxfoundation.org" target="_blank">richard.purdie@linuxfoundation.org</a>><br>
> ><br>
> > diff --git a/meta/recipes-extended/ed/<a href="http://ed_0.5.bb" rel="noreferrer" target="_blank">ed_0.5.bb</a> b/meta/recipes<br>
> > -extended/ed/<a href="http://ed_0.5.bb" rel="noreferrer" target="_blank">ed_0.5.bb</a><br>
> > index 143e636..e2df13f 100644<br>
> > --- a/meta/recipes-extended/ed/<a href="http://ed_0.5.bb" rel="noreferrer" target="_blank">ed_0.5.bb</a><br>
> > +++ b/meta/recipes-extended/ed/<a href="http://ed_0.5.bb" rel="noreferrer" target="_blank">ed_0.5.bb</a><br>
> > @@ -15,7 +15,6 @@ inherit autotools texinfo<br>
> ><br>
> > EXTRA_OECONF = "'CC=${CC}' 'CXX=${CXX}' 'CFLAGS=${CFLAGS}'<br>
> > 'CXXFLAGS=${CXXFLAGS}' 'CPPFLAGS=${CPPFLAGS}' 'LDFLAGS=${LDFLAGS}'"<br>
> ><br>
> > -CONFIGUREOPTS := "${@d.getVar('CONFIGUREOPTS', True).replace('-<br>
> > -disable-dependency-tracking', ' ')}"<br>
> > -CONFIGUREOPTS := "${@d.getVar('CONFIGUREOPTS', True).replace('-<br>
> > -disable-silent-rules', ' ')}"<br>
> > -<br>
> > -<br>
> > +CONFIGUREOPTS_remove = "--disable-dependency-tracking"<br>
> > +CONFIGUREOPTS_remove = "--disable-silent-rules"<br>
> > +EXTRA_OECONF_remove = "--disable-static"<br>
> This will be impossible to undo in a bbappend..<br>
<br>
I appreciate that and we've talked about ways we likely need to extend<br>
the API for that.<br>
<br>
In this case, I would ask why would anyone want/need to add in a<br>
configure option which configure doesn't support?<br></blockquote><div><br></div><div>Right, fair enough, I didn't read the patch carefully enough. Sorry for the distraction. </div></div></div>