Code review comment for lp://staging/~xnox/ubuntu/quantal/sudo/merge

> Upgrading Lucid's sudo to the new merged version asks no questions:

Ah - you're right then, sorry.

> I have left the logic there just for someone who is jumping releases on upgrade
> (e.g. lucid -> quantal or oneiric -> quantal).
> Potentially this is cruft, since an LTS got released & we should be removing
> this duplication of the same config file in two places.

I would actually prefer to see this dropped. Skipping an LTS is definitely not supported for upgrades, and it's better for us to not carry a delta here if we can avoid it, IMHO. But that doesn't have to be done as part of this merge, it can be done later.

> I agree that it's a bug.
> Also sudo_root 8 man page was not installed in the sudo-ldap package on ubuntu.
> People seem to be renaming Vcs-* control fields to XS-Debian-Vcs-*; not sure
> what the consensus is behind this, i.e. is to make debcheckout work on merged
> packages?!

I don't think there's a strong consensus about this. I think it's better to rename the fields so that 'debcheckout' doesn't give wrong results; but I also don't always do this myself.

I've merged this now with only a couple minor changes to the changelog. Thanks!