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.

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

[dotnetcore] Fix loading of glfw3.dll on windows, Unit-Testing of glfw3 functional… #1000

Merged
merged 1 commit into from Feb 23, 2020

Conversation

deccer
Copy link
Contributor

@deccer deccer commented Jan 25, 2020

Purpose of this PR

After a fresh clone of opentk, it was impossible to just run ./build.cmd. It would compile and build, but unit-tests would fail, because nativeloader was not able to locate the proper glfw dll.

  • Description of feature/change.

Unit-Tests were unable to locate glfw3.dll.

  • Which part of OpenTK does this affect (Math, OpenGL, Platform, Input, etc).

Platform (glfw3.dll loading in particular)

  • Links to screenshots, design docs, user docs, etc.

N/A

Testing status

  • Fresh clone of opentk/opentk/master
  • ran build.cmd
    => build successful
    => unit-tests threw error glfw3.dll not found
  • Implemented fix
  • ran build.cmd
    => build successful
    => unit-tests successful

@PJB3005
Copy link
Member

PJB3005 commented Jan 26, 2020

Hrm, while this does fix it on Core I'm afraid this won't work correctly on .NET Framework.

@deccer deccer changed the title Fix loading of glfw3.dll on windows, Unit-Testing of glfw3 functional… [dotnetcore] Fix loading of glfw3.dll on windows, Unit-Testing of glfw3 functional… Jan 26, 2020
@frederikja163
Copy link
Member

Whats status on this? @deccer @PJB3005 ?

@deccer
Copy link
Contributor Author

deccer commented Feb 21, 2020

@frederikja163 As discussed after the last conf call, its only affecting .netcore, not mono, not netframework. but you guys have the last word

@frederikja163 frederikja163 merged commit 2ef4d7e into opentk:master Feb 23, 2020
@varon
Copy link
Member

varon commented Feb 23, 2020

Thanks for this PR @deccer.

You've been an enormous help since you've been around. We hope work with you a ton more in the future!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants