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

Sync with Q2PRO: More filesystem changes #360

Merged
merged 9 commits into from
Nov 22, 2023

Conversation

res2k
Copy link
Contributor

@res2k res2k commented Nov 22, 2023

More upstream filesystem changes (mostly related to game dir handling).

Files from game dir should always override baseq2.
When changing game dir, default.cfg needs to be executed to update
‘newgame’ alias and such, so that single player menu works.

Rather than skipping default.cfg and config.cfg if baseq2/autoexec.cfg
exists, exec baseq2/autoexec.cfg after default.cfg and config.cfg.
Empty string means no game dir is set. Resetting to default is wrong if
DEFGAME is not empty.
Doing so could lead to infinite config exec loops.
Apparently, mkdir("c:") is legal, but mkdir("c:/") gives EACCESS on
Windows. So if path has double slashes after drive part, such as
"c://foo/bar" FS_CreatePath() would attempt to mkdir("c:/") and fail.
Change it to always skip the drive part (and leading slashes).

Fixes NVIDIA#316.
@apanteleev apanteleev merged commit 119c4e7 into NVIDIA:master Nov 22, 2023
2 checks passed
@res2k res2k deleted the q2pro-sync-files branch January 21, 2024 12:32
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.

3 participants