minetest/irrlicht

IrrlichtMt target does not export SDL includes as PUBLIC

JosiahWI opened this issue · 4 comments

It's just flat out incorrect, and it's kind of hilarious that it didn't break for anybody until now. ${SDL_INCLUDES} should be added to the public include directories.

sfan5 commented

Should it? I don't see any SDL_*.h referenced in the 'include' folder.

This is only an issue if source/CIrrDeviceSDL.h is included anywhere in the public includes. I may have sounded a false alarm.

I haven't looked in detail but I think this can be closed. Sorry about that.

sfan5 commented

Okay.