Code review comment for ~paelzer/ubuntu/+source/chrony:merge-4.2-2-jammy

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

Hi Utkarsh,
0526a15 and all the others you asked for were forwarded to Debian and there decided against it.

The reason behind that is that our container story around LXD being a system container is very different to theirs.

Without that motivation I can understand why they didn't want but we needed it.
So that is one of the "fair and ok on both sides" differences between Debian & Ubuntu.

All the other bits you have wondered belong to that context e.g. libcap2-bin is only needed for our container check. The call to the wrapper is only needed for our container handling, ...

---

Finally about d/ch "Drop patches present in v4.2" might seem short, but you'll see that in the former version we have used exactly that term. And all that I could do in the new changelog is repeating that file names which seemed superlfuous.
If you want them added it is just a copy of lines 90-92 up to 29 and done.
So let me know if you think this is really better.

« Back to merge proposal