<div dir="ltr"><div>Thanks, now it's future-proof :)</div><div><br></div><div>Alex<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 9 Sep 2019 at 16:54, Fabio Berton <<a href="mailto:fabio.berton@ossystems.com.br">fabio.berton@ossystems.com.br</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">Hi Alexander!<br>
<br>
I sent a v2 with python3 in depends and dropped the patch.<br>
<br>
Thanks!<br>
<br>
On Fri, Sep 6, 2019 at 5:33 PM Alexander Kanavin <<a href="mailto:alex.kanavin@gmail.com" target="_blank">alex.kanavin@gmail.com</a>> wrote:<br>
><br>
> On Fri, 6 Sep 2019 at 22:12, Otavio Salvador <<a href="mailto:otavio.salvador@ossystems.com.br" target="_blank">otavio.salvador@ossystems.com.br</a>> wrote:<br>
>><br>
>> > As I had previously suggested... can you try adding 'python3' to DEPENDS of mesa, and see if that helps please? I think that would allow avoiding this patch altogether.<br>
>><br>
>> Why a new build dependency if this works just fine?<br>
><br>
><br>
> 1. Less patches is always better than more patches, especially if they are non-upstreamable.<br>
><br>
> 2. The patch reverts to something that meson has deprecated and will likely remove in the future.<br>
> <a href="https://github.com/mesonbuild/meson/blob/master/docs/markdown/Python-3-module.md" rel="noreferrer" target="_blank">https://github.com/mesonbuild/meson/blob/master/docs/markdown/Python-3-module.md</a><br>
><br>
> 3. Meson's currently supported way of detecting python does in fact rely on python's target configuration being available, as it picks up settings from it. I had fixed a similar issue elsewhere recently.<br>
><br>
> Alex<br>
><br>
</blockquote></div>