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

Cagebreak crashes with mpv --fullscreen #78

Open
sodface opened this issue May 4, 2024 · 2 comments
Open

Cagebreak crashes with mpv --fullscreen #78

sodface opened this issue May 4, 2024 · 2 comments

Comments

@sodface
Copy link

sodface commented May 4, 2024

Running the latest development branch of Cagebreak on Alpine with wlroots 0.17.2. I don't normally run mpv with --fullscreen but I was working on something and added it for troubleshooting and was surprised that it caused Cagebreak to crash back to the tty.

Steps to reproduce:

This works:

mpv --idle --load-scripts=no --force-window

And just adding --fullscreen causes a crash:

mpv --idle --load-scripts=no --force-window --fullscreen

I tested starting the same mpv command with Cage, both from tty and from within a nested session inside of Cagebreak and both of those work fine.

Log entries:

 err: wayland.c:1448: failed to read events from the Wayland socket: Broken pipe
 err: wayland.c:2024: failed to roundtrip Wayland display: Broken pipe
 err: wayland.c:2024: failed to roundtrip Wayland display: Broken pipe
 err: wayland.c:2024: failed to roundtrip Wayland display: Broken pipe
N 21:22:49.487721  m-dbus-connection ../modules/module-dbus-connection.c:138:on_connection_closed: <WpDBusConnection:0x7f6074684870> DBus connection closed: Underlying GIOStream returned 0 bytes on an async read
warn: terminal.c:1831: slave exited with signal 1 (Hangup)
 err: wayland.c:1989: failed to flush wayland socket: Broken pipe

Can anyone reproduce?

@project-repo
Copy link
Owner

Hi sodface

Thanks for reporting this! We can indeed reproduce this and have
uploaded a fix to the development branch. Can you check whether this
resolves the issue?

Cheers
project-repo

@sodface
Copy link
Author

sodface commented May 4, 2024

Tested and working, thank you!

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

No branches or pull requests

2 participants