> -----Original Message----- > From: [email protected] > [mailto:[email protected]] On Behalf Of > Khem Raj > Sent: den 19 maj 2018 06:53 > To: Trevor Woerner <[email protected]>; Andre McCurdy > <[email protected]> > Cc: openembeded-devel <[email protected]> > Subject: Re: [oe] [meta-oe][PATCH 01/17] libwebsockets: rely on > cmake.bbclass to put cmake package files in dev package > > On 5/18/18 7:37 PM, Trevor Woerner wrote: > > On Fri, May 18, 2018 at 6:28 PM, Andre McCurdy <[email protected] > > <mailto:[email protected]>> wrote: > > > > On Fri, May 18, 2018 at 2:01 PM, Khem Raj <[email protected] > > <mailto:[email protected]>> wrote: > > > On Fri, May 18, 2018 at 12:07 PM, Andre McCurdy > <[email protected] <mailto:[email protected]>> wrote: > > >> On Thu, May 17, 2018 at 6:56 PM, Khem Raj <[email protected] > <mailto:[email protected]>> wrote: > > >>> Hi Andre > > >>> > > >>> This series is fine, except that it has some formatting > changes mixed with > > >>> cmake related changes. It would be preferable to have them as > separate > > >>> pulls. I reckon that both types of changes are good. > > >> > > >> The formatting cleanups are all minor and probably not worth > > >> submitting on their own. > > >> > > > > > > OK thats fine, drop then in v2 > > > > They are not worth reworking the patch set to create a v2 either > :-) > > > > > > It would be wonderful if we could agree on formatting; if we could, > I'd > > certainly help by taking on some of the reformatting task. > > > > Unfortunately, all such past attempts of which I'm aware never got > > anywhere, and if I remember correctly, the Yocto folks and the OE > folks > > each have their own ideas of what is "correct". > > > > In any case, if these won't get in as is, if Andre doesn't want to > spin > > a v2 without the formatting changes, and everyone is okay with it, I > > could spin a v2 without the formatting changes. > > Please do.
If you do, I think you should spin the patches for the clean ups as well. At least I agree with (most) of them, and on a whole they definitely improve the recipes. And I also agree with Andre, that in general it is too much trouble to generate clean up patches in their own, which is why it might be better to allow some clean ups together with actual changes. Otherwise they will not happen, which in the long run deteriorate the overall state of the recipes. //Peter -- _______________________________________________ Openembedded-devel mailing list [email protected] http://lists.openembedded.org/mailman/listinfo/openembedded-devel
