Phil Hord <phil.h...@gmail.com> writes:
> On Mon, Apr 1, 2013 at 3:02 PM, Jens Lehmann <jens.lehm...@web.de> wrote:
>> Okay, so here is the patch for that. If someone could point out
>> a portable and efficient way to check if a directory is already
>> empty I would be happy to use that to silence the "Cleaned
>> directory" message currently printed also when deinit is run on
>> an already empty directory.
>
> isemptydir() {
> test -d "$(find $1 -maxdepth 0 -empty)"
> }
Hrm, -maxdepth and -empty are not even in POSIX. Folks on GNU
platforms and BSDs (I checked NetBSD 6 and OpenBSD 5.2) should be
fine, but it makes other platforms unhappy.
What is this check used for? To avoid running "rmdir" on non-empty
ones? Saying "cleaning foo/" (or "cleaned foo/") when foo/ is
already empty is not a crime; not omitting an empty one may actually
be a better behaviour from the point of view of repeatability and
uniformity.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
