Date: Sat, 23 Jul 2005 17:01:20 -0700 From: Doug Barton <dougb@FreeBSD.org> To: Pawel Worach <pawel.worach@gmail.com> Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src ObsoleteFiles.inc Message-ID: <42E2DA50.2000205@FreeBSD.org> In-Reply-To: <42E2A029.1090404@gmail.com> References: <200507231942.j6NJgdks037508@repoman.freebsd.org> <42E2A029.1090404@gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
Pawel Worach wrote:
> While you are at it can you add this one too.
Done. Please note for next time that you need to add a comment indicating
why the file was removed. This can easily be found from the CVS logs.
BTW, this is exactly why I don't like this mechanism for cleaning stale
files. This list was, as I predicted it would be, quite literally out of
date when it was committed. This is with all due respect to the effort that
went into producing it. It's the methodology that I'm opposed to here.
I much prefer the dynamic method suggested by myself, mezz, and others which
scans the directories and compares the ages of the files to a known value.
This not only has the benefit of not needing a static list to support it,
but it also has the benefit of alerting you to pieces left behind when you
(for example) add a NO_FOO knob to your make.conf file to avoid building
part of the world.
I would really like to see us reexamine the thought process behind this
before we invest a lot more time into the static method. I think that the
dynamic method will buy us more down the road.
hth,
Doug
--
This .signature sanitized for your protection
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?42E2DA50.2000205>