<div dir="auto"><div><br><div class="gmail_extra"><br><div class="gmail_quote">Em 17 de out de 2017 6:21 AM, "Fabien Lahoudere" <<a href="mailto:fabien.lahoudere@collabora.co.uk">fabien.lahoudere@collabora.co.uk</a>> escreveu:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="elided-text"><div>On Mon, 2017-10-16 at 21:34 +0100, Burton, Ross wrote:</div><blockquote type="cite"><div dir="ltr">On 16 October 2017 at 18:30, Otavio Salvador <span dir="ltr"><<a href="mailto:otavio.salvador@ossystems.com.br" target="_blank">otavio.salvador@ossystems.<wbr>com.br</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote type="cite"><div class="m_6092671508867837162HOEnZb"><div class="m_6092671508867837162h5"><br>
> + file://weston-gl-renderer-Set-<wbr>pitch-correctly-for-subsampled<wbr>-textures.patch \<br>
> + file://fix-missing-header.patc<wbr>h \<br>
<br>
</div></div>Those should be documented on commit log<br>
<div class="m_6092671508867837162HOEnZb"><div class="m_6092671508867837162h5"><br></div></div><br></blockquote><div><br></div><div>Agreed. The missing header one should say what platform causes it to be a problem (in case it never gets upstreamed and someone tests it again on just glibc). And is the pitch patch sufficiently broken that we need to backport it?</div></div></div></div></blockquote><div><br></div></div><div>Without the pitch patch, all applications generating YUV420 will not be displayed correctly. For example "gstlaunch1.0 testvideosrc ! waylandsink" fail.</div><div>I can keep this patch in a bbappend if you don't want it. Just tell me.</div></div></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">Please keep the patch. It fixes a issue and it is not BSP specific, so it belongs to core. Just explain it on commit log.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div><br></div><div>For the header patch, I will improve commit message.</div><div><br></div><div>I see there is an OVERRIDE libc-musl, should I use this patch in SRC_URI_libc-musl?</div></div></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">No, just mention it on commit log...</div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div><br></div><blockquote type="cite"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><br></div><div>Ross </div></div></div></div><font color="#888888">
</font></blockquote><font color="#888888"><div><span><pre>-- <br></pre><div style="width:100ch">Fabien</div></span></div></font></div><br>--<br>
______________________________<wbr>_________________<br>
Openembedded-core mailing list<br>
<a href="mailto:Openembedded-core@lists.openembedded.org">Openembedded-core@lists.<wbr>openembedded.org</a><br>
<a href="http://lists.openembedded.org/mailman/listinfo/openembedded-core" rel="noreferrer" target="_blank">http://lists.openembedded.org/<wbr>mailman/listinfo/openembedded-<wbr>core</a><br>
<br></blockquote></div><br></div></div></div>