Date: Sun, 9 Jun 2002 20:14:45 -0700 (PDT) From: Matthew Dillon <dillon@apollo.backplane.com> To: Mike Barcroft <mike@FreeBSD.ORG> Cc: "Greg 'groggy' Lehey" <grog@FreeBSD.ORG>, cvs-committers@FreeBSD.ORG, cvs-all@FreeBSD.ORG Subject: Re: cvs commit: src/include pwd.h Message-ID: <200206100314.g5A3EjTt005317@apollo.backplane.com> References: <200206091939.g59JdJC05285@freefall.freebsd.org> <20020610004026.GD61036@wantadilla.lemis.com> <200206100049.g5A0nr1P004846@apollo.backplane.com> <20020609211243.C51371@espresso.q9media.com>
next in thread | previous in thread | raw e-mail | index | archive | help
:>
:> For password related calls, the manual page also quite clearly states
:> that <sys/types.h> must be included before <pwd.h>
:
:Yes, this was how old BSD/POSIX worked. It's very different in new
:standards. My changes won't affect any BSD software and only help
:portability.
:
:See my reply to Greg for more details.
:
:> For these reasons I would strongly recommend either reverting this change
:> and leaving at as it was, or removing <sys/types.h> from <pwd.h>
:> and *NOT* adding weird #ifdef's for gid_t, uid_t, etc, as the manual
:> page implies.
:
:I don't hear you complaining about those weird #ifdef's in
:<sys/types.h> :). But seriously, Bruce's designed replacement will
:get rid of those once I've implemented it.
:
:Best regards,
:Mike Barcroft
Which specification are you looking at? I'm all for conforming to
standards but this is new to me.
In anycase, if you are going to remove the sys/types.h dependancy
for pwd.h (and other headers) then you also need to document that in
the related manual pages.
-Matt
Matthew Dillon
<dillon@backplane.com>
To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200206100314.g5A3EjTt005317>