Re: [OE-core] [PATCH] x264: add textrel to INSANE_SKIP

2016-09-26 Thread sujith h
On Mon, Sep 26, 2016 at 11:39 AM, sujith h wrote: > Hi, > > Let me know if this patch is ok. Else I can help make this patch better. > oops, sorry for my earlier mail. This patch is merged :

[OE-core] [PATCH] x264: add textrel to INSANE_SKIP

2016-09-21 Thread Sujith H
From: Fahad Usman Tried by adding CFLAGS_append = " -fpic " to the recipe. But that couldn't help resolve the warning message: x264/r2491+gitAUTOINC+c8a773ebfc-r0/packages-split/x264/usr/lib/libx264.so.144' has relocations in .text [textrel] It was found that this

Re: [OE-core] [PATCH] x264: add textrel to INSANE_SKIP to skip the warning

2016-09-21 Thread Phil Blundell
On Wed, 2016-09-21 at 13:44 +0530, sujith h wrote: > > The above mentioned functions are part of .S files. OK.  In that case, please just mention in the commit message that you can't easily get rid of the TEXTRELs because they are coming from assembler source files that would (I assume) need

Re: [OE-core] [PATCH] x264: add textrel to INSANE_SKIP to skip the warning

2016-09-21 Thread sujith h
On Thu, Sep 15, 2016 at 3:37 PM, Phil Blundell wrote: > On Thu, 2016-09-15 at 12:22 +0530, Sujith H wrote: > > From: Fahad Usman > > > > This change help fix the warning message: > > > > x264/r2491+gitAUTOINC+c8a773ebfc-r0/packages- > >

Re: [OE-core] [PATCH] x264: add textrel to INSANE_SKIP to skip the warning

2016-09-15 Thread Phil Blundell
On Thu, 2016-09-15 at 12:22 +0530, Sujith H wrote: > From: Fahad Usman > > This change help fix the warning message: > > x264/r2491+gitAUTOINC+c8a773ebfc-r0/packages- > split/x264/usr/lib/libx264.so.144' has relocations in .text [textrel] > > Adding textrel to

Re: [OE-core] [PATCH] x264: add textrel to INSANE_SKIP for cyclone5 machine

2016-09-15 Thread sujith h
On Thu, Sep 15, 2016 at 12:15 PM, sujith h wrote: > > > On Thu, Sep 15, 2016 at 10:13 AM, sujith h wrote: > >> >> >> On Thu, Sep 15, 2016 at 4:41 AM, Burton, Ross >> wrote: >> >>> >>> On 12 September 2016 at 07:13, Sujith H

[OE-core] [PATCH] x264: add textrel to INSANE_SKIP to skip the warning

2016-09-15 Thread Sujith H
From: Fahad Usman This change help fix the warning message: x264/r2491+gitAUTOINC+c8a773ebfc-r0/packages-split/x264/usr/lib/libx264.so.144' has relocations in .text [textrel] Adding textrel to INSANE_SKIP resolves this issue. This issue was observed in cyclone5 and

Re: [OE-core] [PATCH] x264: add textrel to INSANE_SKIP for cyclone5 machine

2016-09-15 Thread sujith h
On Thu, Sep 15, 2016 at 10:13 AM, sujith h wrote: > > > On Thu, Sep 15, 2016 at 4:41 AM, Burton, Ross > wrote: > >> >> On 12 September 2016 at 07:13, Sujith H wrote: >> >>> +# PIC can't be enabled for 32-bit x86 and cyclone5 >>>

Re: [OE-core] [PATCH] x264: add textrel to INSANE_SKIP for cyclone5 machine

2016-09-14 Thread sujith h
On Thu, Sep 15, 2016 at 4:41 AM, Burton, Ross wrote: > > On 12 September 2016 at 07:13, Sujith H wrote: > >> +# PIC can't be enabled for 32-bit x86 and cyclone5 >> INSANE_SKIP_${PN}_append_x86 = " textrel" >> +INSANE_SKIP_${PN}_append_cyclone5 = "

Re: [OE-core] [PATCH] x264: add textrel to INSANE_SKIP for cyclone5 machine

2016-09-14 Thread Burton, Ross
On 12 September 2016 at 07:13, Sujith H wrote: > +# PIC can't be enabled for 32-bit x86 and cyclone5 > INSANE_SKIP_${PN}_append_x86 = " textrel" > +INSANE_SKIP_${PN}_append_cyclone5 = " textrel" > Can this be generalised or is it absolutely specific to cyclone5? If it

[OE-core] [PATCH] x264: add textrel to INSANE_SKIP for cyclone5 machine

2016-09-12 Thread Sujith H
From: Fahad Usman This change help fix the warning message: x264/r2491+gitAUTOINC+c8a773ebfc-r0/packages-split/x264/usr/lib/libx264.so.144' has relocations in .text [textrel] Adding textrel to INSANE_SKIP resolves this issue. Signed-off-by: Fahad Usman