-
-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
fix(main/openal-soft): disable dlopen() of opensles for luanti #23149
base: master
Are you sure you want to change the base?
Conversation
Instead of reverting the patches, would it be possible to disable the |
7061dd3
to
f2b8f1a
Compare
Thank you, I did not really notice that earlier, but yes, adding |
I did not mean to add that option globally. There are many source files which depend on that option. So, I am not sure if |
f2b8f1a
to
efd95b0
Compare
Well, I did not think that other stuff will be affected because it doesn't seem used in the Termux build of this package, but I do not fully understand the problem, since I have no idea why only one app is affected by the problem and not other apps, and yes you are correct, so I will write this exactly how you suggested. |
efd95b0
to
242d3ac
Compare
242d3ac
to
f4fae4f
Compare
@Biswa96 something still seemed confusing to me, even more so than this problem already is, so I carefully double checked everything, and unfortunately, I have to report the following:
I have now thoroughly tested this, and it appears that at the very least, in order for the |
Fixes [Bug]: after openal-soft bump to 1.24.2, sound stopped working in x11/luanti #23148
mpv --ao=openal
andmplayer -ao openal
continue to work with or without this reversion, so it seems ok to me. however I do not understand why this is only required forluanti
, and I have not found any other reverse dependency ofopenal-soft
that was negatively affected by the bump toopenal-soft
1.24.2.