Re: sudo fix

[Jeroen van Wolffelaar <jeroen@wolffelaar.nl>]

On Wed, Jun 29, 2005 at 06:50:51PM +0200, Markus Kolb wrote:
> Jeroen van Wolffelaar wrote on Wed, Jun 29, 2005 at 16:06:59 +0200:
> > 
> > That being said, I failed to find the actual patch/package in the
> > original mail in this thread. Could you please download the fixed
> > packages as referenced in #315115, and (1) check whether they work and
> > not have regressions, and (2) whether there is a difference with the
> > patch prepared by you, and if so, whether that's significant in any way?
> 
> Your patch does the same in fixing #315115 for Sarge.
> In addition your patch changes a few other trivial things not in
> relation to #315115 and adds a few lines to manpages.
> Is this ok for security updates?
> I think, at least it should be mentioned in Changefile.

Huh?

$ debdiff /org/ftp.debian.org/ftp/pool/main/s/sudo/sudo_1.6.8p7-1.1.dsc sudo_1.6.8p7-1.1sarge1.dsc | diffstat
 ldap.c                        |    2 -
 parse.yacc                    |    4 ---
 sudo-1.6.8p7/config.guess     |    6 ++++-
 sudo-1.6.8p7/config.sub       |   11 ++++++---
 sudo-1.6.8p7/debian/changelog |    8 +++++++
 sudo.c                        |   16 ++++----------
 sudo.tab.c                    |   48 +++++++++++++++++++----------------------- 7 files changed, 47 insertions(+), 48 deletions(-)
$

I fail to see the changes you describe, as far as I can see, this is minimal.
sudo.tab.c is autogenerated, hence that is has a bit more changes.

--Jeroen

-- 
Jeroen van Wolffelaar
Jeroen@wolffelaar.nl (also for Jabber & MSN; ICQ: 33944357)
http://Jeroen.A-Eskwadraat.nl