-
Notifications
You must be signed in to change notification settings - Fork 22.2k
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
Fix dll load failure in virtual environments on Windows #39622
Conversation
@ashwithadevasya Could you please test whether this fix is sufficient to solve your problem? |
💊 CI failures summary and remediationsAs of commit 3ef5e73 (more details on the Dr. CI page): 💚 💚 Looks good so far! There are no failures yet. 💚 💚 This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.Please report bugs/suggestions on the GitHub issue tracker or post in the (internal) Dr. CI Users group. This comment has been revised 13 times. |
d9eb55f
to
3ce448b
Compare
@peterjc123 do you think it would be possible to add a comment at the top of this function explaining what it is doing, how it is going about doing it, and how some alternate plausible implementation strategies (e.g., the ones from before this bug) don't work? |
Co-authored-by: Nikita Shulga <nikita.shulga@gmail.com>
21df73d
to
3ef5e73
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@malfet is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
Fixes #39620.