Import crashes due to a regex_error

[inducer]

Import crashes due to a regex_error

[larsmans]

Turns out the following program causes the same crash, GCC 4.8.4:

#include <regex>

static const std::regex ver_regex("^OpenCL ([0-9]+)\\.([0-9]+).*");

int main() {}

Works with GCC 4.9.0, though. I'm starting to doubt whether C++11 on Linux is mature enough for production use.

[larsmans]

Ah: <regex> was simply not implemented until GCC 4.9.0. So the minimum requirement for compiling PyOpenCL is now that version. That might be something to document.

[inducer]

4.9 seems like a somewhat steep requirement... @yuyichao, opinions?

[larsmans]

FYI, both Ubuntu 14.04 LTS and CentOS 7 still ship 4.8. Ubuntu has 4.9 as an optional package that require setting environment variables. I didn't find it in EPEL.

There doesn't seem to be a CPython C API for re...

[yuyichao]

So my opinion is that std::regex was the most obvious way to port the old code and I was on gcc 4.9.0 at that time so it was available (and had no idea it was only added in that version....).

It doesn't have to use a regular expression as shown in #86....

[yuyichao]

And FWIW, the original code for getting platform version from string in python is still there. The ability to query this info is added in order to decide whether to use the callback API for asynchronously releasing the event.

On a related note, the Event.set_callback API was added a while back only in the CFFI branch so I didn't bother adding it to the doc. Now the cffi branch is the master and seems that it is still not documented yet.