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

Add Linux patcher script #3

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

Conversation

rakelley
Copy link

@rakelley rakelley commented Oct 5, 2023

I've put together a basic patching script to save Linux users like me a little time/research. I figured the best option for portability was to compile dtkit-patch locally rather than have you include and test another binary in your releases, which adds dependencies on cargo and git, but that shouldn't be a significant hurdle for any Linux user.

If you have any changes, please let me know. I didn't find a public repo for the docs at https://dmf-docs.darkti.de/index#/installing-mods but would be happy to submit a PR for that as well if it exists.

@Julianoe
Copy link

Julianoe commented Nov 17, 2023

For people stumbling on this PR request, install rustup (toolchain for the programming language Rust) and it will include cargo which is Rust package manager

@Bax12
Copy link

Bax12 commented Nov 20, 2023

Would a simple script containing wine ./tools/dtkit-patch.exe --toggle ./bundle not be sufficient?
While a native dtkit-patch is nice, you need wine/proton for the game anyways.

@withoutclass
Copy link

This worked perfectly for me, but I already had rust installed. I would agree that if wine must already be installed on the system and available on a users path to have the game work that the wine approach would be superior.

@danreeves
Copy link

We should just vendor the linux dtkit-patch binary rather than building it on demand

@rakelley
Copy link
Author

I've updated the script to prefer using wine, keeping the native patcher only as a fallback or manual override via -f|--force

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.

5 participants