8000 use NativeLibrary to try loading libgamemodeauto.so.0 by navi-desu · Pull Request #68 · goatcorp/XIVLauncher.Core · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

use NativeLibrary to try loading libgamemodeauto.so.0 #68

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

Merged
merged 1 commit into from
Aug 12, 2023

Conversation

navi-desu
Copy link
Contributor

this removes the reliance on path checking, instead moving it to the (appropriate) dlopen(3) call that dotnet does under the hood. that call loads libs based on different variables that are set by the user. thus allowing the user to have the lib in different paths scanned by /etc/ld.so.cache, or override it by setting LD_LIBRARY_PATH.

this issue should fix #67 (and the associated gentoo bug https://bugs.gentoo.org/911936)

i tested this patch on my gentoo machine, but not in flatpaks, and i am unsure if flatpaks have the /app/lib in it's ld path. if anyone does now, please let me now. depending on it, the file check for the flatpak specific path might need to be re-introduced

this removes the reliance on path checking, instead moving it to the
(appropriate) dlopen(3) call that dotnet does under the hood. that call
loads libs based on different variables that are set by the user. thus
allowing the user to have the lib in different paths scanned by
/etc/ld.so.cache, or override it by setting LD_LIBRARY_PATH.

Signed-off-by: Anna (navi) Figueiredo Gomes <navi@vlhl.dev>
@Blooym Blooym self-requested a review August 12, 2023 22:20
Copy link
Contributor
@Blooym Blooym left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested against a flatpak build and from dotnet run inside of a Fedora Workstation 38 container, game loads perfectly fine with gamemode detected and used on boot.

@Blooym Blooym merged commit 8f1933d into goatcorp:main Aug 12, 2023
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.

add /usr/lib64 as gamemode location
2 participants
0