-
Notifications
You must be signed in to change notification settings - Fork 93
Linux support #4927
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
base: develop
Are you sure you want to change the base?
Linux support #4927
Conversation
Signed-off-by: Rye <[email protected]>
Signed-off-by: Rye <[email protected]>
Fix various files missing linden_common Signed-off-by: Rye <[email protected]>
Restore changes for GLIB based dbus support from archive/develop Signed-off-by: Rye <[email protected]>
Signed-off-by: Rye <[email protected]>
Signed-off-by: Rye <[email protected]>
Signed-off-by: Rye <[email protected]>
Signed-off-by: Rye <[email protected]>
Signed-off-by: Rye <[email protected]>
Signed-off-by: Rye <[email protected]>
Signed-off-by: Rye <[email protected]>
Signed-off-by: Rye <[email protected]>
Signed-off-by: Rye <[email protected]>
Signed-off-by: Rye <[email protected]>
Signed-off-by: Rye <[email protected]>
Signed-off-by: Rye <[email protected]>
Signed-off-by: Rye <[email protected]>
Signed-off-by: Rye <[email protected]>
Signed-off-by: Rye <[email protected]>
Signed-off-by: Rye <[email protected]>
Signed-off-by: Rye <[email protected]>
Signed-off-by: Rye <[email protected]>
Signed-off-by: Rye <[email protected]>
Signed-off-by: Rye <[email protected]>
Signed-off-by: Rye <[email protected]>
Signed-off-by: Rye <[email protected]>
Signed-off-by: Rye <[email protected]>
Signed-off-by: Rye <[email protected]>
Signed-off-by: Rye <[email protected]>
Signed-off-by: Rye <[email protected]>
We default to and only use aliased lines. Signed-off-by: Rye <[email protected]>
…ce package Signed-off-by: Rye <[email protected]>
Add GL constants needed for compatibility with physicsextensions_source package
Improve fix for line width crashes under certain Mesa GL drivers
| case LLFilePicker::FFLOAD_RAW: | ||
| filter_vec.push_back({ "RAW files (*.raw)", "raw" }); | ||
| break; | ||
| case LLFilePicker::FFLOAD_MODEL: |
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.
I noticed that I can only upload DAE files for models on Linux.
So I think this isn't quite correct. The Windows and Mac implementation also includes *.gtlf and *.glb files for LLFilePicker::FFLOAD_MODEL. I guess it's missing here, too.
| case LLFilePicker::FFLOAD_GLTF: | ||
| filter_vec.push_back({ "glTF (*.gltf; *.glb)", "gltf;glb" }); | ||
| break; | ||
| case LLFilePicker::FFLOAD_COLLADA: |
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.
Couldn't LLFilePicker::FFLOAD_COLLADA be dropped completely? I couldn't find any usage besides other case blocks anywhere.
| Comment=Client for the On-line Virtual World, Second Life\n\ | ||
| Exec=${installation_prefix}/secondlife\n\ | ||
| Path=${installation_prefix}\n\ | ||
| Exec=secondlife\n\ |
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.
This doesn't work on Plasma. It complains Could not find the program 'secondlife'. And according to https://specifications.freedesktop.org/desktop-entry/latest/exec-variables.html Exec should be an absolute path or the executable needs to be in $PATH. Path only sets the working directory for the application but isn't used to lookup the executable.
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.
This is incorrect. Because the desktop entry is created from a bash script, the variable (represented by the $ sign) is substituted for the actual (absolute) path.
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.
Yes, it is substituted. But not in the Exec line anymore. Right now it results in a desktop file like:
Path=/home/nici/installation/dir
Exec=secondlife
And this would require that I add /home/nici/installation/dir to $PATH. The version before the change was correct:
Exec=/home/nici/installation/dir/secondlife
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.
In which case, this is likely a difference in how Plasma and Gnome works (and likely different in COSMIC, et al.)
Seemingly, GNOME will interpret the PATH variable as the launch directory (and thus, the working directory), and if what you're saying is correct, then KDE is doing this another way. Easily fixed, will likely PR some point next week.
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.
Yes, it might be something like that. The spec says absolute path or look up with $PATH. I guess Gnome does it a bit more relaxed and Plasma sticks to the letters.
|
Additional fixes in #4954 |
…yland Signed-off-by: Rye <[email protected]>
Signed-off-by: Rye <[email protected]>
Signed-off-by: Rye <[email protected]>
Linux support for the viewer. Still very much a WIP.
Please target develop-linux if you'd like to contribute to linux support. We're trying to keep develop more or less clean until we've shipped 2025.08.