<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Fri, Jun 22, 2018 at 12:36 AM Richard Purdie &lt;<a href="mailto:richard.purdie@linuxfoundation.org">richard.purdie@linuxfoundation.org</a>&gt; wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Fri, 2018-06-22 at 02:07 +0500, Christopher Larson wrote:<br>
&gt; From: Christopher Larson &lt;<a href="mailto:chris_larson@mentor.com" target="_blank">chris_larson@mentor.com</a>&gt;<br>
&gt; <br>
&gt; It&#39;s not common practice to make -doc rdep on man when man pages are<br>
&gt; included, and doing so prevents the recipe from building for non-<br>
&gt; gplv3<br>
&gt; <br>
&gt; Signed-off-by: Christopher Larson &lt;<a href="mailto:chris_larson@mentor.com" target="_blank">chris_larson@mentor.com</a>&gt;<br>
&gt; ---<br>
&gt;  meta/recipes-kernel/perf/<a href="http://perf.bb" rel="noreferrer" target="_blank">perf.bb</a> | 1 -<br>
&gt;  1 file changed, 1 deletion(-)<br>
&gt; <br>
&gt; diff --git a/meta/recipes-kernel/perf/<a href="http://perf.bb" rel="noreferrer" target="_blank">perf.bb</a> b/meta/recipes-<br>
&gt; kernel/perf/<a href="http://perf.bb" rel="noreferrer" target="_blank">perf.bb</a><br>
&gt; index 51f5597eb92..5e710f61834 100644<br>
&gt; --- a/meta/recipes-kernel/perf/<a href="http://perf.bb" rel="noreferrer" target="_blank">perf.bb</a><br>
&gt; +++ b/meta/recipes-kernel/perf/<a href="http://perf.bb" rel="noreferrer" target="_blank">perf.bb</a><br>
&gt; @@ -239,7 +239,6 @@ PACKAGE_ARCH = &quot;${MACHINE_ARCH}&quot;<br>
&gt;  PACKAGES =+ &quot;${PN}-archive ${PN}-tests ${PN}-perl ${PN}-python&quot;<br>
&gt;  <br>
&gt;  RDEPENDS_${PN} += &quot;elfutils bash&quot;<br>
&gt; -RDEPENDS_${PN}-doc += &quot;man&quot;<br>
&gt;  RDEPENDS_${PN}-archive =+ &quot;bash&quot;<br>
&gt;  RDEPENDS_${PN}-python =+ &quot;bash python python-modules ${@bb.utils.con<br>
&gt; tains(&#39;PACKAGECONFIG&#39;, &#39;audit&#39;, &#39;audit-python&#39;, &#39;&#39;, d)}&quot;<br>
&gt;  RDEPENDS_${PN}-perl =+ &quot;bash perl perl-modules&quot;<br>
<br>
We&#39;re certainly not consistent but:<br>
<br>
<a href="http://git.yoctoproject.org/cgit.cgi/poky/tree/meta/classes/manpages.bbclass#n13" rel="noreferrer" target="_blank">http://git.yoctoproject.org/cgit.cgi/poky/tree/meta/classes/manpages.bbclass#n13</a><br>
<br>
RDEPENDS_${MAN_PKG} += &quot;${@bb.utils.contains(&#39;PACKAGECONFIG&#39;, &#39;manpages&#39;, &#39;man-db&#39;, &#39;&#39;, d)}&quot;<br>
<br>
man-db is gplv2 too...<br></blockquote><div><br></div><div>Ah, well spotted, I didn&#39;t know about that one. If that&#39;s the case, I&#39;ll drop this patch and rework both it and manpages.bbclass to make the rdep conditional on license compatibility. Thanks!<br></div></div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr">Christopher Larson<br>kergoth at gmail dot com<br>Founder - BitBake, OpenEmbedded, OpenZaurus<br>Senior Software Engineer, Mentor Graphics</div></div></div></div></div>