Re: [PATCH] submodule deinit: clarify work tree removal message
- From
Phil Hord <phil.hord@gmail.com>
- Date
- Apr 16, 2013, 13:32 UTC
- Message-ID
- <CABURp0rkeYc7K0COhc1+96Q2Ox2TaNRpMSmQYOiwBpgPJbsafA@mail.gmail.com>
- In-Reply-To
- <5159D9A8.30901@web.de>
On Mon, Apr 1, 2013 at 3:02 PM, Jens Lehmann <Jens.Lehmann@web.de> wrote:
Show 5 quoted lines
> 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)"
}Sorry for the late reply. I see this patch is already in master (which is fine with me).
Thanks, Phil