autoconf-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: AC_PROG_MKDIR_P documentation


From: Stepan Kasal
Subject: Re: AC_PROG_MKDIR_P documentation
Date: Sun, 4 Jun 2006 13:51:13 +0200
User-agent: Mutt/1.4.2.1i

Hello,

On Thu, Jun 01, 2006 at 08:23:27PM +0200, Ralf Wildenhues wrote:
> I went ahead and checked these BSD systems:
> http://www.freebsd.org/cgi/cvsweb.cgi/src/bin/mkdir/mkdir.c
> http://cvsweb.netbsd.org/bsdweb.cgi/src/bin/mkdir/mkdir.c
> http://www.openbsd.org/cgi-bin/cvsweb/src/bin/mkdir/mkdir.c
> 
> All of them support -m.
> 
> FreeBSD has a race-free mkdir -p since rev. 1.23 of mkdir.c, but I
> wouldn't trust it before rev. 1.25, dated 2002, in FreeBSD 5.0.
> 
> NetBSD has a race-free mkdir -p since rev. 1.28 of mkdir.c, but I
> wouldn't trust it before rev. 1.29, dated 2003, in NetBSD 2.0.2.
> 
> OpenBSD has a race-free mkdir -p since rev. 1.7 of mkdir.c, 1998,
> OpenBSD 2.4, but looking at the source code, I found a small
> bug in it (which is not so relevant to intended usage though):
> http://cvs.openbsd.org/cgi-bin/query-pr-wrapper?full=yes&numbers=5141

thank you, Ralf, for the review.  It seems that autoconf.texi needs
to be corrected; patch attached.

> > > If yes, is there any way to detect them, too?

> (matching right from `uname -rs' output seems to ugly and
> maintenance-intensive to me),

After our private communication, I'm fully convinced.  We don't want
to go down that path.

(One problem is, that we don't know about a generic way to
distinguish real *BSD from GNU/k*bsd systems which contain only the
BSD kernel.)

> Hmm, could take to AC_REQUIRE([AC_CANONICAL_HOST]) and match from
> there [...] but that may all be overkill.  Not sure.

In this light, I'd rather back out my proposal.  If GNU mkdir is not
found, just use install-sh, and that's it.

OK to commit the above doc patch?

Have a nice day,
        Stepan

Attachment: autoconf-20060604-bsd.patch
Description: Text document


reply via email to

[Prev in Thread] Current Thread [Next in Thread]