Skip to content

Added support for detecting ImGUI library - #75

Open
Quix0r wants to merge 1 commit into
eventhorizon5:masterfrom
Quix0r:fixes/find-imgui-headers
Open

Added support for detecting ImGUI library#75
Quix0r wants to merge 1 commit into
eventhorizon5:masterfrom
Quix0r:fixes/find-imgui-headers

Conversation

@Quix0r

@Quix0r Quix0r commented Aug 16, 2025

Copy link
Copy Markdown
Contributor

This has fixed a compiler error when attempting to find imgui.h.

@eventhorizon5

Copy link
Copy Markdown
Owner

Thank you, I'll get this merged, I'm mainly thinking about the right way to get the Dear ImGui feature working since it's new. Ogre comes built with ImGUI support inside it, and Skyscraper can find some of the headers using that, since it's using Ogre's Overlay interface to access ImGui. If ImGui is installed separately, Ogre and Skyscraper might each be built with a different version of the library, causing issues.

eventhorizon5 added a commit that referenced this pull request Aug 16, 2025
@eventhorizon5

Copy link
Copy Markdown
Owner

I've done a manual merge of your changes, thank you. I'll see what can be done about getting ImGui integrated properly so that there's no library mismatch issues. I've disabled the ImGui code in the simulator for now, mostly because it was crashing on shutdown and also is unused at the moment.

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.

2 participants