-
Notifications
You must be signed in to change notification settings - Fork 256
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
Make JIT usable on Windows #21
Comments
If you give me push access to the repo I should be able to set it all up |
Just added you as a “collaborator” on the repo, I think this gives you the permission to push? |
Yes! |
So it seems it's not easy to compile on Windows: https://ci.appveyor.com/project/badboy/rbpf/build/job/9kmut2g40uto44qp I could skip all jit things in Windows builds, but that would severly strip down the library |
Yeah, having the JIT is great, but… I don't know if there is something equivalent on Windows to the So I'd go for a “first step” without JIT on Windows, I think. Does it compile without the JIT? |
According to various internet resources |
AppVeyor was added with #23. Renaming the issue, for the record: JIT compilation remains to be adapted for Windows. |
@badboy Where can I check if the Windows build was triggered, and if it was successful? Maybe there's some badge somewhere we can add to the README for quick access to the builds, do you know about that? |
Hmm, it seems collaborator is not enough as I don't have the permissions to set any webhooks.
In the settings you will also find a badge then. And from the overview page you can trigger a build. |
Thanks! I need to provide “Organizations and teams Read-only access” to AppVeyor in order to login with my GitHub account. I have to check that it's fine on my side, might take a couple of days before I add the project. |
Reopening. I forgot the issue was about the JIT. |
Mention that we support all of Linux/MacOS/Windows, but that the JIT does not work on Windows (see issue #21 about that).
We have a new JIT that works on Windows (currently as an opt-in feature). #86 |
Would be great to have rbpf compiling on Windows.
Quoting @badboy:
See issue #19.
The text was updated successfully, but these errors were encountered: