Skip to content

Switched validation to more linux compatible method that enables selection of ELF 32 Bit Binary files #1713

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

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

AmyNicole8715
Copy link

  1. Modified the file selection dialog on Linux to use "All Files (*)" filter, which allows seeing all executable files regardless of extension, including ELF binaries.
  2. Enhanced the validation logic to accept both named Steam binaries and detect executable files with "steam" in their name, allowing the application to properly recognize ELF executables.
  3. Added a utility method isFileExecutable() that verifies if a file exists and is a regular file.

@CLAassistant
Copy link

CLAassistant commented Apr 12, 2025

CLA assistant check
All committers have signed the CLA.

@AmyNicole8715
Copy link
Author

Hang on I found a new error that happens due to the proton issue

@AmyNicole8715
Copy link
Author

Hang on I found a new error that happens due to the proton issue

Ok, users who use this will need to remove the .forceproton, but thats still not 100% solved I still seem to be struggling to get it to start the modded part of the operation, it just doesn't throw errors now

@AmyNicole8715
Copy link
Author

Alright, so far it seems it should work but I cannot be certain that it is due to when my build of the fix loads it does not give me the linux instructions anymore for setting the launch parameters in Valheim, that said I hate programming so I am opting to start the game regardless will report back if I can manage to get "start modded" to properly launch it

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.

2 participants