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

Compile fails on MSYS2 due to missing assert include in epos.c #182

Closed
Nep2Disk opened this issue Jan 11, 2025 · 2 comments · May be fixed by #185
Closed

Compile fails on MSYS2 due to missing assert include in epos.c #182

Nep2Disk opened this issue Jan 11, 2025 · 2 comments · May be fixed by #185

Comments

@Nep2Disk
Copy link

When I was building with MSYS2 on windows I was unable to continue building the sdl_win32 version due to a missing include for assert in epos.c.

To fix it on my end I just added the include for that header.

If the exact error is wanted it was this:
epos.c 17:5: warning: implicit declaration of function 'assert'
@freshollie
Copy link
Collaborator

I've updated the readme as I no longer recommend trying to build in msys when trying to build the PC ports

@bitten2up
Copy link
Contributor

was working on building with msys2 yesterday and this is only the first pitfall

this can be solved by just adding a #include <assert.h> at the top locally for the time being (this file has cross platform support for both unix-likes using gcc and windows using fucking msvc for whatever reason, this is the only thing making it unable to work)

second potential pitfall is an out of memory error in another file that i don't remember its name rn

third pitfall is having to define multiple commands by hand in the make command because they dont have prefixes for some reason

fourth pitfall has to do with how files are stored in windows (\r\n vs \n) and it assuming it will always be in unix format

and thats where i stopped off last night...

@bitten2up bitten2up mentioned this issue Jan 14, 2025
Open
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

Successfully merging a pull request may close this issue.

DRAFT: msys2 working bitten2up/sa2
3 participants
@freshollie @bitten2up @Nep2Disk