<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, May 31, 2013 at 12:35 PM, Eric Bénard <span dir="ltr"><<a href="mailto:eric@eukrea.com" target="_blank">eric@eukrea.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Le Fri, 31 May 2013 12:04:11 -0300,<br>
<div><div class="h5">Otavio Salvador <<a href="mailto:otavio@ossystems.com.br">otavio@ossystems.com.br</a>> a écrit :<br>
<br>
> On Fri, May 31, 2013 at 12:02 PM, Eric Bénard <<a href="mailto:eric@eukrea.com">eric@eukrea.com</a>> wrote:<br>
><br>
> > Le Fri, 31 May 2013 12:00:08 -0300,<br>
> > Otavio Salvador <<a href="mailto:otavio@ossystems.com.br">otavio@ossystems.com.br</a>> a écrit :<br>
> ><br>
> > > On Fri, May 31, 2013 at 11:15 AM, Eric Bénard <<a href="mailto:eric@eukrea.com">eric@eukrea.com</a>> wrote:<br>
> > ><br>
> > > > Le Fri, 31 May 2013 10:18:44 -0300,<br>
> > > > Otavio Salvador <<a href="mailto:otavio@ossystems.com.br">otavio@ossystems.com.br</a>> a écrit :<br>
> > > ><br>
> > > > > On Wed, May 29, 2013 at 3:04 PM, Eric Bénard <<a href="mailto:eric@eukrea.com">eric@eukrea.com</a>><br>
> > wrote:<br>
> > > > ><br>
> > > > > > - this allow to build qt5 with OpenGL ES support for i.MX5 and<br>
> > i.MX6<br>
> > > > > > - tested on i.MX51, i.MX53 and i.MX6Q<br>
> > > > > ><br>
> > > > > > Signed-off-by: Eric Bénard <<a href="mailto:eric@eukrea.com">eric@eukrea.com</a>><br>
> > > > > ><br>
> > > > ><br>
> > > > > ...<br>
> > > > ><br>
> > > > ><br>
> > > > > >  recipes-qt/qt5/qtbase/mx5/qeglfshooks_imx5.cpp |  105<br>
> > > > > > ++++++++++++++++++++++++<br>
> > > > > >  recipes-qt/qt5/qtbase_5.0.2.bbappend           |   68<br>
> > +++++++++++++++<br>
> > > > > ><br>
> > > > ><br>
> > > > > ...<br>
> > > > ><br>
> > > > > Thinking more about it, I think we should put these inside a meta-qt5<br>
> > > > > directory so we know what will be 'included'. Otherwse we may need to<br>
> > > > pick<br>
> > > > > every file depending on each layer and it might be difficult to<br>
> > > > understand<br>
> > > > > what is in use and what is not.<br>
> > > > ><br>
> > > > please apply as is and create a patch on top of it to achieve the<br>
> > > > organization you prefer.<br>
> > > ><br>
> > ><br>
> > > No reason to apply one patch which we know that needs rework.<br>
> > ><br>
> > > Do you agree with my argument?<br>
> > ><br>
> > no ;-)<br>
> ><br>
><br>
> Well; it would be easier if you could explain why. Do you mind to elaborate<br>
> it a little more?<br>
><br>
</div></div>if the argument is "No reason to apply one patch which we know that<br>
needs rework" :<br>
Changing the organization of the bbappend is not just a mater of editing<br>
the patch in a few minutes, it also means testing with and without<br>
meta-qt5 and that's very long (and I already did that for v1 when we<br>
discussed that initially).<br>
2 days ago you told me that the RFC was fine, now you tell me that<br>
needs rework and as I don't have immediate time to rework it so if you<br>
want to change the organization or use Chris' way to add layers (which<br>
is very elegant) either create a patch to rework this one or rework the<br>
patch before applying or drop it for the moment and I may work again<br>
on that later (and in that case, please reply with the organization<br>
you want so that the work is done only one time).<br></blockquote><div><br></div><div style>I asked you to send it as proper patch for review. I didn't noticed you have put the bbappend in same level as the other so when I noticed it I commented.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
if the argument is "it might be difficult to understand what is in use<br>
and what is not" :<br>
as soon as you have automatic addition of sublayers you fall in a case<br>
where it's not easy to understand what is in use or not whatever is<br>
your directory organization.<br></blockquote><div><br></div><div style>I'd put it as:</div><div style><br></div><div style>recipes-...</div><div style>meta-qt5/recipes-qt/qt5/...</div><div style><br></div><div style>
Or as Chris suggested:</div><div style><br></div><div style>recipes-...</div><div style>qt5-layer/recipes-qt/qt5</div><div style><br></div><div style>Both ways looks good for me.</div><div style><br></div><div style>I will mark them as 'Changes requested' in patchwork.</div>
<div style><br></div><div style>Regards,</div><div style><br></div></div><div><br></div>-- <br>Otavio Salvador                             O.S. Systems<br><a href="http://www.ossystems.com.br" target="_blank">http://www.ossystems.com.br</a>        <a href="http://projetos.ossystems.com.br" target="_blank">http://projetos.ossystems.com.br</a><br>
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750<br>
</div></div>