Skip to content
This repository was archived by the owner on Dec 3, 2023. It is now read-only.

Conversation

@digimax20
Copy link
Contributor

Fixes for Windows(MinGW64)

mPluginLoader.loadPlugin("Codec_FreeImage");
mPluginLoader.loadPlugin("Plugin_ParticleFX");
mPluginLoader.loadPlugin("RenderSystem_GL3Plus"); //We'll use OpenGL on Windows too, to make it easier to develop
mPluginLoader.loadPlugin("RenderSystem_Direct3D11");
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are you getting errors without these? Normally Ember should run without having to load the DirectX-plugins.


auto renderSystem = mRoot->getAvailableRenderers().front();
//auto renderSystem = mRoot->getAvailableRenderers().front();
const auto& renderers = mRoot->getAvailableRenderers();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We used to support both DirectX and OpenGL, but in the end it just became a huge hassle since it meant I had to provide shaders for both GLSL and HLSL. So in the end we decided on only supporting OpenGL.
In the long run we want to move to a PBS based rendering flow, in which we hopefully won't need to write any shaders. And in the even longer run we want to see Vulkan support in Ogre, making away with both DirectX and OpenGL.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, I didn’t come here for a long time. Yes, I wanted to make the game work on DirectX. But as I understand it, this is not worth doing. You can apply changes without this code.

@erikogenvik erikogenvik force-pushed the master branch 7 times, most recently from cd7713b to 21b7f7e Compare April 15, 2021 20:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants