Insufficient platform headers are now published for external driver projects
Bug #1367435 reported by
Kevin DuBois
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Mir |
Confirmed
|
Medium
|
Kevin DuBois | ||
mir (Ubuntu) |
Confirmed
|
Medium
|
Unassigned |
Bug Description
Insufficient platform headers are now published for external driver projects to use.
Related branches
lp://staging/~kdub/mir/no-mirtestdraw
- Alberto Aguirre (community): Approve
- Alan Griffiths: Approve
- PS Jenkins bot (community): Approve (continuous-integration)
- Cemil Azizoglu (community): Approve
-
Diff: 837 lines (+125/-379)16 files modifiedexamples/CMakeLists.txt (+0/-3)
examples/render_overlays.cpp (+46/-11)
examples/testdraw/mesa_graphics_region_factory.cpp (+0/-42)
tests/integration-tests/CMakeLists.txt (+1/-1)
tests/integration-tests/client/CMakeLists.txt (+0/-8)
tests/integration-tests/graphics/android/CMakeLists.txt (+6/-0)
tests/integration-tests/graphics/android/android_graphics_region_factory.cpp (+43/-77)
tests/integration-tests/graphics/android/graphics_region_factory.h (+9/-11)
tests/integration-tests/graphics/android/patterns.cpp (+5/-5)
tests/integration-tests/graphics/android/patterns.h (+0/-4)
tests/integration-tests/graphics/android/test_buffer_integration.cpp (+6/-9)
tests/integration-tests/graphics/android/test_client_render.cpp (+9/-10)
tests/integration-tests/graphics/android/testdraw/CMakeLists.txt (+0/-34)
tests/unit-tests/CMakeLists.txt (+0/-2)
tests/unit-tests/draw/CMakeLists.txt (+0/-5)
tests/unit-tests/draw/test_draw_patterns.cpp (+0/-157)
Changed in mir: | |
milestone: | none → 0.10.0 |
status: | Triaged → In Progress |
Changed in mir: | |
milestone: | 0.10.0 → none |
To post a comment you must log in.
I get your point. And we will need to re-expose some platform headers.
However while no one is using those headers we're using that as a reasonable excuse to say "nobody's use of the platform ABI has been broken by this header change".
So I think yes, carry on with exposing headers as required. But this is not a bug.