Skip to content

Fix compilation error#118

Open
paradox9955 wants to merge 1 commit intoMorsGames:masterfrom
paradox9955:patch-1
Open

Fix compilation error#118
paradox9955 wants to merge 1 commit intoMorsGames:masterfrom
paradox9955:patch-1

Conversation

@paradox9955
Copy link

Recently, there have been reports of SM64Plus failing to compile as soon as the compiling process starts. The error in question goes something like this: armips.cpp:1414:34: note: 'int64_t' is defined in header '<cstdint>'; this is probably fixable by adding '#include <cstdint>'. This PR fixes that.

@MorsGames
Copy link
Owner

Are you on Windows or Linux? I'm not having any compilation issues with the latest version of MSYS2.

I mean if it's helping you I don't see a problem adding a simple include statement (assuming it doesn't affect compilation on other platforms, which I'll test) but I thought I'd ask.

@paradox9955
Copy link
Author

Are you on Windows or Linux? I'm not having any compilation issues with the latest version of MSYS2.

I mean if it's helping you I don't see a problem adding a simple include statement (assuming it doesn't affect compilation on other platforms, which I'll test) but I thought I'd ask.

I'm on windows. Fresh install of MSYS2 and everything. It seems to only happen on newer versions of GCC.

@paradox9955
Copy link
Author

@SeongGino
Copy link

Can at least confirm on the Linux side that this is still an issue, and pulling in OP's branch instead allows it to build just fine.

@sks316
Copy link

sks316 commented Aug 17, 2025

Can confirm. Building master does not work on EndeavourOS Linux, pulling OP's branch fixes this.

@code-irisnk
Copy link

This fixed building on Arch too. Please merge.

@JordanLongstaff
Copy link

This fixed building on Arch too. Please merge.

Reminder that this comment was made nearly three months ago.

@paradox9955
Copy link
Author

This fixed building on Arch too. Please merge.

Reminder that this comment was made nearly three months ago.

Mors seems to be planning a release for 4.0 with this change already implemented. Not sure when it'll release, but it'll come eventually.
Screenshot_20251126_094330

@space-escalator
Copy link

Can confirm that this fix works on Fedora 43 in January 2026.

@malucard
Copy link

malucard commented Feb 6, 2026

Still happens, still the same fix. It's currently not possible to use SM64 Plus at all without this fix.

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.

8 participants