-
-
Notifications
You must be signed in to change notification settings - Fork 123
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
crash running example on linux with wayland #29
Comments
Thanks for the report. I haven't tried the example on Wayland. Looks like we can't get a current monitor for a hidden window on Wayland. I wonder if this is a winit bug? Searching their issue tracker didn't come up with anything. |
Seems to also happen with x11 on manjaro with gnome https://gist.github.com/caemor/c2c0a3dbf0a4fa83cdbd0c81b82546e2 |
That's another good data point. The only thing I have to add is that it works on Windows and macOS. @caemor Unfortunately that backtrace does not tell us much. Can you get one from a debug build? |
Yes here is the output with more errors from the debug build: Debug Backtrace
|
@caemor You have some interesting errors from vulkan. Out of curiosity, have you tried the So that's not actually the same bug as what was originally reported. I've opened #36 we can continue the conversation there. Thanks for the infos! |
I've run into the same bug trying to put together a simple test program:
Here's the code: use pixels::{wgpu::Surface, Pixels, SurfaceTexture};
use winit::event_loop::EventLoop;
use winit::window::Window;
fn main() -> Result<(), Box<dyn std::error::Error>> {
let event_loop = EventLoop::new();
let window = Window::new(&event_loop)?;
let surface = Surface::create(&window);
let size = window.inner_size().to_physical(window.hidpi_factor());
let width = size.width.round() as u32;
let height = size.height.round() as u32;
let surface_texture = SurfaceTexture::new(width, height, surface);
let pixels = Pixels::new(width, height, surface_texture);
} Dependencies:
|
Thanks @joshtriplett That looks like #36 |
To followup here, this can be worked around by forcing WINIT_UNIX_BACKEND=x11 cargo run --release --example minimal-winit --features minimal-winit |
I believe rust-windowing/winit#1190 will fix this issue. |
- Rather than setting the position and window size based on available screen space, this PR just creates a window with a default size and doesn't attempt to set the position. - Closes #29
- Rather than setting the position and window size based on available screen space, this PR just creates a window with a default size and doesn't attempt to set the position. - Closes #29
hello, just thought I'd file this here:
The text was updated successfully, but these errors were encountered: