Re: [OE-core] [PATCH] webkitgtk, gcr, libsecret: force ARM mode

2015-10-12 Thread Martin Jansa
On Sat, Oct 10, 2015 at 11:45:43PM +0200, Carlos Alberto Lopez Perez wrote: > On 10/10/15 17:32, Martin Jansa wrote: > > Did you enable thumb in that build? It's not enabled by default. > > > > ARM_INSTRUCTION_SET is for recipes to use arm when distro selects thumb by > > default. > > I assumed

Re: [OE-core] [PATCH] webkitgtk, gcr, libsecret: force ARM mode

2015-10-12 Thread Alexander Kanavin
On 10/11/2015 12:45 AM, Carlos Alberto Lopez Perez wrote: So, could we set ARM_INSTRUCTION_SET=arm only for armv6 or older machines in this 3 recipes? Indeed. Webkit has other issues on armv6 or older (JIT fails to build if you try to), so I believe the upstream developers test the builds

Re: [OE-core] [PATCH] webkitgtk, gcr, libsecret: force ARM mode

2015-10-12 Thread Burton, Ross
On 11 October 2015 at 10:51, Phil Blundell wrote: > > * gcr, libsecret are new since switch to gcc 5.2 > > The problem with libsecret seems to be caused by its valgrind support > code (which contains asm statements that only work in ARM state). I > wonder if you can just turn

Re: [OE-core] [PATCH] webkitgtk, gcr, libsecret: force ARM mode

2015-10-12 Thread Carlos Alberto Lopez Perez
On 12/10/15 13:17, Alexander Kanavin wrote: > On 10/11/2015 12:45 AM, Carlos Alberto Lopez Perez wrote: > >> So, could we set ARM_INSTRUCTION_SET=arm only for armv6 or older >> machines in this 3 recipes? > > Indeed. Webkit has other issues on armv6 or older (JIT fails to build if > you try to),

Re: [OE-core] [PATCH] webkitgtk, gcr, libsecret: force ARM mode

2015-10-11 Thread Phil Blundell
On Fri, 2015-10-09 at 17:11 +0200, Martin Jansa wrote: > * gcr, libsecret are new since switch to gcc 5.2 The problem with libsecret seems to be caused by its valgrind support code (which contains asm statements that only work in ARM state). I wonder if you can just turn WITH_VALGRIND off rather

Re: [OE-core] [PATCH] webkitgtk, gcr, libsecret: force ARM mode

2015-10-10 Thread Khem Raj
On Sat, Oct 10, 2015 at 5:13 AM, Carlos Alberto Lopez Perez wrote: > On 09/10/15 17:11, Martin Jansa wrote: >> * gcr, libsecret are new since switch to gcc 5.2 >> * webkitgtk is old but it was removed while migrating from >> webkit-gtk to webkitgtl (like

Re: [OE-core] [PATCH] webkitgtk, gcr, libsecret: force ARM mode

2015-10-10 Thread Martin Jansa
Did you enable thumb in that build? It's not enabled by default. ARM_INSTRUCTION_SET is for recipes to use arm when distro selects thumb by default. On Sat, Oct 10, 2015, 14:13 Carlos Alberto Lopez Perez wrote: > On 09/10/15 17:11, Martin Jansa wrote: > > * gcr, libsecret

Re: [OE-core] [PATCH] webkitgtk, gcr, libsecret: force ARM mode

2015-10-10 Thread Carlos Alberto Lopez Perez
On 10/10/15 17:32, Martin Jansa wrote: > Did you enable thumb in that build? It's not enabled by default. > > ARM_INSTRUCTION_SET is for recipes to use arm when distro selects thumb by > default. I assumed it was enabled by default if it was listed on the list printed on TUNE_FEATURES when

Re: [OE-core] [PATCH] webkitgtk, gcr, libsecret: force ARM mode

2015-10-10 Thread Khem Raj
On Sat, Oct 10, 2015 at 2:45 PM, Carlos Alberto Lopez Perez wrote: > On 10/10/15 17:32, Martin Jansa wrote: >> Did you enable thumb in that build? It's not enabled by default. >> >> ARM_INSTRUCTION_SET is for recipes to use arm when distro selects thumb by >> default. > > I

Re: [OE-core] [PATCH] webkitgtk, gcr, libsecret: force ARM mode

2015-10-10 Thread Carlos Alberto Lopez Perez
On 09/10/15 17:11, Martin Jansa wrote: > * gcr, libsecret are new since switch to gcc 5.2 > * webkitgtk is old but it was removed while migrating from > webkit-gtk to webkitgtl (like REQUIRED_DISTRO_FEATURES were) :/ > > Signed-off-by: Martin Jansa > --- > diff --git