Skip to content
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

Added a launch script for the editor on Linux #6441

Draft
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

britzl
Copy link
Contributor

@britzl britzl commented Feb 24, 2022

This change adds a launch script for the editor on Linux. The launch script will detect common problems when launching Defold on Linux and either try to automatically fix the problem or suggest a solution to the user.

Fixes #6372

@britzl britzl marked this pull request as draft February 24, 2022 14:55
@JustAPotota
Copy link
Contributor

This script's libffi detection won't work for Arch-based distros like Manjaro since they store libraries directly in /usr/lib/ instead of /usr/lib/x86_64-linux-gnu/. It should be fine to just add additional checks for /usr/lib/libffi.so.6 and /usr/lib/libffi.so.7 alongside the Debain paths. It'd also be nice to have different error messages for Arch:

glxinfo could not be found
Install using 'sudo pacman -S mesa-utils'
Defold requires libffi.so.6 or libffi.so.7
Install using 'sudo pacman -S libffi7'

@savegame
Copy link

path to libs in script are

/usr/lib/x86_64-linux-gnu/

but in Red Hat based distros like Fedora, they are in

/usr/lib64

Comment on lines +33 to +34
# export MESA_VERSION=$(glxinfo | grep "OpenGL version string" | sed 's/OpenGL version string: .* Mesa \(.*\)/\1/')
# export MESA_LOADER_DRIVER_OVERRIDE=i965
Copy link
Contributor

Choose a reason for hiding this comment

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

I guess we should avoid commented out code, unless there's some documentation explaining 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.

Create Linux launch script
4 participants