Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 12 Nov 2014 14:07:23 -0800
From:      NGie Cooper <yaneurabeya@gmail.com>
To:        Warner Losh <imp@freebsd.org>
Cc:        "svn-src-head@freebsd.org" <svn-src-head@freebsd.org>, "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>, "src-committers@freebsd.org" <src-committers@freebsd.org>, "bdrewery@freebsd.org" <bdrewery@freebsd.org>
Subject:   Re: svn commit: r274186 - head/share/mk
Message-ID:  <CAGHfRMCGwUb8wQGriaSJYoE4PpH4PXcW7uFJW8WEBNjoucTUiw@mail.gmail.com>
In-Reply-To: <D1251D73-0B14-41D5-B2C8-C2D82453CE29@gmail.com>
References:  <201411061719.sA6HJgFu062138@svn.freebsd.org> <D1251D73-0B14-41D5-B2C8-C2D82453CE29@gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Nov 6, 2014 at 9:21 AM, Garrett Cooper <yaneurabeya@gmail.com> wrot=
e:
> On Nov 6, 2014, at 9:19, Warner Losh <imp@freebsd.org> wrote:
>
>> Author: imp
>> Date: Thu Nov  6 17:19:41 2014
>> New Revision: 274186
>> URL: https://svnweb.freebsd.org/changeset/base/274186
>>
>> Log:
>>  Ignore errors from rm -rf to support high -j builds. This is, at best,
>>  a kludge. However, it also effectively works around the issues for
>>  high -j builds on systems that do not have the rm fixes.
>>
>>  A better fix would be to rmdir here, and fix the places where we're
>>  sloppy and not list all the files we create in CLEANFILES, should
>>  anybody have the time to chase them all to ground.
>
> I=E2=80=99ll say that bsd.progs.mk is a huge problem here because CLEANFI=
LES is handled <number of PROGS> times.
>
> Dealing with bsd.progs.mk and its recursive nature is quite entertaining.=
..

Another question related to this commit...

Why aren't return codes ignored for rm -f ${CLEANFILES} ?

bdrewery@ ran into an issue at $work where the build failed after
r268376 because he ran into ESTALE with rm:

13:51 <@bdrewery> rm: fts_open: Stale NFS file handle

This commit will allow the build to continue when dealing with ESTALE
on directories, but it won't fix the case where we hit ESTALE with
CLEANFILES.

Thanks!



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAGHfRMCGwUb8wQGriaSJYoE4PpH4PXcW7uFJW8WEBNjoucTUiw>