Code review comment for ppa-dev-tools:fix-lp2038651-bionic-not-included

Revision history for this message
Bryce Harrington (bryce) wrote :

> supported_esm() currently also includes trusty and xenial. Should we test for
> those as well, or are you worried about the test becoming wrong once those
> become unsupported via esm?

Exactly, this is destined to bitrot but by ignoring those two I'm hoping to put that day off a few years. :-)

For purposes of the code, checking just one of them is going to be sufficient, and bionic will be around the longest of the three.

« Back to merge proposal