Re: svn commit: r660749 - /httpd/httpd/branches/2.2.x/STATUS

2008-06-14 Thread Ruediger Pluem



On 05/28/2008 01:20 PM, Nick Kew wrote:

On Tue, 27 May 2008 22:26:09 -
[EMAIL PROTECTED] wrote:


+   rpluem says: Thanks. Sorry for being picky, but after revisiting
the patch
+   again I ask myself how we can be sure that the chroot function is
actually
+   present on the platform we compile.


$ man chroot
...
CONFORMING TO
   SVr4, SVID, 4.4BSD, X/OPEN.

How many unix platforms support none of the above?


I guess not many. So lets backport it and see if we get a bug report
where this fails to build. This may also help to create a good test
for detecting it correctly



Would #ifdef HAVE_UNISTD_H be an appropriate wrapper for this,
or could you have a unistd without chroot?



I don't know.

Regards

RĂ¼diger


Re: svn commit: r660749 - /httpd/httpd/branches/2.2.x/STATUS

2008-06-08 Thread Ruediger Pluem



On 05/28/2008 01:20 PM, Nick Kew wrote:

On Tue, 27 May 2008 22:26:09 -
[EMAIL PROTECTED] wrote:


+   rpluem says: Thanks. Sorry for being picky, but after revisiting
the patch
+   again I ask myself how we can be sure that the chroot function is
actually
+   present on the platform we compile.


$ man chroot
...
CONFORMING TO
   SVr4, SVID, 4.4BSD, X/OPEN.

How many unix platforms support none of the above?



My man chroot says

CONFORMING TO
   SVr4, 4.4BSD, SUSv2 (marked LEGACY).  This  function  is  not  part  of
   POSIX.1-2001.

So it does not seem to be part of POSIX. But I have no strong opinion. Maybe
another developer could chime in and push me to one or another side :-).

Regards

RĂ¼diger


Re: svn commit: r660749 - /httpd/httpd/branches/2.2.x/STATUS

2008-05-28 Thread Nick Kew
On Tue, 27 May 2008 22:26:09 -
[EMAIL PROTECTED] wrote:

 +   rpluem says: Thanks. Sorry for being picky, but after revisiting
 the patch
 +   again I ask myself how we can be sure that the chroot function is
 actually
 +   present on the platform we compile.

$ man chroot
...
CONFORMING TO
   SVr4, SVID, 4.4BSD, X/OPEN.

How many unix platforms support none of the above?

Would #ifdef HAVE_UNISTD_H be an appropriate wrapper for this,
or could you have a unistd without chroot?

-- 
Nick Kew

Application Development with Apache - the Apache Modules Book
http://www.apachetutor.org/