powerpc64 workaround

powerpc64 workaround

[Breno Leitao]

[-- Attachment #1.1: Type: text/plain, Size: 592 bytes --]

Hello Takashi,

Fourteen years ago you commited a patch in alsa-libs entitled "added
workaround for ppc64." and I am curious about it.  What is this
workaround for? Is this still needed?

Currently it seems to be causing some issues on ppc64le (ABI v2) when
building this package, and we are reverting it downstream in order to
avoid FTBFS on alsa-libs: 

https://github.com/alpinelinux/aports/pull/911

https://github.com/buildroot/buildroot/blob/master/package/alsa-lib/0005-remove-powerpc64-workaround.patch

Anyway, do you think it is still important?

Thank you,
Breno

[-- Attachment #1.2: Type: application/pgp-signature, Size: 819 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]

Re: powerpc64 workaround

[Takashi Iwai]

On Mon, 20 Feb 2017 20:32:35 +0100,
Breno Leitao wrote:
> 
> Hello Takashi,
> 
> Fourteen years ago you commited a patch in alsa-libs entitled "added
> workaround for ppc64." and I am curious about it.  What is this
> workaround for? Is this still needed?

It's a very old kludge, and I don't think it's needed for the recent
glibc.

> Currently it seems to be causing some issues on ppc64le (ABI v2) when
> building this package, and we are reverting it downstream in order to
> avoid FTBFS on alsa-libs: 
> 
> https://github.com/alpinelinux/aports/pull/911
> 
> https://github.com/buildroot/buildroot/blob/master/package/alsa-lib/0005-remove-powerpc64-workaround.patch
> 
> Anyway, do you think it is still important?

Care to submit the patch to alsa-devel ML instead of link?
Then I can merge the fix.


thanks,

Takashi

Re: powerpc64 workaround

[Breno Leitao]

[-- Attachment #1.1: Type: text/plain, Size: 627 bytes --]

Hi Takashi,

On Mon, Feb 20, 2017 at 10:15:15PM +0100, Takashi Iwai wrote:
> > Currently it seems to be causing some issues on ppc64le (ABI v2) when
> > building this package, and we are reverting it downstream in order to
> > avoid FTBFS on alsa-libs: 
> > 
> > https://github.com/alpinelinux/aports/pull/911
> > 
> > https://github.com/buildroot/buildroot/blob/master/package/alsa-lib/0005-remove-powerpc64-workaround.patch
> > 
> > Anyway, do you think it is still important?
> 
> Care to submit the patch to alsa-devel ML instead of link?
> Then I can merge the fix.

Cool. I will send it soon. Thank you!

[-- Attachment #1.2: Type: application/pgp-signature, Size: 819 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]