Re: [OE-core] [PATCH] x264: add textrel to INSANE_SKIP for cyclone5 machine
On Thu, Sep 15, 2016 at 12:15 PM, sujith hwrote: > > > 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 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 >>> can't be generalised then I feel that this should belong in the cyclone5 >>> BSP. >>> >> >> We found this with cyclone5 BSP. I will try to figure out how if we can >> see it with qemuarm or other BSP's too. >> > > I tried with qemuarm, it wasn't reproducible. But with imx6qsabresd from > meta-fsl-arm, it was reproducible. So I believe we can generalize this > patch. I will send the updated one shortly. Thanks for the pointer Ross. > I forgot to mention that I posted this change in meta-altera for review. I got the feedback from meta-altera to post the change for review here: https://github.com/kraj/meta-altera/pull/37. Sharing this for the reference. > >> >>> >>> Ross >>> >> >> >> >> -- >> സുജിത് ഹരിദാസന് >> Bangalore >> Contributor to KDE project >> Contributor to Yocto project >> http://fci.wikia.com/wiki/Anti-DRM-Campaign >> http://sujithh.info >> C-x C-c >> > > > > -- > സുജിത് ഹരിദാസന് > Bangalore > Contributor to KDE project > Contributor to Yocto project > http://fci.wikia.com/wiki/Anti-DRM-Campaign > http://sujithh.info > C-x C-c > -- സുജിത് ഹരിദാസന് Bangalore Contributor to KDE project Contributor to Yocto project http://fci.wikia.com/wiki/Anti-DRM-Campaign http://sujithh.info C-x C-c -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH] x264: add textrel to INSANE_SKIP for cyclone5 machine
On Thu, Sep 15, 2016 at 10:13 AM, sujith hwrote: > > > 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 = " textrel" >>> >> >> Can this be generalised or is it absolutely specific to cyclone5? If it >> can't be generalised then I feel that this should belong in the cyclone5 >> BSP. >> > > We found this with cyclone5 BSP. I will try to figure out how if we can > see it with qemuarm or other BSP's too. > I tried with qemuarm, it wasn't reproducible. But with imx6qsabresd from meta-fsl-arm, it was reproducible. So I believe we can generalize this patch. I will send the updated one shortly. Thanks for the pointer Ross. > >> >> Ross >> > > > > -- > സുജിത് ഹരിദാസന് > Bangalore > Contributor to KDE project > Contributor to Yocto project > http://fci.wikia.com/wiki/Anti-DRM-Campaign > http://sujithh.info > C-x C-c > -- സുജിത് ഹരിദാസന് Bangalore Contributor to KDE project Contributor to Yocto project http://fci.wikia.com/wiki/Anti-DRM-Campaign http://sujithh.info C-x C-c -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH] x264: add textrel to INSANE_SKIP for cyclone5 machine
On Thu, Sep 15, 2016 at 4:41 AM, Burton, Rosswrote: > > 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 > can't be generalised then I feel that this should belong in the cyclone5 > BSP. > We found this with cyclone5 BSP. I will try to figure out how if we can see it with qemuarm or other BSP's too. > > Ross > -- സുജിത് ഹരിദാസന് Bangalore Contributor to KDE project Contributor to Yocto project http://fci.wikia.com/wiki/Anti-DRM-Campaign http://sujithh.info C-x C-c -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
Re: [OE-core] [PATCH] x264: add textrel to INSANE_SKIP for cyclone5 machine
On 12 September 2016 at 07:13, Sujith Hwrote: > +# 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 can't be generalised then I feel that this should belong in the cyclone5 BSP. Ross -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
[OE-core] [PATCH] x264: add textrel to INSANE_SKIP for cyclone5 machine
From: Fahad UsmanThis 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 --- meta/recipes-multimedia/x264/x264_git.bb | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/meta/recipes-multimedia/x264/x264_git.bb b/meta/recipes-multimedia/x264/x264_git.bb index fa44f9c..379961a 100644 --- a/meta/recipes-multimedia/x264/x264_git.bb +++ b/meta/recipes-multimedia/x264/x264_git.bb @@ -48,6 +48,7 @@ do_install() { oe_runmake install DESTDIR=${D} } -# PIC can't be enabled for 32-bit x86 +# PIC can't be enabled for 32-bit x86 and cyclone5 INSANE_SKIP_${PN}_append_x86 = " textrel" +INSANE_SKIP_${PN}_append_cyclone5 = " textrel" -- 1.9.1 -- ___ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core