[Beignet] wrong CMAKE_LIBRARY_ARCHITECTURE on x86_64

[Zhigang Gong]

Agreed.

Igor, this is all for multiarch support which means installing both 32 bit and 64 bit beignet libraries on the same 64 bit system,
so we need to give different icd name for the two libraries and need to install them to different system directories. Now beignet
relies on CMAKE to provide those predefined variables, and it works fine with Debian and Ubuntu but seems broken on Fedora.

If you haven't planned to support multiarch on Fedora, then the 1.0.2 version should just work fine. And You can apply my patch
to fix the cosmetic icd file name issue.

Thanks,
Zhigang Gong.

> -----Original Message-----
> From: Beignet [mailto:beignet-bounces at lists.freedesktop.org] On Behalf Of
> Rebecca N. Palmer
> Sent: Tuesday, March 17, 2015 8:33 PM
> To: beignet at lists.freedesktop.org; i.gnatenko.brain at gmail.com
> Subject: Re: [Beignet] wrong CMAKE_LIBRARY_ARCHITECTURE on x86_64
> 
> That the file gets named intel-beignet-.icd rather than intel-beignet.icd is a
> purely cosmetic issue: it should work either way, but won't support multiarch,
> as the names of .icd files need to be distinct for co-installability but are
> otherwise irrelevant.
> 
> If you want working multiarch, you'll need to actually set
> CMAKE_LIBRARY_ARCHITECTURE (or with the patch,
> BEIGNET_LIBRARY_ARCHITECTURE), e.g. with cmake's -D option; again it
> doesn't matter what you set it to, as long as the values are distinct.
> 
> _______________________________________________
> Beignet mailing list
> Beignet at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/beignet