-
Notifications
You must be signed in to change notification settings - Fork 73
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
ReShade: Use ALTEXEPATH
to install ReShade DLLs for Custom Commands with ONLY_CUSTOMCMD=1
#1092
Conversation
sorry for the late test. I got sidetracked with other things, but.... it seems its still installing into the custom command dir steamtinkerlaunch.log |
Oops, that's because I'm an idiot and removed a line that is supposed to be there... |
92d090a
to
3b6ffb0
Compare
Should work now with 3b6ffb0. |
Works as expected for my case of using persona 3 reload with the reloaded mod manager. Ill also test to see if everything works as expected when there is no custom command. Unfortunately, I don't think I have any games that have launchers in a different directory than the main executable (Star Wars, maybe?). |
I'll do some testing tomorrow of that case as well then keeping an extra eye on it, but sounds like this is in a good position! Thanks! |
Didn't get a chance to test this again today, but once I have time I will and will merge this hopefully soon into the week. |
no worries I haven't been able to test extensively but I did fire up a few different games and mad sure it installs Reshade games no regressions there |
Seems safe in my tests as well. Pending CI this will be merged :-) |
Fixes #1089.
This PR adds an extra check to
setFullGameExePath
, where if an alternative EXE path is defined, this should be used in place of the custom command EXE directory. This makes sense to me, as the order of priority should be to useALTEXEPATH
above all other paths when deciding where to put things, and otherwise either the custom command folder (if custom commands are enabled) or the game EXE folder should be used.This should address an issue where when installing ReShade with
ONLY_CUSTOMCMD
enabled, ReShade would install shaders to the ALTEXEPATH (as this is handled bysetShadDest
) but it would not do this for the ReShade DLLs - handled by a call ininstallReshade
tosetShadDestDir
, which in turn callssetFullGameExePath
to get the path where the DLLs should be installed.This will need some testing to ensure this doesn't cause any regressions. The following cases should be testing: