gleb-sevruk / pycrunch-trace

Time Travel Debugging for Python
https://pytrace.com
MIT License
152 stars 12 forks source link

Install fails due to Cython build issue #7

Open Zac-HD opened 3 years ago

Zac-HD commented 3 years ago
ImportError: Building module pycrunch_trace.client.networking.strategies.native_write_strategy failed:
    FutureWarning: Cython directive 'language_level' not set, using 2 for now (Py2).
        This will change in a later release!
        File: pycrunch_trace/client/networking/strategies/native_write_strategy.pyx

I get the same error under Windows and Linux. Cython docs suggest that you shouldn't rely on the user having Cython installed - providing prebuilt wheels for Windows, Linux, and OSX would be great (and is not too hard with GitHub Actions), but even just shipping the generated C code would be an improvement I think.

Is there a way to disable the native tracer in the meantime, as a slow-but-effective workaround?

gleb-sevruk commented 3 years ago

Is this really a fail or just a warning? I usually get those warnings but everything works fine.

As for non-Cython version, there is old SimpleTracer, but it may be a bit out-of-date: https://github.com/gleb-sevruk/pycrunch-trace/blob/master/pycrunch_trace/client/api/trace.py#L57

Andrej730 commented 2 months ago

I get the same warnings but they don't break the build process, there probably was some other issue in the traceback after the attached logs.