Subject: Re: CVS commit: src/lib/libc/gen
To: Elad Efrat <elad@NetBSD.org>
From: Quentin Garnier <cube@cubidou.net>
List: source-changes
Date: 12/31/2005 02:04:13
--jr/gb2Ce1GM9KKZD
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Sat, Dec 31, 2005 at 02:17:00AM +0200, Elad Efrat wrote:
> Quentin Garnier wrote:
>=20
> > Please re-re-re-revert (or whatever) that change.
>=20
> Test attached patch.

Well, that fixes potential issues from leaving errno in a weird value,
but that doesn't fix the fact that we don't return NULL and set errno to
ENOENT when a path component doesn't exist.

That was the very subject of lib/30943.  In the end you agreed on the
behaviour realpath(3) should have, and suddenly you're changing your
mind because it's convenient for you?  How reasonable is that?

At least, re-open the PR when you have such a mind change.  Inverting
the behaviour of a function does not help in keeping bugs away IMO.
Quite the contrary.

--=20
Quentin Garnier - cube@cubidou.net - cube@NetBSD.org
"When I find the controls, I'll go where I like, I'll know where I want
to be, but maybe for now I'll stay right here on a silent sea."
KT Tunstall, Silent Sea, Eye to the Telescope, 2004.

--jr/gb2Ce1GM9KKZD
Content-Type: application/pgp-signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.6 (NetBSD)

iQEVAwUBQ7XZDdgoQloHrPnoAQKMBgf/ZS34i5fbWOeN67FuJPjzXjmm0Ai1r5mQ
83zxCU6a0BecR9DK9ZlRFBimeFuuTJPzAAFlFJ+y9Tln0lzu6F1a0e8U8ORCYDNP
DqQ1RkNMJ/+cKpwTkSdg8+xfN7JLbOZVKrTUVhqVQAONgbq+aTypNlTDG+hr8YuW
z93RBF8WwbbcpNBUhNmAaz6l6eGAVWWLZKjrX+AVc2Q7YJUpngXiVxToA9Utv+DM
3qWWdF5JYthv7TNt2eem4tJljT8FQmMXzMX+xFyCfy67iC1ddBI5i/L+mRDTMbIA
OIzFWheD5SlS1Q+omV48k2v9D5uschZcXdPTy0piJMVEOHEl//bU7A==
=Zk71
-----END PGP SIGNATURE-----

--jr/gb2Ce1GM9KKZD--