[regression] Clients log verbose info on startup: "Loader: Loading module: ..."
Bug #1414883 reported by
Daniel van Vugt
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Mir |
Fix Released
|
Medium
|
Daniel van Vugt | ||
| mir (Ubuntu) |
Fix Released
|
Medium
|
Unassigned | ||
Bug Description
Clients log to stdout (against the wishes of the client/toolkit author):
[1422330274.158057] Loader: Loading modules from: bin/../
[1422330274.158235] Loader: Loading module: bin/../
[1422330274.158542] Loader: Loading module: bin/../
[1422330274.158658] Loader: Loading module: bin/../
This is something we need to avoid doing. You should only interrupt someone else's output stream if you have a serious error to report.
Related branches
lp:~vanvugt/mir/log-level
- Alan Griffiths: Needs Fixing
- PS Jenkins bot (community): Approve (continuous-integration)
- Chris Halse Rogers: Needs Fixing
- Robert Carr (community): Abstain
- Mir development team: Pending requested
-
Diff: 720 lines (+138/-83)29 files modifiedcommon-ABI-sha1sums (+1/-1)
debian/control (+2/-2)
debian/libmircommon4.install (+1/-1)
examples/glog_logger.cpp (+9/-0)
examples/glog_logger.h (+4/-0)
include/common/mir/logging/logger.h (+10/-0)
platform-ABI-sha1sums (+1/-1)
server-ABI-sha1sums (+1/-1)
src/client/default_connection_configuration.cpp (+5/-2)
src/client/logging/input_receiver_report.cpp (+2/-1)
src/client/logging/perf_report.cpp (+1/-0)
src/client/logging/rpc_report.cpp (+7/-6)
src/common/CMakeLists.txt (+1/-1)
src/common/logging/dumb_console_logger.cpp (+9/-1)
src/common/logging/dumb_console_logger.h (+3/-2)
src/common/logging/logger.cpp (+3/-3)
src/common/symbols.map (+2/-1)
src/server/default_server_configuration.cpp (+5/-2)
src/server/display_server.cpp (+2/-0)
tests/include/mir_test_doubles/mock_logger.h (+44/-0)
tests/include/mir_test_doubles/null_logger.h (+1/-0)
tests/mir_test_doubles/null_logger.cpp (+4/-0)
tests/unit-tests/android_input/input_reader.cpp (+5/-17)
tests/unit-tests/graphics/mesa/test_display.cpp (+3/-8)
tests/unit-tests/graphics/test_graphics_platform.cpp (+1/-5)
tests/unit-tests/logging/message_processor_report.cpp (+2/-8)
tests/unit-tests/logging/test_compositor_report.cpp (+4/-1)
tests/unit-tests/logging/test_display_report.cpp (+3/-10)
tests/unit-tests/logging/test_legacy_input_report.cpp (+2/-9)
lp:~vanvugt/mir/no-common-log
- Chris Halse Rogers: Abstain
- Alan Griffiths: Needs Fixing
- Kevin DuBois (community): Approve
- Alexandros Frantzis (community): Approve
- PS Jenkins bot (community): Approve (continuous-integration)
-
Diff: 12 lines (+1/-1)1 file modifiedsrc/client/default_connection_configuration.cpp (+1/-1)
Revision history for this message
| Daniel van Vugt (vanvugt) wrote | #1 |
| Changed in mir: | |
| assignee: | nobody → Daniel van Vugt (vanvugt) |
| Changed in mir: | |
| milestone: | 0.11.0 → 0.12.0 |
Revision history for this message
| PS Jenkins bot (ps-jenkins) wrote | #2 |
| Changed in mir: | |
| status: | In Progress → Fix Committed |
| Changed in mir: | |
| milestone: | 0.12.0 → 0.13.0 |
| Changed in mir (Ubuntu): | |
| importance: | Undecided → Medium |
| status: | New → Triaged |
| Changed in mir: | |
| status: | Fix Committed → Fix Released |
Revision history for this message
| Launchpad Janitor (janitor) wrote | #3 |
| Changed in mir (Ubuntu): | |
| status: | Triaged → Fix Released |
To post a comment you must log in.
The regression started at (I think):
-------
revno: 2254 [merge]
author: Andreas Pokorny <email address hidden>
committer: Alan Griffiths <email address hidden>
branch nick: development-branch
timestamp: Fri 2015-01-23 16:25:00 +0000
message:
Add server-side platform probing support.
Modules are checked in MIR_SERVER_
Modules are probed to determine whether they can drive the available hardware. If a real platform - Android or Mesa - is capable of driving the hardware then it is used; otherwise the stub platform is loaded.
-------