<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Apr 8, 2014 at 10:16 AM, Fathi Boudra <span dir="ltr"><<a href="mailto:fathi.boudra@linaro.org" target="_blank">fathi.boudra@linaro.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="">On 8 April 2014 20:05, Khem Raj <<a href="mailto:raj.khem@gmail.com">raj.khem@gmail.com</a>> wrote:<br>
> On Tue, Apr 8, 2014 at 7:00 AM, Fathi Boudra <<a href="mailto:fathi.boudra@linaro.org">fathi.boudra@linaro.org</a>> wrote:<br>
>> do_install() {<br>
>> - oe_runmake prefix="${prefix}" DESTDIR="${D}" install<br>
>> + ${MAKE} NO_PYTHON=1 prefix=${prefix} plugin_dir=${libdir}/trace-cmd/plugins DESTDIR=${D} install<br>
>> }<br>
><br>
> can you explain/document this change<br>
<br>
</div>sure. trace-cmd doesn't have a build system generated Makefile. It's<br>
hand written.<br>
To do the install properly, one has to pass prefix/plugin_dir.<br>
In addition, the previous command in the recipe doesn't do the right<br>
thing anymore (you'll end with duplicated prefix /usr/usr/).</blockquote></div><br>You don't need to avoid the use of oe_runmake to pass variables in, that's what EXTRA_OEMAKE is for :) See <a href="https://github.com/MentorEmbedded/meta-mentor/commit/4639877">https://github.com/MentorEmbedded/meta-mentor/commit/4639877</a> for a version of this fix that uses it instead.<br>
-- <br>Christopher Larson<br>clarson at kergoth dot com<br>Founder - BitBake, OpenEmbedded, OpenZaurus<br>Maintainer - Tslib<br>Senior Software Engineer, Mentor Graphics
</div></div>