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

[util] Allow switching log file at runtime #3516

Closed
wants to merge 1 commit into from

Conversation

AlpyneDreams
Copy link
Contributor

Part of #3411

Required since d3d8 is a single DLL, but links d3d9 as a static library, so we need a way to override the initialization-time log filename.

@AlpyneDreams AlpyneDreams mentioned this pull request Jun 19, 2023
13 tasks
@misyltoad
Copy link
Collaborator

Is there a reason to static link d3d9?

@misyltoad
Copy link
Collaborator

I do not really see the point in overcomplicating this, let's just dynamically link and keep the log files the same.

@misyltoad misyltoad closed this Jun 19, 2023
@WinterSnowfall
Copy link
Contributor

@Joshua-Ashton Dynamically linking caused a lot of crashing on load issues on Windows. Not entirely sure why, but Windows compatibility drastically increased when d8vk switched to static linking. Maybe it's something we need to look deeper into.

@AlpyneDreams AlpyneDreams deleted the d3d9/log branch June 19, 2023 22:07
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