[00:45] -GitHub[m]:#mir-server- **[MirServer/mir]** wmww opened [pull request #2506](https://github.com/MirServer/mir/pull/2506): Add SurfaceReadyRegistrar to replace SurfaceReadyObserver... (full message at https://libera.ems.host/_matrix/media/r0/download/libera.chat/6adc9a3e020cceff5d5332aa9f14c23de0378bee)
[00:46] -GitHub[m]:#mir-server- **[MirServer/mir]** wmww edited [pull request #2506](https://github.com/MirServer/mir/pull/2506): Add SurfaceReadyRegistrar to replace SurfaceReadyObserver
[17:07] <grayson-g[m]> Has anyone else been having issues rebuilding the wayland-generator acceptance tests? Looking at the CMakeLists it seems that the target is `wayland_generator_test_generated_files`, but that isn't doing it. I tried `touch`ing the expected.cpp/h and building again to no avail.
[17:09] <grayson-g[m]> Also, I'm curious if anyone knows why we have the Wayland protocol repeated in the codebase (once in `src/wayland/protocol/wayland.xml' and another in `tests/acceptance-tests/wayland-generator/protocol.xml`. It seems like the `protocol.xml` is only used in building the Wayland generator tests. Couldn't we just build the tests from `wayland.xml`? I'm assuming if not it's because of some CMake hackery that I don't know about.
[17:09] <grayson-g[m]> s///, s///, s/`?/\`?/
[17:10] <grayson-g[m]>  * Also, I'm curious if anyone knows why we have the Wayland protocol repeated in the codebase (once in src/wayland/protocol/wayland.xml and another in tests/acceptance-tests/wayland-generator/protocol.xml. It seems like the `protocol.xml`is only used in building the Wayland generator tests. Couldn't we just build the tests from`wayland.xml\`? I'm assuming if not it's because of some CMake hackery that I don't know about.
[17:10] <grayson-g[m]>  * Also, I'm curious if anyone knows why we have the Wayland protocol repeated in the codebase (once in src/wayland/protocol/wayland.xml and another in tests/acceptance-tests/wayland-generator/protocol.xml.) It seems like the `protocol.xml`is only used in building the Wayland generator tests. Couldn't we just build the tests from`wayland.xml\`? I'm assuming if not it's because of some CMake hackery that I don't know about.
[17:32] <grayson-g[m]> Update: looks like `expected.cpp/h` is building successfully and being placed in `build/tests/acceptance-tests/wayland-generator/` but isn't copying as determined in `build/bin/update_wayland_generator.sh`. Weird.
[19:12] <sophie-w> Hmm, sorry, I don't see anything obvious. Are you still having problems?
[19:17] <grayson-g[m]> I just manually copied the file over and all is gravy now. No clue why that would be happening. I read over the scripts and everything is looking right? It got all my paths correct and everything.
[20:18] -GitHub[m]:#mir-server- **[MirServer/mir]** graysonguarino assigned graysonguarino to [issue #2499](https://github.com/MirServer/mir/issues/2499): Upgrade to wl_seat v7
[20:28] -GitHub[m]:#mir-server- **[MirServer/mir]** wmww edited [pull request #2371](https://github.com/MirServer/mir/pull/2371): Modernize surface observer
[23:12] -GitHub[m]:#mir-server- **[MirServer/mir]** n3rdopolis opened [issue #2507](https://github.com/MirServer/mir/issues/2507): Mir tries to link clients with wrong Wayland libraries since Cmake 3.22+... (full message at https://libera.ems.host/_matrix/media/r0/download/libera.chat/636f8a0c05c8f36426ab7dd579a44d2cd67f566a)
[23:12] -GitHub[m]:#mir-server- **[MirServer/mir]** n3rdopolis edited [issue #2507](https://github.com/MirServer/mir/issues/2507): Mir tries to link clients with wrong Wayland libraries since Cmake 3.22+