Skip to content

Bump pinned winit to fix X11 request_user_attention panic - #80

Draft
warp-agent-staging[bot] wants to merge 1 commit into
factory/benchmark-base-winit-wm-hintsfrom
benchmark-run-97-trial-6858-attempt-1/01a0dac0-e5b0-7d21-913f-d853343a4ca8/pin-winit-x11-attention-fix
Draft

warp-agent-staging[bot] wants to merge 1 commit into
factory/benchmark-base-winit-wm-hintsfrom
benchmark-run-97-trial-6858-attempt-1/01a0dac0-e5b0-7d21-913f-d853343a4ca8/pin-winit-x11-attention-fix

Conversation

@warp-agent-staging

Copy link
Copy Markdown

What

Bumps the pinned winit dependency to the revision that fixes the X11 crash
behind Sentry WARP-CLIENT-BETA-STABLE-89S1.

On Linux/X11, ringing the terminal bell for an inactive window calls
Window::request_user_attention, which set the window's WM hints in a way that
panicked when the request failed to send on a dead X11 connection, aborting
Warp. The winit fix treats that send failure as recoverable (logs and continues).

Change

  • Cargo.toml: point the winit git dependency at
    05e9fc9a6cabe9e39c8062740e56319f8b3204b5 (was a4e0ecb5…).
  • Cargo.lock: regenerated the winit and dpi entries for the new revision
    via cargo update -p winit --precise.

Companion winit change: warpdotdev/winit#36

Verification

  • cargo update -p winit --precise 05e9fc9a… resolved and fetched the new
    revision cleanly, updating both lockfile entries.
  • The winit fix itself compiles and passes clippy with the x11 feature
    (see the companion PR).

Point the winit git dependency at the revision that stops
Window::request_user_attention from panicking when the WM-hints request
fails to send on a dead X11 connection (Sentry WARP-CLIENT-BETA-STABLE-89S1).

Updates the winit and dpi entries in Cargo.lock to match.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants