<div dir="ltr"><div dir="ltr"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">I'll welcome a patch that makes dependency failures point to the Dockerfiles, though</blockquote><div>Seems it can be done with meson >= 0.50 - here we can dump a message if dependency failure.</div><div>But it seems to be look as not good solution (one dependency - one message).</div><div><br></div><div>As i see in the Dockerfiles listed dependency on liblzma.</div><div>So if meson-change is not preferable in that case - so at least readme-file has to point us about dependencies.</div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Feb 26, 2019 at 12:21 PM Petri Latvala <<a href="mailto:petri.latvala@intel.com">petri.latvala@intel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Tue, Feb 26, 2019 at 12:03:07PM +0200, Sergii Romantsov wrote:<br>
> Hello, Petri.<br>
> <br>
> <br>
> > As I understood you, just installing liblzma-dev makes the build<br>
> > successful, right?<br>
> <br>
> <br>
> If to be more specific, installing liblzma-dev makes meson-configuration<br>
> successful.<br>
> <br>
> Making this change just lead to better error messages when it's not<br>
> > installed?<br>
> <br>
> <br>
> Yes, it will show which package exactly leaked instead of unclear error.<br>
> <br>
> What is the distro you're using?<br>
> <br>
> <br>
> 4.15.0-45-generic #48~16.04.1-Ubuntu SMP Tue Jan 29 18:03:48 UTC 2019<br>
> x86_64 x86_64 x86_64 GNU/Linux<br>
> <br>
> I think it happens due to specific of meson-dependency - how it looks for a<br>
> package.<br>
> So command (when liblzma-dev is absent, but liblzma5 is installed)<br>
>  'pkg-config --cflags --libs libdw'<br>
> gives:<br>
> "Package liblzma was not found in the pkg-config search path.<br>
> Perhaps you should add the directory containing `liblzma.pc'<br>
> to the PKG_CONFIG_PATH environment variable<br>
> Package 'liblzma', required by 'libdw', not found"<br>
> <br>
> So in my understanding its just because of lack of liblzma.pc<br>
<br>
<br>
Indeed. I checked Ubuntu 16.04's libdw-dev and it:<br>
<br>
1) Has liblzma in Requires.private in libdw.pc<br>
2) Does not depend on liblzma-dev<br>
<br>
I don't see the bug reported, and I suspect it won't be fixed, with<br>
16.04's support ending this year for the desktop version...<br>
<br>
<br>
However, on the patch, I'll have to nak the meson.build change. Having<br>
better error messages is not a good reason for useless dependency<br>
declarations. I'll welcome a patch that makes dependency failures<br>
point to the Dockerfiles, though.<br>
<br>
<br>
-- <br>
Petri Latvala<br>
</blockquote></div>