fix(linux): isolate AppImage GStreamer plugins to prevent ABI mismatch#444
Merged
fix(linux): isolate AppImage GStreamer plugins to prevent ABI mismatch#444
Conversation
…plugins The AppImage bundles GStreamer from CI (Ubuntu 24.04, GStreamer 1.24). Previously, host plugin directories (/usr/lib/gstreamer-1.0/) were appended as fallback. This caused ABI version mismatches — host plugins compiled against a different GStreamer version fail with undefined symbol errors (gst_util_floor_log2, mpg123_open_handle64, etc.), leaving WebKit without usable codecs for YouTube playback. Since PR #434 installs the full GStreamer codec suite on CI, the AppImage is fully self-contained. Remove the host fallback and block host plugin scanning to prevent ABI conflicts across distro GStreamer versions.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Merged
facusturla
pushed a commit
to facusturla/worldmonitor
that referenced
this pull request
Feb 27, 2026
…plugins (koala73#444) The AppImage bundles GStreamer from CI (Ubuntu 24.04, GStreamer 1.24). Previously, host plugin directories (/usr/lib/gstreamer-1.0/) were appended as fallback. This caused ABI version mismatches — host plugins compiled against a different GStreamer version fail with undefined symbol errors (gst_util_floor_log2, mpg123_open_handle64, etc.), leaving WebKit without usable codecs for YouTube playback. Since PR koala73#434 installs the full GStreamer codec suite on CI, the AppImage is fully self-contained. Remove the host fallback and block host plugin scanning to prevent ABI conflicts across distro GStreamer versions.
markewicz
pushed a commit
to markewicz/worldmonitor
that referenced
this pull request
Feb 27, 2026
CSS changes (main.css, settings-window.css): - Add @media (prefers-reduced-motion: reduce) — kills all 60+ infinite CSS animations for users who opt in, saves battery on low-end devices - Raise --text-faint #555→#777, --text-ghost koala73#444→#666 for WCAG-passing contrast on dark background (~3:1 from <2:1) - Remove max-height: 400px on mobile panels — content was silently clipped; panels now scroll internally with overflow-y: auto - Constrain .intel-findings-dropdown to min(380px, 92vw) on ≤480px screens — previously overflowed off right edge on narrow phones - Bump header height 40px→48px on mobile for comfortable touch target - Raise .variant-option, .findings-badge, .layer-toggle font-size from 9px to 11px on mobile for readability on high-DPI screens - Add min-height: 44px to settings tabs and runtime pills (WCAG 2.5.5) JS changes (CountersPanel, DeckGLMap, 4 data panels, badge): - CountersPanel: gate 60fps RAF loop via IntersectionObserver + document visibilitychange — loop pauses when panel is off-screen or tab hidden, resumes automatically; add matching cleanup in destroy() - DeckGLMap: replace 500ms setInterval news-pulse with a RAF loop that checks elapsed time (same 500ms cadence) — cooperates with browser frame scheduling, skips work when tab is hidden or render is paused - MacroSignalsPanel, StablecoinPanel, ETFFlowsPanel, ServiceStatusPanel: add !document.hidden && !this.element.classList.contains('hidden') guard so background-tab and off-screen panels skip their periodic fetches - IntelligenceGapBadge: skip 10s refresh when tab is hidden; clamp context menu coordinates to viewport bounds (prevents off-screen menu) https://claude.ai/code/session_01PgapzQLrB8RisivM6WvCz7
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
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.
Summary
GST_PLUGIN_SYSTEM_PATH_1_0as safety netProblem
Users on various Linux distros reported GStreamer errors and no YouTube playback:
Root cause: The AppImage bundles GStreamer 1.24 from CI (Ubuntu 24.04), but the code appended host plugin directories as fallback. Host plugins compiled against a different GStreamer version (older or newer) fail with undefined symbol errors due to ABI mismatch. This left WebKit with zero usable codec plugins.
Why this is safe now
PR #434 added the full GStreamer codec suite (base, good, bad, ugly, libav, gl) to CI. With
bundleMediaFramework: true, the AppImage bundles all necessary plugins. No host fallback needed.Separate issue: Exec format error (aarch64 on x86_64)
One user also reported
Exec format errorwhen running an aarch64 AppImage — this is an architecture mismatch (wrong download), not a code bug.Test plan
Failed to load pluginGStreamer warnings from host paths