Repository navigation
feat(linux): openSUSE setup support + WebKitGTK detection fix - #31
Merged
Merged
Conversation
The WebKitGTK pre-check ran `ldconfig -p` through $PATH only. openSUSE keeps ldconfig in /usr/sbin, which is not on a normal user's PATH, so the installer reported "tinyjs needs the WebKitGTK runtime" on machines that have it. Try ldconfig, /sbin/ldconfig and /usr/sbin/ldconfig, then fall back to pkg-config and the library file when no ldconfig is runnable at all. Add openSUSE to the missing-runtime hints.
setup.sh now reads /etc/os-release and names the right dev packages in its error hints (openSUSE: gcc-c++, make, pkgconf-pkg-config, gtk3-devel, webkitgtk3-devel, libayatana-appindicator3-devel — webkitgtk3-devel is WebKitGTK 4.1's dev package there, the "3" meaning GTK3). README, the site install blurbs, the platforms skill reference and TODO-linux get the zypper equivalents for the runtime, build and GStreamer packages.
openSUSE's default repositories ship no AppIndicator/StatusNotifier host package (gnome-shell-extension-appindicator is not there; GNOME users get the extension from extensions.gnome.org), so the zypper install line could never succeed. Leave zypper without a package list so the requirement still reports the missing tray and why.
tarwin
added a commit
that referenced
this pull request
Oct 2, 2026
tarwin
added a commit
that referenced
this pull request
Oct 2, 2026
Owner
|
Thanks @samcharles93! Shipped in v0.44.1. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds openSUSE to the Linux setup path, verified on openSUSE Tumbleweed (x86_64).
setup.shreads/etc/os-releaseand prints the right dev packages per distro. On openSUSE the WebKitGTK 4.1 dev package iswebkitgtk3-devel(the "3" is GTK3, not an older WebKit) — README, TODO-linux, the platforms skill reference and the site install blurbs carry the zypper equivalents for runtime, build and GStreamer packages.docs/installno longer false-fails on openSUSE:ldconfiglives in/usr/sbin, which a normal user's$PATHdoes not include, so the WebKitGTK check reported a miss on machines that have it. It now triesldconfig,/sbin/ldconfig,/usr/sbin/ldconfig, thenpkg-config, then the library file.runtime/bridge.js: dropped the zypper tray hint. openSUSE's default repos ship no AppIndicator host package, so the suggested install could not succeed; the requirement still reports the missing tray and why.Verification:
./setup.shon Tumbleweed: exit 0 — prebuilt tjs v26.6.0 downloaded, launcher compiled,lddresolves AppIndicator, PipeWire, X11 and XTest.sudo zypper install gcc-c++ make pkgconf-pkg-config gtk3-devel webkitgtk3-devel libayatana-appindicator3-devel.ldconfigon$PATH; exits 1 with all four distro hints when the library is absent.sh -n,node --check,git diff --checkclean.