Mir test fails with protobuf3: Protobuf-can-be-reloaded (SEGFAULT)
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Mir |
Invalid
|
High
|
Unassigned | ||
mir (Ubuntu) |
Invalid
|
High
|
Unassigned | ||
protobuf (Ubuntu) |
Fix Released
|
High
|
Alan Griffiths |
Bug Description
builds with 3.0.0-7, then has one failing test:
test 19
Start 19: Protobuf-
19: Test command: /<<BUILDDIR>
19: Test timeout computed to be: 9.99988e+06
19/19 Test #19: Protobuf-
95% tests passed, 1 tests failed out of 19
Total Test time (real) = 125.73 sec
The following tests FAILED:
19 - Protobuf-
Errors while running CTest
Makefile:85: recipe for target 'test' failed
make[2]: *** [test] Error 8
~~~~~~~
The reason this matters:
The principle downstream from Mir (Unity8) loads Mir as one of a number of plugins, several of which use protobuf. That means that we're prone to being loaded, unloaded and reloaded and don't have control of this.
We've encountered problems in this scenario before that have been traced to protobuf so we have a specific test. But other Unity8 plugins may encounter the same issue.
As seen in comment #25 the problem here is entirely down to a change in the behavior of libmirprotobuf-lite - it no longer unloads on dlclose().
This is only a problem because there is code (see patch in comment #20) that assumes that the library will not be used after a call to google:
tags: | added: ftbfs |
Changed in mir (Ubuntu): | |
status: | New → Confirmed |
importance: | Undecided → Critical |
milestone: | none → ubuntu-16.09 |
Changed in mir (Ubuntu): | |
assignee: | nobody → Alan Griffiths (alan-griffiths) |
Changed in mir: | |
importance: | Undecided → Critical |
importance: | Critical → High |
Changed in mir (Ubuntu): | |
importance: | Critical → High |
Changed in mir: | |
milestone: | none → 0.25.0 |
summary: |
- mir fails to build with protobuf3 + Mir test fails with protobuf3: Protobuf-can-be-reloaded (SEGFAULT) |
description: | updated |
Changed in mir: | |
assignee: | nobody → Alan Griffiths (alan-griffiths) |
status: | New → In Progress |
Changed in mir: | |
status: | Confirmed → Invalid |
Changed in mir (Ubuntu): | |
status: | Confirmed → Invalid |
description: | updated |
maybe wait for https:/ /launchpad. net/ubuntu/ +source/ protobuf/ 3.0.0-7
* Added patch by Srivats P to restore New*Callback into google::protobuf
namespace. Required for ostinato (#835435). (Closes: #836200).