Mir

Code review comment for lp://staging/~robertcarr/mir/android-test-powerd-cli

Revision history for this message
Daniel van Vugt (vanvugt) wrote :

OK, so it's not really a fix for bug 1239955 as much as a workaround.

I share Kevin's concern and wonder if this workaround applies to any relevant case where we really need it. I mean, don't our armhf builds run on headless or at least non-Android systems anyway? And if it's for developers then it doesn't help them either because they will run the tests directly (not debian/rules).

P.S. The final chunk looks like it's doing nothing effectively.

review: Needs Information

« Back to merge proposal