Code review comment for ~lucaskanashiro/ubuntu/+source/postfix:xenial-lp-1881196

Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

Fix is reasonable, small and seems correct.
CL entries LGTM

The one thing I have to call out is that the SRU Description still talks about the full minor release backport. Which is correct for focal, but wrong for Bionic/Xenial where you only backport one change.

Therefore you get my +1 (generally a good fix), but under the condition to add a secondary SRU template in the bug that is for B/X releases describing that situation.

review: Approve

« Back to merge proposal