Created attachment 74602 [details] weston-info output The weston demo weston-screenshooter doesn't work anymore in 1.0.4. I investigated it a bit and the function display_handle_mode is not called to set the output width and height and because of this the set_buffer_size fails. I tried fixing it by adding a wl_display_dispatch and wl_display_roundtrip before set_buffer_size, but display_handle_mode is still not called.
Closing this bug as we figured out in irc that Alexander was trying to run the screenshot command from the command line. weston-screenshot is a helper application that weston launches when the user takes a screenshot with the super+s key binding.
Hmm, but did weston_screenshooter not complain about permission denied? If it didn't, might be worth to fix it. Maybe weston-screenshooter forgets to enable libwayland-client error reporting? This would also be a nice case to expect a certain protocol error in code, catch it, and print a friendly error message in plain English. I don't recall if we actually have a way to pull protocol error specifics from libwayland-client.
[krh@tokamak mesa]$ WAYLAND_DEBUG=1 /usr/libexec/weston-screenshooter [808109.600] -> wl_display@1.get_registry(new id wl_registry@2) [808109.934] wl_registry@2.global(1, "wl_display", 1) [808110.076] wl_registry@2.global(2, "wl_compositor", 2) [808110.182] wl_registry@2.global(3, "screenshooter", 1) [808110.295] -> wl_registry@2.bind(3, "screenshooter", 1, new id [unknown]@3) [808110.421] wl_registry@2.global(4, "text_cursor_position", 1) [808110.522] wl_registry@2.global(5, "text_model_factory", 1) [808110.631] wl_registry@2.global(6, "wl_data_device_manager", 1) [808110.727] wl_registry@2.global(7, "wl_shm", 1) [808110.839] -> wl_registry@2.bind(7, "wl_shm", 1, new id [unknown]@4) [808110.993] wl_registry@2.global(8, "wl_seat", 1) [808111.097] wl_registry@2.global(9, "input_method", 1) [808111.196] wl_registry@2.global(10, "wl_output", 1) [808111.303] -> wl_registry@2.bind(10, "wl_output", 1, new id [unknown]@5) [808111.444] wl_registry@2.global(11, "wl_drm", 2) [808111.545] wl_registry@2.global(12, "wl_shell", 1) [808111.640] wl_registry@2.global(13, "desktop_shell", 1) [808111.772] wl_registry@2.global(14, "screensaver", 1) [808111.876] wl_registry@2.global(15, "input_panel", 1) [808111.975] wl_registry@2.global(16, "workspace_manager", 1) [808112.079] -> wl_display@1.sync(new id wl_callback@6) [808112.284] wl_display@1.error(screenshooter@3, 0, "screenshooter failed: permission denied") but without WAYLAND_DEBUG it just exits silently. You're right, we should print a warning to tell the user it's not supposed to be run directly.
Added a warning (and changed bug resolution to NOTABUG): commit c4a42ca6b50ac0543b62e7967b615ac6d3de5289 Author: Kristian Høgsberg <krh@bitplanet.net> Date: Wed Feb 13 13:40:58 2013 -0500 screenshot: Print warning if we're not launched from weston weston-screenshooter is a helper binary that weston launches to write the screenshot to disk. If somebody tries to launch it by hand, print a warning and mention the screenshot keybinding.
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.