sm64plus icon indicating copy to clipboard operation
sm64plus copied to clipboard

Fix compilation error

Open paradox9955 opened this issue 7 months ago • 8 comments

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.

paradox9955 avatar Jul 24 '25 01:07 paradox9955

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.

MorsGames avatar Aug 01 '25 19:08 MorsGames

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 avatar Aug 01 '25 20:08 paradox9955

Also. This is the same error that these people are getting.

  • https://github.com/MorsGames/sm64plus/issues/113
  • https://github.com/MorsGames/sm64plus/issues/116
  • https://github.com/MorsGames/sm64plus/issues/117
  • https://github.com/MorsGames/sm64plus/issues/120

paradox9955 avatar Aug 01 '25 21:08 paradox9955

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.

SeongGino avatar Aug 13 '25 04:08 SeongGino

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

sks316 avatar Aug 17 '25 02:08 sks316

This fixed building on Arch too. Please merge.

code-irisnk avatar Aug 31 '25 09:08 code-irisnk

This fixed building on Arch too. Please merge.

Reminder that this comment was made nearly three months ago.

JordanLongstaff avatar Nov 26 '25 14:11 JordanLongstaff

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

paradox9955 avatar Nov 26 '25 15:11 paradox9955

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

space-escalator avatar Jan 18 '26 17:01 space-escalator

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

malucard avatar Feb 06 '26 16:02 malucard