make fail Aug 16 10:11 EST

[osmanfb]

At link time I get multiple definition errors

/usr/bin/ld: /usr/local/lib/libraylib.a(rtextures.c.o): in function stbi_failure_reason': rtextures.c:(.text+0x277): multiple definition of stbi_failure_reason'; src/imageio.o:/home/osman/MATH/mathilda/src/external/stb/stb_image.h:973: first defined here
/usr/bin/ld: /usr/local/lib/libraylib.a(rtextures.c.o): in function stbi_image_free': rtextures.c:(.text+0x63a): multiple definition of stbi_image_free'; src/imageio.o:/home/osman/MATH/mathilda/src/external/stb/stb_image.h:1102: first defined here
/usr/bin/ld: /usr/local/lib/libraylib.a(rtextures.c.o): in function stbi_set_flip_vertically_on_load': rtextures.c:(.text+0x659): multiple definition of stbi_set_flip_vertically_on_load'; src/imageio.o:/home/osman/MATH/mathilda/src/external/stb/stb_image.h:1117: first defined here
/usr/bin/ld: /usr/local/lib/libraylib.a(rtextures.c.o): in function `stbi_load':
..... a lot more ...

I added -zmuldefs to make it link, but don't know if this was a bad idea ?

[stblake]

Fixed in c0b1674 (pushed to main).

Root cause. Both src/imageio.c and raylib's libraylib.a (rtextures.c.o) bake in their own copy of the vendored stb_image implementation, and each exports the stbi_* functions with external linkage. So when USE_GRAPHICS=1, the two copies collide at link time — the multiple definition of stbi_load / stbi_failure_reason / stbi_image_free ... errors you saw.

About -z muldefs. It does make the link succeed, but I'd recommend against keeping it: it doesn't merge anything, it just tells the linker to silently keep whichever copy it sees first and discard the rest. That's an ODR violation — you'd end up with one arbitrary stb version servicing both Mathilda and raylib, and it also disables duplicate-symbol protection for every other symbol in the program, so a genuine accidental clash elsewhere would no longer be caught.

The fix scopes our copy of stb to a single object file instead. src/imageio.c now defines STB_IMAGE_STATIC / STB_IMAGE_WRITE_STATIC before the includes, which gives every stb function internal linkage: the code lives in imageio.o alone and exports no stbi_* symbols at all, so there's nothing left for raylib's copy to collide with. This is safe because imageio.c is the only translation unit that both defines and calls those functions. (One companion change: -Wunused-function is added to that file's scoped #pragma GCC diagnostic block, since internal linkage turns any stb entry point we don't call into an "unused static function".)

After you pull: please drop the -z muldefs flag — it's no longer needed, and removing it restores real duplicate-symbol protection.

Verified here against raylib 5.5: a full make with USE_GRAPHICS=1 links cleanly, nm imageio.o | grep stbi shows the symbols as local (t, not T), and PNG/JPEG Export/Import still round-trip correctly. Let me know if the link is still unhappy on your setup.