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

Link to C bindings of release 1.4.0 is broken on Windows #191

Closed
mtth-bfft opened this issue Jan 17, 2024 · 2 comments
Closed

Link to C bindings of release 1.4.0 is broken on Windows #191

mtth-bfft opened this issue Jan 17, 2024 · 2 comments
Labels
CI Relative to the Continuous Integration wontfix This will not be worked on

Comments

@mtth-bfft
Copy link

The temporary fix in #172 is not used when producing releases in the CI. Due to that fix missing, linking to release 1.4.0 in C on Windows produces the same errors as described in that issue:

unresolved external symbol __imp_NtWriteFile
unresolved external symbol __imp_RtlNtStatusToDosError
@commial commial added this to the mla v1.5.0 milestone Mar 8, 2024
@commial commial added the CI Relative to the Continuous Integration label Jun 17, 2024
@commial
Copy link
Contributor

commial commented Jun 17, 2024

Linked to #173

@commial commial removed this from the mla v1.5.0 milestone Jun 17, 2024
@commial commial added the wontfix This will not be worked on label Jun 17, 2024
@commial
Copy link
Contributor

commial commented Jun 17, 2024

As specified in #173 , it seems the problem is the need to link with ntdll.lib when linking with mla.lib.

I'll update the documentation accordingly.
If you still have a problem, please re-open this issue

@commial commial closed this as completed Jun 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
CI Relative to the Continuous Integration wontfix This will not be worked on
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@commial @mtth-bfft