[issue28017] bluetooth.h on big endian needs GNU C extensions

2016-09-09 Thread Martin Panter
Martin Panter added the comment: For the record (and my curiosity) I think the header file was fixed in 2014: https://git.kernel.org/cgit/bluetooth/bluez.git/commit?id=cf52a40 -- ___ Python tracker

[issue28017] bluetooth.h on big endian needs GNU C extensions

2016-09-09 Thread Roundup Robot
Roundup Robot added the comment: New changeset 91017e2202ae by Benjamin Peterson in branch 'default': compile with -std=c99 instead of -std=gnu99; use kiddie-gloves with bluetooth/bluetooh.h (#28017) https://hg.python.org/cpython/rev/91017e2202ae --

[issue28017] bluetooth.h on big endian needs GNU C extensions

2016-09-08 Thread Benjamin Peterson
Benjamin Peterson added the comment: stc99 means "C99 + GCC extensions that aren't incompatible with the standard". gnuc99 means "sort of C99 with whatever GNU weirdness" On Thu, Sep 8, 2016, at 11:25, Christian Heimes wrote: > > Christian Heimes added the comment: > > I checked my local

[issue28017] bluetooth.h on big endian needs GNU C extensions

2016-09-08 Thread Christian Heimes
Christian Heimes added the comment: I checked my local bluetooth.h. It looks like it won't compile on big endian platforms either. Python uses some GCC features like __attribute__. Do they all work in C99 mode? -- ___ Python tracker

[issue28017] bluetooth.h on big endian needs GNU C extensions

2016-09-08 Thread Benjamin Peterson
Benjamin Peterson added the comment: Hmm, I prefer to compile with stdc99. I wonder if we should consider this (presumably quite old) bluetooth.h to not exist if it doesn't work with c99. On Thu, Sep 8, 2016, at 02:49, Christian Heimes wrote: > > Christian Heimes added the comment: > > It

[issue28017] bluetooth.h on big endian needs GNU C extensions

2016-09-08 Thread Christian Heimes
Christian Heimes added the comment: It compiles now. Sorry for the duplicate ticket. -- resolution: -> fixed stage: needs patch -> resolved status: open -> closed ___ Python tracker

[issue28017] bluetooth.h on big endian needs GNU C extensions

2016-09-08 Thread Christian Heimes
Christian Heimes added the comment: Let's give it a try. -- ___ Python tracker ___ ___ Python-bugs-list

[issue28017] bluetooth.h on big endian needs GNU C extensions

2016-09-08 Thread Roundup Robot
Roundup Robot added the comment: New changeset b5b2bb56d303 by Christian Heimes in branch 'default': Issue 28017: Use -std=gnu99 to get C99 with GNU extensions for bluetooth.h on big endian. https://hg.python.org/cpython/rev/b5b2bb56d303 -- nosy: +python-dev

[issue28017] bluetooth.h on big endian needs GNU C extensions

2016-09-08 Thread Martin Panter
Martin Panter added the comment: I don’t have access to the buildbot. Are you sure it uses __typeof__? The error message says “typeof” without the underscores. I suspect that it would work with the underscores. -- nosy: +martin.panter superseder: -> PPC64 Fedora socket and ssl

[issue28017] bluetooth.h on big endian needs GNU C extensions

2016-09-08 Thread Christian Heimes
New submission from Christian Heimes: PPC Fedora build bot is not able to build the _ssl and _socket module. It looks like bluetooth.h is not compatible with std=c99. It uses some GNU C extensions like __extension__, __attribute__((packed)) and __typeof__. The C99 variant -std=gnu99 should do