<div dir="ltr">Probably yes, I don&#39;t know why it was written in this way before and I&#39;ve just blindly followed it to fix the issue I was seeing.<div><br></div><div>v2 coming soon.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Dec 5, 2015 at 12:55 AM, Richard Purdie <span dir="ltr">&lt;<a href="mailto:richard.purdie@linuxfoundation.org" target="_blank">richard.purdie@linuxfoundation.org</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Fri, 2015-12-04 at 21:47 +0100, Martin Jansa wrote:<br>
&gt; * we don&#39;t want the do_package signature depending on INHERIT variable<br>
&gt; * e.g. just adding the own-mirrors causes texinfo to rebuild:<br>
&gt;   # bitbake-diffsigs BUILD/sstate-diff/*/*/texinfo/*do_package.sig*<br>
&gt;   basehash changed from 015df2fd8e396cc1e15622dbac843301 to 9f1d06c4f238c70a99ccb6d8da348b6a<br>
&gt;   Variable INHERIT value changed from<br>
&gt;   &#39; rm_work blacklist blacklist report-error ${PACKAGE_CLASSES} ${USER_CLASSES} ${INHERIT_DISTRO} ${INHERIT_BLACKLIST} sanity&#39;<br>
&gt;   to<br>
&gt;   &#39; rm_work own-mirrors blacklist blacklist report-error ${PACKAGE_CLASSES} ${USER_CLASSES} ${INHERIT_DISTRO} ${INHERIT_BLACKLIST} sanity&#39;<br>
&gt;<br>
&gt; Signed-off-by: Martin Jansa &lt;<a href="mailto:Martin.Jansa@gmail.com">Martin.Jansa@gmail.com</a>&gt;<br>
&gt; ---<br>
&gt;  meta/recipes-extended/texinfo/<a href="http://texinfo_6.0.bb" rel="noreferrer" target="_blank">texinfo_6.0.bb</a> | 5 ++++-<br>
&gt;  1 file changed, 4 insertions(+), 1 deletion(-)<br>
&gt;<br>
&gt; diff --git a/meta/recipes-extended/texinfo/<a href="http://texinfo_6.0.bb" rel="noreferrer" target="_blank">texinfo_6.0.bb</a> b/meta/recipes-extended/texinfo/<a href="http://texinfo_6.0.bb" rel="noreferrer" target="_blank">texinfo_6.0.bb</a><br>
&gt; index 8fb715a..b0a6975 100644<br>
&gt; --- a/meta/recipes-extended/texinfo/<a href="http://texinfo_6.0.bb" rel="noreferrer" target="_blank">texinfo_6.0.bb</a><br>
&gt; +++ b/meta/recipes-extended/texinfo/<a href="http://texinfo_6.0.bb" rel="noreferrer" target="_blank">texinfo_6.0.bb</a><br>
&gt; @@ -9,8 +9,11 @@ LIC_FILES_CHKSUM = &quot;file://COPYING;md5=d32239bcb673463ab874e80d47fae504&quot;<br>
&gt;<br>
&gt;  PROVIDES_append_class-native = &quot; texinfo-replacement-native&quot;<br>
&gt;<br>
&gt; +DOC_COMPRESS_ENABLED = &quot;${@ &#39;1&#39; if &#39;compress_doc&#39; in (d.getVar(&#39;INHERIT&#39;, True) or &#39;&#39;).split() else &#39;0&#39; }&quot;<br>
&gt; +DOC_COMPRESS_ENABLED[vardepvalue] = &quot;${DOC_COMPRESS_ENABLED}&quot;<br>
&gt; +<br>
&gt;  def compress_pkg(d):<br>
&gt; -    if &quot;compress_doc&quot; in (d.getVar(&quot;INHERIT&quot;, True) or &quot;&quot;).split():<br>
&gt; +    if d.getVar(&quot;DOC_COMPRESS_ENABLED&quot;, True) == &quot;1&quot;:<br>
<br>
</div></div>Could we use:<br>
<br>
if bb.data.inherits_class(&quot;compress_doc&quot;, d):<br>
<br>
?<br>
<br>
Cheers,<br>
<br>
Richard<br>
<br>
</blockquote></div><br></div>