| Saviq | Morn' | 08:09 |
|---|---|---|
| alan_g[m] | Saviq, any reviews you'd like me to do in exchange for https://github.com/MirServer/mir/pull/2825? | 09:46 |
| Saviq | alan_g will trade you this https://github.com/MirServer/spread-mir-ci/pull/4 | 09:48 |
| Saviq | alan_g so found one difference between previous (pre-last-refactor) and current behaviour - X11 outputs are now named `OUT-1` rather than `unknown-1` | 10:26 |
| Saviq | Which is likely fine, just highlighting if this is unexpected | 10:26 |
| alan_g[m] | I hadn't noticed, but it maintains the "unique names" invarient, so it is fine. | 10:28 |
| Saviq | I mean, it's a fringe enough use case. But apart from them being unique, we want them stable… | 10:29 |
| Saviq | (outside of the x11 platform, at least) | 10:29 |
| Saviq | Which sounds to me like we don't have tests that guarantee that. | 10:29 |
| -GitHub[m]:#mir-server- **[MirServer/mir]** Saviq marked [pull request #2822](https://github.com/MirServer/mir/pull/2822): Add fedora rawhide as ready for review | 10:31 | |
| alan_g[m] | True, we don't. I'd argue pre-existing and non-trivial (as the code is currently in the KMS support) | 10:32 |
| -GitHub[m]:#mir-server- **[MirServer/mir]** Saviq opened [issue #2826](https://github.com/MirServer/mir/issues/2826): Add tests guaranteeing stable output names | 10:34 | |
| -GitHub[m]:#mir-server- | 10:34 | |
| -GitHub[m]:#mir-server- > We currently have no tests that guarantee output names, which may mean we break users with static display configs. | 10:34 | |
| -GitHub[m]:#mir-server- enhancement | 10:34 | |
| -GitHub[m]:#mir-server- **[MirServer/mir]** bors[bot] merged [pull request #2825](https://github.com/MirServer/mir/pull/2825): More refactoring of display configuration | 12:10 | |
| -GitHub[m]:#mir-server- **[MirServer/mir]** AlanGriffiths opened [pull request #2827](https://github.com/MirServer/mir/pull/2827): Add a side-by-side layout to the .display configuration | 12:46 | |
| -GitHub[m]:#mir-server- | 12:46 | |
| -GitHub[m]:#mir-server- > When creating a .display configuration include multiple layouts (`all_at_00` and `side_by_side`) | 12:46 | |
| Saviq | alan_g how would you say we should deal with this warning?... (full message at <https://libera.ems.host/_matrix/media/v3/download/libera.chat/ddbfc081c48418fe5a8782f71515ca23f9456270>) | 14:15 |
| Saviq | FWIW this is new with GCC 13 on Rawhide: | 14:16 |
| Saviq | https://github.com/MirServer/mir/actions/runs/4123170278/jobs/7122373300#step:8:4948 | 14:16 |
| Saviq | https://gcc.gnu.org/bugzilla/show_bug.cgi?id=106393 | 14:17 |
| alan_g[m] | I agree that the lifetime is (with non-local knowledge) OK. This seems a suitable case for a #pragma. But maybe worthwhile checking if it still errors if we switch to a pointer? | 14:24 |
| Saviq | Will do. | 14:27 |
| -GitHub[m]:#mir-server- **[MirServer/mir]** Saviq unassigned graysonguarino from [issue #640](https://github.com/MirServer/mir/issues/640): The cursor does not support attaching an icon (e.g. for Drag-n-Drop) | 16:29 | |
| -GitHub[m]:#mir-server- **[MirServer/mir]** Saviq assigned to [pull request #2827](https://github.com/MirServer/mir/pull/2827): Add a side-by-side layout to the .display configuration | 16:31 | |
| -GitHub[m]:#mir-server- **[MirServer/mir]** Saviq self-requested a review for [pull request #2827](https://github.com/MirServer/mir/pull/2827): Add a side-by-side layout to the .display configuration | 16:33 | |
| -GitHub[m]:#mir-server- **[MirServer/wlcs]** Saviq unassigned graysonguarino from [issue #266](https://github.com/MirServer/wlcs/issues/266): Test wl_output features up to version 4 | 16:34 | |
| -GitHub[m]:#mir-server- **[MirServer/ubuntu-frame]** Saviq unassigned graysonguarino from [issue #102](https://github.com/MirServer/ubuntu-frame/issues/102): If the containing directory of `diagnostic-path` is deleted and recreated content is not displayed | 16:34 | |
| -GitHub[m]:#mir-server- **[MirServer/mir]** Saviq assigned to [pull request #2822](https://github.com/MirServer/mir/pull/2822): Add fedora rawhide | 16:41 | |
| -GitHub[m]:#mir-server- **[MirServer/wlcs]** i509VCB opened [issue #273](https://github.com/MirServer/wlcs/issues/273): position_window_absolute might not work in all window managment schemes... (full message at <https://libera.ems.host/_matrix/media/v3/download/libera.chat/76e969c4e56c18863d36b6992521a53eb7bd7f63>) | 16:48 | |
| -GitHub[m]:#mir-server- **[MirServer/ubuntu-frame-osk]** bors[bot] edited [pull request #49](https://github.com/MirServer/ubuntu-frame-osk/pull/49): [Merged by Bors] - snap: pin libfeedback at 0.0.1 | 16:52 | |
| -GitHub[m]:#mir-server- **[MirServer/ubuntu-frame-osk]** bors[bot] closed [pull request #49](https://github.com/MirServer/ubuntu-frame-osk/pull/49): [Merged by Bors] - snap: pin libfeedback at 0.0.1 | 16:52 | |
| -GitHub[m]:#mir-server- **[MirServer/ubuntu-frame-osk]** bors[bot] edited [pull request #48](https://github.com/MirServer/ubuntu-frame-osk/pull/48): snap: refresh dependencies | 16:52 | |
| -GitHub[m]:#mir-server- **[MirServer/ubuntu-frame-osk]** bors[bot] edited [pull request #47](https://github.com/MirServer/ubuntu-frame-osk/pull/47): edge: add handling of the `edge` branch | 16:52 | |
| Saviq | 👋 | 17:04 |
| -GitHub[m]:#mir-server- **[MirServer/mir]** bors[bot] merged [pull request #2822](https://github.com/MirServer/mir/pull/2822): Add fedora rawhide | 17:46 | |
| -GitHub[m]:#mir-server- **[MirServer/ubuntu-frame-osk]** bors[bot] edited [pull request #47](https://github.com/MirServer/ubuntu-frame-osk/pull/47): [Merged by Bors] - edge: add handling of the `edge` branch | 18:37 | |
| -GitHub[m]:#mir-server- **[MirServer/ubuntu-frame-osk]** bors[bot] closed [pull request #47](https://github.com/MirServer/ubuntu-frame-osk/pull/47): [Merged by Bors] - edge: add handling of the `edge` branch | 18:37 | |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!