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

[BUGFIX?] Reset CWD before Preloader #2629

Merged

Conversation

NotHyper-474
Copy link
Contributor

@NotHyper-474 NotHyper-474 commented May 29, 2024

When running the game from the command line and the current working directory is not the executable's directory the Preloader will fail to find the files it needs, this PR aims to fix this problem. This could perhaps also fix #2444 although it seems to be kinda unrelated.

I know that ideally you'd use lime run PLATFORM to run the game without recompiling, but this would not be the case for when downloading the game, specially for Linux players that for whatever reason would like to run it through a terminal without having to change the CWD.

From what I tested it doesn't break anything. Here's a before and after of the problem:
(Before was done in Windows cuz I'm too lazy to recompile, but it should be consistent regardless of the platform)

before.mp4
after.mp4

@EliteMasterEric
Copy link
Member

Tested on my machine and this does indeed fix this issue!

@EliteMasterEric EliteMasterEric merged commit b487adb into FunkinCrew:develop May 30, 2024
@NotHyper-474 NotHyper-474 deleted the bugfix/cwd-preloader-issue branch May 30, 2024 16:47
@EliteMasterEric EliteMasterEric added this to the 0.4.0 milestone Jun 7, 2024
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.

None yet

2 participants