-
Notifications
You must be signed in to change notification settings - Fork 54
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
Feature/linux gtk removal #1225
Feature/linux gtk removal #1225
Conversation
Remove old GTK code and wire up some missing fltk logic in llfilepicker This will leave linux_crash_logger as an empy stub which could be used as a skeleton in case crash logging gets implemented. Otherwise linux_crash_logger and what is left of it in some cmake files can be deleted.
This revealed openal has also a dependebcy on sndio. Consequently the dependency was moved to OPENMAL.cmake and canot be dropped.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the contribution!
We have two build errors so far:
/home/runner/work/viewer/viewer/indra/newview/llfilepicker.cpp:1165:11: error: enumeration value 'FFSAVE_GLTF' not handled in switch [-Werror,-Wswitch]
switch ((ESaveFilter) filter)
^~~~~~~~~~~~~~~~~~~~
/home/runner/work/viewer/viewer/indra/newview/llfilepicker.cpp:1228:11: error: enumeration values 'FFLOAD_GLTF', 'FFLOAD_MATERIAL', and 'FFLOAD_MATERIAL_TEXTURE' not handled in switch [-Werror,-Wswitch]
switch ((ELoadFilter) filter)
^~~~~~~~~~~~~~~~~~~~
FFSAVE_GLTF,FFLOAD_GLTF, FFLOAD_MATERIAL, FFLOAD_MATERIAL_TEXTURE
Thank you for starting the builds. I fixed those missing cases. Sadly I do not get the error locally, so it is a bit of a shot in the dark and I hope there won't be any more incoming |
Thanks, starting a new build. |
@marchcat fyi, while looking at this I got a feeling those filetypes are also not handled for OSX (at least partially). I suggest someone with access to a Mac giving that a short investigation. |
This will remove the last obsolete parts of LL_GTK
This source is not used anymore, would be highly outdated (targeting GTK2 ) and could not even be used, as there is no GTK 3P.