Repository navigation
docs(desktop): point the GTK4 launch-event workaround at wailsapp/wails#6000 (GDK-295) - #43
Merged
Merged
Conversation
… (GDK-295) The cold-start argv workaround existed because wails' default Linux host never emits ApplicationLaunchedWithUrl. That hole is now filed and fixed upstream as wailsapp/wails#6000 — runtime-proven A/B in a headless container, GTK3 non-regression included. Point every comment that states the hole at that number, and say what becomes possible when a pin containing it lands: Linux moves to DeferToEvent like Windows. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
midagedev
force-pushed
the
docs/gdk295-upstream-ref
branch
from
August 19, 2026 07:26
0759490 to
33ccd2c
Compare
midagedev
added a commit
that referenced
this pull request
Aug 19, 2026
… must fail fast and named (GDK-317) (#44) Three runs died on the same step today (PR #43 twice, main 6295922 once), and none of them in the tests: `npx playwright install-deps chromium` hung with zero output from 07:35:54 until the step's 22-minute kill. The download half has had bounded retries all along; the apt half was deliberately unbounded, on the reasoning that a wrapper would replace a named error with a generic one. Run 32228508406 measured the inversion: the unbounded call is what produced the generic timeout. Now both halves are bounded the same way — 420s (1.6x the slowest measured healthy apt) twice, a lock re-wait between attempts, and a failure line that still names which half died and why it is not the tests. Step budget re-derived to 27 minutes so our own error always outruns the runner's. FAIL-first is the measured hang itself; this cannot be reproduced locally (ubuntu runner apt), which is why it goes through a PR. Co-authored-by: midagedev <midagedev@users.noreply.github.com> Co-authored-by: Claude Opus 5 <noreply@anthropic.com>
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.
Comment-only. desktop/ hits the two CI jobs that cannot run locally, so this goes through a PR per the merge rule.
desktop/main.go: the three comments that state "GTK4 does not emit the event" now carry the upstream fix's address (fix(v3/linux): emit launch URL and file-association events on GTK4 wailsapp/wails#6000) and the follow-up (move Linux to DeferToEvent when a pin containing it lands).desktop/README.md: the platform table row for linux points at the same PR.Local gates:
go build ./...0,go vet0 (both modules),desktop go test ./... -count=1ok.🤖 Generated with Claude Code