[PATCH] netbsd: Replace nonstandard __WORDSIZE with a more portable solution

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

On 14.12.2015 15:46, David Henningsson wrote:
> 
> 
> On 2015-12-10 06:23, Kamil Rytarowski wrote:
>> There is no way to check CPU type in a portable way across ABIs. 
>> Checking for sizeof(void*) is reasonable since most platforms
>> will report correct values. One exception is x32, but since it's
>> halfbaked never finished and almost not needed any more - we can
>> ignore it.
>> 
>> The check is needed only to print a debug message, no functional 
>> change.
> 
> There is also a reference to __WORDSIZE in
> src/pulsecore/sample-util.h.
> 

Right, I missed it.

>> This change fixes build on NetBSD.
> 
> What error are you getting, and how come it does not affect 
> src/pulsecore/sample-util.h as well?
> 

The error is about unknown symbol.

This is due to being on a pile of local patches to get it build. I'm
extracting single changes and upstreaming one after another.


I'm heading for version 2 of this patch.

Thanks!
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2

iQIcBAEBCAAGBQJWb1R9AAoJEEuzCOmwLnZswS4P/24Z70HKmskq/CNJ9FGolXDh
ljOPsOIxOdU3f+992gGx9pGuq8RAFtqAWZf2U3mgb0nXMrA18Jfl4TPUW/+g9o1a
zPnR78F1uXNO++GQIUPOrbk9UkP/j82quOmmS9ri7dsQCpK2N/ToCel9QT1etNqs
zGw6EBKLt7zN3s8Dgjj1VYlBpbuqNj/1cB5DTfvqqOB6c9LsU/fnjOFUm6zkHCZ0
d00SlOMzDoQ1xaddik0+SBlg/L+rfAVmNyxWuo9WEE8zo4bF02FbCk+MKi/o1qSn
uPm3L2Grp0vBv1+JDYpznInOClW5Vsm/TWV+o6mHN+NB/mCtIAW9V8gV+xwwbysr
uDZ4HqpQnQ3Mipj2Upc/TIIEXwOArP3gfUTdGqAooNhhpG26HoVeOZiiBC5bkcNk
Rnl07iuYcHuYWRWa64YFi9OC+D2M9RgSZMWlhvN46kJAwTxBlvPZlgSKNtQbzRk/
0MJVVq+A5Nccl/9SCJrL4ZUiKon1z5iVtfqmBsGOaWstDOHHW1mB/R1dX5umKsWM
1lqE2m0JynRuSabKRKqnNknrT6oTbHsVlQb/6gptME8Sd1LFyAZ9zUE2t3OT5kGV
KAXmash4sKAL3MspILhuIFcSTxG08DMmu73jo7zOUeDgiC5bOLZDaDJJo18629AY
wDyH1jeKVDU142j55uVx
=LhdQ
-----END PGP SIGNATURE-----


[Index of Archives]     [Linux Audio Users]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux