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

#947 broke building on x86_64-pc-windows-msvc/i686-pc-windows-msvc with VS2019 #958

Closed
jrb0001 opened this issue Dec 31, 2019 · 7 comments · Fixed by #960
Closed

#947 broke building on x86_64-pc-windows-msvc/i686-pc-windows-msvc with VS2019 #958

jrb0001 opened this issue Dec 31, 2019 · 7 comments · Fixed by #960

Comments

@jrb0001
Copy link
Contributor

jrb0001 commented Dec 31, 2019

[sdl2-sys 0.32.6] SDL_string.obj : error LNK2019: Verweis auf nicht aufgel�stes externes Symbol "memset" in Funktion "SDL_vsnprintf_REAL". [...\target\x86_64-pc-windows-msvc\debug\build\sdl2-sys-48b20b00f4980a00\out\build\SDL2.vcxproj]
[sdl2-sys 0.32.6] ...\target\x86_64-pc-windows-msvc\debug\build\sdl2-sys-48b20b00f4980a00\out\build\Release\SDL2.dll : fatal error LNK1120: 1 nicht aufgel�ste Externe [...\target\x86_64-pc-windows-msvc\debug\build\sdl2-sys-48b20b00f4980a00\out\build\SDL2.vcxproj]
@Cobrand
Copy link
Member

Cobrand commented Jan 2, 2020

What did it break exactly? What's the error message?

@jrb0001
Copy link
Contributor Author

jrb0001 commented Jan 2, 2020

Oops, I can't believe I forgot to include the error message. Anyways, it is the same error that #951 fixed (and #956 fixed the regression introduced by that). See also my comment in #947.

@Cobrand
Copy link
Member

Cobrand commented Jan 2, 2020

It is the same error that #951 fixed... so it is fixed by #956 and fine now or is this still happening on the latest version of master?

@jrb0001
Copy link
Contributor Author

jrb0001 commented Jan 3, 2020

Yeah, it was fixed by both #951 and I verified it to work before submitting #956. But those patches are not applied anymore after merging #947 because the SDL2 version number is different and the patching code checks for that.

@Cobrand
Copy link
Member

Cobrand commented Jan 3, 2020

I see, so if the patch is still up to date, we just have to rename the 2.0.9 in the filename to 2.0.10, otherwise we should update it if there are conflicts, or remove it if it's not necessary anymore.

@rlabrecque
Copy link
Contributor

Sorry for the spam!

I thought it was going to be that easy, but it required creating the diff with the 2.0.10 version of the file being patched.

MinGW was probably also broken before #960 .

Cobrand added a commit that referenced this issue Jan 6, 2020
Fix errors building on Windows with bundled #958
@jrb0001
Copy link
Contributor Author

jrb0001 commented Jan 6, 2020

@rlabrecque MinGW was broken by #951 and fixed by #956. #947 broke VS2019 but I think MinGW still worked and now you fixed everything again with #960.

tanis2000 pushed a commit to tanis2000/rust-sdl2 that referenced this issue Jan 24, 2020
sypwex pushed a commit to sypwex/rust-sdl2 that referenced this issue Jun 2, 2024
sypwex pushed a commit to sypwex/rust-sdl2 that referenced this issue Jun 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants