Comment 45 for bug 1890858

Revision history for this message
Seth Arnold (seth-arnold) wrote :

Wow, thanks so much for the archaeology, Christian. That removal commit explains so much.

I prefer (b) for this case:

- fewer moving pieces in the change is more likely to be reliable
- adding abstractions/nameservice to a profile that didn't already have it is a fairly large increase in permissions
- if the code in question is going to be gone in ten years, 'planning for the future' is much less important
- apparently this functionality isn't widely used in Focal, otherwise we would have seen more than this.

It's a bit strange to come to a different conclusion the next day; I could probably be talked into (a) if someone else feels strongly about it. But the simpler approach is my preference now.

Thanks