miral::ActiveOutputsMonitor (and therefore miral-shell --window-manager tiling) are broken by lp:mir
Bug #1645284 reported by
Alan Griffiths
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Mir |
Fix Released
|
Medium
|
Alan Griffiths | ||
MirAL |
Fix Released
|
High
|
Alan Griffiths | ||
mir (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
The change from DisplayConfigur
This is because they need call "server.
The internal reports avoid this by being wired into the DefaultServerCo
Related branches
lp://staging/~alan-griffiths/mir/add-mir-Server-add_pre_init_callback
- Daniel van Vugt: Approve
- Kevin DuBois (community): Approve
- Cemil Azizoglu (community): Needs Fixing
- Andreas Pokorny (community): Approve
- Mir CI Bot: Approve (continuous-integration)
-
Diff: 89 lines (+30/-2)3 files modifiedinclude/server/mir/server.h (+6/-1)
src/server/server.cpp (+16/-0)
src/server/symbols.map (+8/-1)
lp://staging/~alan-griffiths/miral/fix-1645284
- Gerry Boland (community): Approve
-
Diff: 12 lines (+2/-1)1 file modifiedmiral/active_outputs.cpp (+2/-1)
Changed in miral: | |
status: | New → In Progress |
importance: | Undecided → High |
assignee: | nobody → Alan Griffiths (alan-griffiths) |
milestone: | none → 0.5 |
summary: |
- Downstream DisplayConfigurationObservers cannot get the initial display - configuration + miral::ActiveOutputsMonitor (and therefore miral-shell --window-manager + tiling) are broken by lp:mir |
Changed in miral: | |
status: | In Progress → Fix Committed |
Changed in miral: | |
status: | Fix Committed → Fix Released |
Changed in mir: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
Hmm. This might not be a Mir limitation after all...
Maybe it is only that MirAL doesn't have a mechanism to allow registration between calling "server- >apply_ settings( );" and "server->run();"