Mir

Code review comment for lp:~alan-griffiths/mir/fix-1526658

Revision history for this message
Alan Griffiths (alan-griffiths) wrote :

> There's an additional fix I want to do on this bug this week:
> Add debian packaging magic to simply force the broken packages to be
> uninstalled.

Removing these older packages is a workaround that works for most cases. However, a hypothetical parallel installation that requires mesa2 would be broken by forced removal.

Debian packaging magic to ensure that any installed mir-graphics-drivers-desktop/mir-graphics-drivers-android is updated would be useful though.

> Because once you've loaded and tried to probe mir-client-platform-mesa2
> finding out it's not the right one, it's already too late. It has already
> leaked a handle to itself (which will happen with this branch too).

The point of this MP is that we don't probe the badly behaved mesa2 once we've found that mesa4 works.

> That change is much more important than this one. But I haven't reviewed this
> in detail yet.

I disagree. Automating a workaround for our failed support for side-by-side installations is far less important than fixing the support.

« Back to merge proposal