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:[email protected]] On Behalf Of
> Rebecca N. Palmer
> Sent: Tuesday, March 17, 2015 8:33 PM
> To: [email protected]; [email protected]
> 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
> [email protected]
> http://lists.freedesktop.org/mailman/listinfo/beignet

_______________________________________________
Beignet mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/beignet

Reply via email to