Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Case sensitive filesystem support #461

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mdluhosch
Copy link

@mdluhosch mdluhosch commented May 31, 2024

Hi,

I am currently playing around with this repo under Linux. One issue I had is that linux file systems are case sensitive. I thought maybe you want to have the "correct" names also in the main repo.
Also on some spots the <algorithm> header was missing (on linux the std::find does not work without it)
The change in MfdAlign.cpp seems to be needed because the winegcc compiler is a bit more strict.

Kind Regards
Michael

…w missing headers. With this it should be easier to compile on a case sensitive file system

Signed-off-by: Michael Dluhosch <[email protected]>
@TheGondos
Copy link
Contributor

I'm curious why you did not have problems with Orbitersdk.h/orbitersdk.h?

@mdluhosch
Copy link
Author

I did not compile everything so far. Currently have some trouble with wine on my distro. So probably I did not yet stumble upon source files with the "incorrect" orbitersdk.h

@TheGondos
Copy link
Contributor

Maybe someone could try enabling case sensitivity on a Windows box and see what happens. It would be easier to do a complete check this way.

Good luck with your plan. I gave up compiling with winegcc, it was easier to port the whole thing to linux^^

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