<div dir="ltr">On 22 March 2017 at 22:17, Patel, Vedang &lt;<a href="mailto:vedang.patel@intel.com">vedang.patel@intel.com</a>&gt; wrote:<br>&gt;<br>&gt; Hi Jussi,<br>&gt;<div><br></div><div>Hi Vedang,</div><div><br></div><div>If you can make your email client use &quot;&gt;&quot; for quoting that would be great (so it&#39;s possible to see who&#39;s talking even in plain text).</div><div><br></div><div>PACKAGECONFIG comments below.</div><div><br>&gt; On Tue, 2017-03-21 at 08:36 +0000, Kukkonen, Jussi wrote:<br>&gt; On 21 March 2017 at 02:31, Vedang Patel &lt;<a href="mailto:vedang.patel@intel.com">vedang.patel@intel.com</a>&lt;mailto:<a href="mailto:vedang.patel@intel.com">vedang.patel@intel.com</a>&gt;&gt; wrote:<br>&gt; Some options like python bindings, debug support, crypto are hardcoded<br>&gt; inside the recipe. Change that to make those option configurable using<br>&gt; PACKAGECONFIG.<br>&gt;<br>&gt; Signed-off-by: Vedang Patel &lt;<a href="mailto:vedang.patel@intel.com">vedang.patel@intel.com</a>&lt;mailto:<a href="mailto:vedang.patel@intel.com">vedang.patel@intel.com</a>&gt;&gt;<br>&gt; ---<br>&gt;  meta/recipes-support/libxslt/<a href="http://libxslt_1.1.29.bb">libxslt_1.1.29.bb</a>&lt;<a href="http://libxslt_1.1.29.bb">http://libxslt_1.1.29.bb</a>&gt; | 9 +++++++--<br>&gt;  1 file changed, 7 insertions(+), 2 deletions(-)<br>&gt;<br>&gt; diff --git a/meta/recipes-support/libxslt/<a href="http://libxslt_1.1.29.bb">libxslt_1.1.29.bb</a>&lt;<a href="http://libxslt_1.1.29.bb">http://libxslt_1.1.29.bb</a>&gt; b/meta/recipes-support/libxslt/<a href="http://libxslt_1.1.29.bb">libxslt_1.1.29.bb</a>&lt;<a href="http://libxslt_1.1.29.bb">http://libxslt_1.1.29.bb</a>&gt;<br>&gt; index be747e608d9d..d362118aa307 100644<br>&gt; --- a/meta/recipes-support/libxslt/<a href="http://libxslt_1.1.29.bb">libxslt_1.1.29.bb</a>&lt;<a href="http://libxslt_1.1.29.bb">http://libxslt_1.1.29.bb</a>&gt;<br>&gt; +++ b/meta/recipes-support/libxslt/<a href="http://libxslt_1.1.29.bb">libxslt_1.1.29.bb</a>&lt;<a href="http://libxslt_1.1.29.bb">http://libxslt_1.1.29.bb</a>&gt;<br>&gt; @@ -22,7 +22,7 @@ S = &quot;${WORKDIR}/libxslt-${PV}&quot;<br>&gt;<br>&gt;  BINCONFIG = &quot;${bindir}/xslt-config&quot;<br>&gt;<br>&gt; -inherit autotools pkgconfig binconfig-disabled lib_package<br>&gt; +inherit autotools pkgconfig binconfig-disabled lib_package distutils-common-base<br>&gt;<br>&gt;  # We don&#39;t DEPEND on binutils for ansidecl.h so ensure we don&#39;t use the header<br>&gt;  do_configure_prepend () {<br>&gt; @@ -33,7 +33,12 @@ do_configure_prepend () {<br>&gt;         touch ${S}/doc/xsltproc.1<br>&gt;  }<br>&gt;<br>&gt; -EXTRA_OECONF = &quot;--without-python --without-debug --without-mem-debug --without-crypto&quot;<br>&gt; +PACKAGECONFIG ??= &quot;python libxslt-debug libxslt-mem-debug libxslt-crypto&quot;<br>&gt;<br>&gt;<br>&gt; You change all the defaults, is this on purpose? It should be noted in the commit message in any case.<br>&gt;<br>&gt; Can you elabore on your concern? I am not exactly following it.<br>&gt;<br>&gt; the configs won&#39;t be enabled unless the corresponding features are explicitly enabled in the image. I am using libxslt- prefix to make sure someone does not inadvertently enables the features by using the generic flags (python, debug, ... ).<br>&gt;There seems to be some confusion here.<br></div><div><br></div><div>I think there&#39;s some confusion here about PACKAGECONFIG and image features: PACKAGECONFIG is completely recipe specific -- it only controls this recipe. There&#39;s no need to use a naming prefix as these variables are only visible inside the recipe. Changing them from outside (like local.conf) is possible but requires a syntax like</div><div>    PACKAGECONFIG_pn-libxslt = &quot;python crypto&quot; </div><div>so there&#39;s no need to fear namespace mixups.</div><div><br></div><div>Second, this line: </div><div>    PACKAGECONFIG ??= &quot;python libxslt-debug libxslt-mem-debug libxslt-crypto&quot;<br></div><div>sets he default value of libxslt PACKAGECONFIG. Meaning that these options will be given to configure: </div><div>    &quot;--with-python=&lt;ver&gt; --with-debug --with-mem-debug --with-crypto&quot;</div><div>Currently the EXTRA_OECONF line explicitly disables all these so this looks like a change in defaults.</div><div><br></div><div>If in doubt (or I didn&#39;t make sense) you should run &quot;bitbake -cconfigure libxslt&quot; with both versions and check what ends up in the configure options in $WORKDIR/temp/log.do_configure.</div><div><br></div><div>Cheers,</div><div>  Jussi</div><div><br></div><div><br></div><div><br></div><div>&gt; I do see a bug here, In the first line, python should be replaced with libxslt-python. I will fix it in the next version of this patch.</div><div>&gt;<br></div><div>&gt; Thanks,<br>&gt; Vedang Patel<br>&gt; Software Engineer<br>&gt; Intel Corporation<br>&gt;<br>&gt; +PACKAGECONFIG[libxslt-python] = &quot;--with-python=${PYTHON_BASE_VERSION}, --without-python&quot;<br>&gt; +PACKAGECONFIG[libxslt-debug] = &quot;--with-debug, --without-debug&quot;<br>&gt; +PACKAGECONFIG[libxslt-mem-debug] = &quot;--with-mem-debug, --without-mem-debug&quot;<br>&gt; +PACKAGECONFIG[libxslt-crypto] = &quot;--with-crypto, --without-crypto&quot;<br>&gt; +<br>&gt;  # older versions of this recipe had ${PN}-utils<br>&gt;  RPROVIDES_${PN}-bin += &quot;${PN}-utils&quot;<br>&gt;  RCONFLICTS_${PN}-bin += &quot;${PN}-utils&quot;<br>&gt; --<br>&gt; 2.7.3<br>&gt;<br>&gt; --<br>&gt; _______________________________________________<br>&gt; Openembedded-core mailing list<br>&gt; <a href="mailto:Openembedded-core@lists.openembedded.org">Openembedded-core@lists.openembedded.org</a>&lt;mailto:<a href="mailto:Openembedded-core@lists.openembedded.org">Openembedded-core@lists.openembedded.org</a>&gt;<br>&gt; <a href="http://lists.openembedded.org/mailman/listinfo/openembedded-core">http://lists.openembedded.org/mailman/listinfo/openembedded-core</a><br>&gt;<br>&gt;<br></div></div>