Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Install fails due to Cython build issue #7

Open
Zac-HD opened this issue Aug 23, 2021 · 2 comments
Open

Install fails due to Cython build issue #7

Zac-HD opened this issue Aug 23, 2021 · 2 comments

Comments

@Zac-HD
Copy link
Contributor

Zac-HD commented Aug 23, 2021

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
Copy link
Owner

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
Copy link
Contributor

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants