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

Fixes for Windows #27

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

digimax20
Copy link
Contributor

Fixes for Windows(MinGW64)

@@ -188,16 +188,37 @@ void OgreSetup::createOgreSystem() {
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.

#endif

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