X-Authentication-Warning:	delorie.com: mail set sender to djgpp-workers-bounces using -f
X-Recipient:	djgpp-workers AT delorie DOT com
Date:	Wed, 30 Apr 2014 20:24:51 +0300
From:	Eli Zaretskii <eliz AT gnu DOT org>
Subject:	Re: [PATCH] break xstat.c into pieces
In-reply-to:	<CAA2C=vD-a_Tx+ZzseEuZU_-U7eJYEsLFNwWVdKLmm85_7_N9qA@mail.gmail.com>
X-012-Sender:	halo1 AT inter DOT net DOT il
To:	djgpp-workers AT delorie DOT com
Message-id:	<83r44elogs.fsf@gnu.org>
References:	<CAA2C=vBuWHmVNrRAcGDSXYLc8Eg_xsyMVszPzrxqHGrpVq2vmw AT mail DOT gmail DOT com> <83wqeh9pwq DOT fsf AT gnu DOT org> <CAA2C=vD_97x3-A0Y52JDfbmmFiuVzF_Ufub6imhmuhTsdsNr4w AT mail DOT gmail DOT com> <83vbu19o4f DOT fsf AT gnu DOT org> <CAA2C=vDAWcPpCex4koZXSdorEid7w1=QTAWip7t0n-exMH+vrQ AT mail DOT gmail DOT com> <83r44p9mjr DOT fsf AT gnu DOT org> <CAA2C=vBWH49B_zu-1rAqxBvHoxKzZL+pqOANTE0H3MPQ2OkROg AT mail DOT gmail DOT com> <CAA2C=vD-a_Tx+ZzseEuZU_-U7eJYEsLFNwWVdKLmm85_7_N9qA AT mail DOT gmail DOT com>
Reply-To:	djgpp-workers AT delorie DOT com
Errors-To:	nobody AT delorie DOT com
X-Mailing-List:	djgpp-workers AT delorie DOT com
X-Unsubscribes-To:	listserv AT delorie DOT com

> Date: Wed, 30 Apr 2014 19:50:29 +0300
> From: Ozkan Sezer <sezeroz AT gmail DOT com>
> 
> On 4/22/14, Ozkan Sezer <sezeroz AT gmail DOT com> wrote:
> > On 4/22/14, Eli Zaretskii <eliz AT gnu DOT org> wrote:
> [...]
> > OK, leaving things to you guys.
> >
> 
> PING:
> 
> If there is still interest in this

The interest never dwindled, thanks for persevering.

> I've been using the two patches inlined below (also attached) for
> some time and they work well for me.
> 
> 
> * mkdir.c: replace access() call with _chmod(). this loses the extra
> _fixpath()+remote rootdir check done by access(), but it shortens
> things by not pulling in extra dependencies.

What does this set errno to when the argument is "C:/" or some other
root directory?

> --- /dev/null
> +++ src/libc/posix/sys/stat/statbits.c	24 Apr 2014 07:51:10 -0000

I really don't see why a single short variable should have its own
source file.  I think I suggested to add the variable to crt1.c.
Could you please do that instead?

Thanks.

- Raw text -