-
Notifications
You must be signed in to change notification settings - Fork 188
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
Faraday test suit exits with "native function not supported at this point" #2090
Comments
Full log is here |
I think the issue is that we haven't considered someone using https://github.com/toland/patron/blob/v0.13.3/ext/patron/session_ext.c#L814-L817 |
I tried to make a spec for this in master...Shopify:thread_call_without_gvl-bug but I can't figure out how to make the function native. I tried functions which I think should be native and close-enough to identity such as |
@chrisseaton I can't see any diff on that link, could you check it? |
Hmm sorry looks like I 'tidied up' this branch and deleted it. It's not in my reference log either. Sorry about that. |
|
I run with latest truffleruby https://github.com/lostisland/faraday/runs/1248204647?check_suite_focus=true |
Could you file a separate issue? |
done: #2122 |
Tests passed but in the end I see this exception:
The text was updated successfully, but these errors were encountered: