From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp3-g21.free.fr (smtp3-g21.free.fr [212.27.42.3]) by yocto-www.yoctoproject.org (Postfix) with ESMTP id 32779E01489 for ; Fri, 31 May 2013 13:06:22 -0700 (PDT) Received: from e6520eb (tal33-3-82-233-81-124.fbx.proxad.net [82.233.81.124]) (Authenticated sender: eukrea) by smtp3-g21.free.fr (Postfix) with ESMTPSA id 9FB84A61EF; Fri, 31 May 2013 22:06:15 +0200 (CEST) Date: Fri, 31 May 2013 22:06:13 +0200 From: Eric =?UTF-8?B?QsOpbmFyZA==?= To: Otavio Salvador Message-ID: <20130531220613.2fa9b912@e6520eb> In-Reply-To: References: <1369850680-18324-1-git-send-email-eric@eukrea.com> <20130531161556.5c06374b@e6520eb> <20130531170240.3a1c27f5@e6520eb> <20130531173524.137895c4@e6520eb> Organization: =?UTF-8?B?RXVrcsOpYQ==?= Electromatique X-Mailer: Claws Mail 3.9.0 (GTK+ 2.24.18; x86_64-redhat-linux-gnu) Mime-Version: 1.0 Cc: "meta-freescale@yoctoproject.org" Subject: Re: [meta-fsl-arm][PATCH] qt5: add mx5 and mx6 support X-BeenThere: meta-freescale@yoctoproject.org X-Mailman-Version: 2.1.13 Precedence: list List-Id: Usage and development list for the meta-fsl-* layers List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 31 May 2013 20:06:24 -0000 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Le Fri, 31 May 2013 13:23:29 -0300, Otavio Salvador a =C3=A9crit : > On Fri, May 31, 2013 at 12:35 PM, Eric B=C3=A9nard wrot= e: >=20 > > Le Fri, 31 May 2013 12:04:11 -0300, > > Otavio Salvador a =C3=A9crit : > > > > > On Fri, May 31, 2013 at 12:02 PM, Eric B=C3=A9nard = wrote: > > > > > > > Le Fri, 31 May 2013 12:00:08 -0300, > > > > Otavio Salvador a =C3=A9crit : > > > > > > > > > On Fri, May 31, 2013 at 11:15 AM, Eric B=C3=A9nard > > wrote: > > > > > > > > > > > Le Fri, 31 May 2013 10:18:44 -0300, > > > > > > Otavio Salvador a =C3=A9crit : > > > > > > > > > > > > > On Wed, May 29, 2013 at 3:04 PM, Eric B=C3=A9nard > > > > wrote: > > > > > > > > > > > > > > > - this allow to build qt5 with OpenGL ES support for i.MX5 = and > > > > i.MX6 > > > > > > > > - tested on i.MX51, i.MX53 and i.MX6Q > > > > > > > > > > > > > > > > Signed-off-by: Eric B=C3=A9nard > > > > > > > > > > > > > > > > > > > > > > ... > > > > > > > > > > > > > > > > > > > > > > recipes-qt/qt5/qtbase/mx5/qeglfshooks_imx5.cpp | 105 > > > > > > > > ++++++++++++++++++++++++ > > > > > > > > recipes-qt/qt5/qtbase_5.0.2.bbappend | 68 > > > > +++++++++++++++ > > > > > > > > > > > > > > > > > > > > > > ... > > > > > > > > > > > > > > Thinking more about it, I think we should put these inside a > > meta-qt5 > > > > > > > directory so we know what will be 'included'. Otherwse we may > > need to > > > > > > pick > > > > > > > every file depending on each layer and it might be difficult = to > > > > > > understand > > > > > > > what is in use and what is not. > > > > > > > > > > > > > please apply as is and create a patch on top of it to achieve t= he > > > > > > organization you prefer. > > > > > > > > > > > > > > > > No reason to apply one patch which we know that needs rework. > > > > > > > > > > Do you agree with my argument? > > > > > > > > > no ;-) > > > > > > > > > > Well; it would be easier if you could explain why. Do you mind to > > elaborate > > > it a little more? > > > > > if the argument is "No reason to apply one patch which we know that > > needs rework" : > > Changing the organization of the bbappend is not just a mater of editing > > the patch in a few minutes, it also means testing with and without > > meta-qt5 and that's very long (and I already did that for v1 when we > > discussed that initially). > > 2 days ago you told me that the RFC was fine, now you tell me that > > needs rework and as I don't have immediate time to rework it so if you > > want to change the organization or use Chris' way to add layers (which > > is very elegant) either create a patch to rework this one or rework the > > patch before applying or drop it for the moment and I may work again > > on that later (and in that case, please reply with the organization > > you want so that the work is done only one time). > > >=20 > 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. Otavio sometimes you push the pedantry very far : the difference between the initial patch and what you call the "proper patch" is that the "proper patch" doesn't have 3 letters (RFC) in the subject and now I understand that having "RFC" in the subject means you didn't review the patch. > > if the argument is "it might be difficult to understand what is in use > > and what is not" : > > as soon as you have automatic addition of sublayers you fall in a case > > where it's not easy to understand what is in use or not whatever is > > your directory organization. > > >=20 > I'd put it as: >=20 > recipes-... > meta-qt5/recipes-qt/qt5/... >=20 > Or as Chris suggested: >=20 > recipes-... > qt5-layer/recipes-qt/qt5 >=20 > Both ways looks good for me. >=20 please choose ONE way so that the next patch iteration get THE right one for you. > I will mark them as 'Changes requested' in patchwork. >=20 perfect. Eric