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.

Sign up for GitHub

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

Jump to bottom

JIT on Windows is currently not supported in phpseclib #10223

Closed
rossaddison opened this issue Sep 20, 2023 · 5 comments
Closed

JIT on Windows is currently not supported in phpseclib #10223

rossaddison opened this issue Sep 20, 2023 · 5 comments

Comments

@rossaddison
Copy link

phpseclib/phpseclib#1941
@psalm-github-bot
Copy link

Hey @rossaddison, can you reproduce the issue on https://psalm.dev ?

@rossaddison
Copy link
Author

rossaddison commented Sep 20, 2023 •
edited

No please refer to the stack trace in the above link. I am currently trying to run

Psalm 5.15.0@5c774aca4746caf3d239d9c8cadb9f882ca29352

@danog
Copy link
Collaborator

danog commented Sep 20, 2023 •
edited

Psalm does not use phpseclib, but it does include bootstrap.php during analysis, which is causing the issue due to your project using phpseclib.

I recommend simply defining PHPSECLIB_ALLOW_JIT=1 when running Psalm, I'll also open a PR @ phpseclib to move the check from bootstrap.php to the Math engine, instead.

@danog danog closed this as completed Sep 20, 2023
@danog
Copy link
Collaborator

danog commented Sep 20, 2023

Ref phpseclib/phpseclib#1942

@rossaddison rossaddison mentioned this issue Sep 23, 2023
Closed
@rossaddison
Copy link
Author

Temporary hack: Comment out the above code I mentioned.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@danog @rossaddison