Re: [PATCH] submodule deinit: clarify work tree removal message

2013-04-17 Thread Jens Lehmann
Am 17.04.2013 07:16, schrieb Junio C Hamano: > Phil Hord writes: > >> On Mon, Apr 1, 2013 at 3:02 PM, Jens Lehmann 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

Re: [PATCH] submodule deinit: clarify work tree removal message

2013-04-16 Thread Junio C Hamano
Phil Hord writes: > On Mon, Apr 1, 2013 at 3:02 PM, Jens Lehmann 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 curren

Re: [PATCH] submodule deinit: clarify work tree removal message

2013-04-16 Thread Phil Hord
On Mon, Apr 1, 2013 at 3:02 PM, Jens Lehmann 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 dein

[PATCH] submodule deinit: clarify work tree removal message

2013-04-01 Thread Jens Lehmann
The output of "git submodule deinit sub" of a populated submodule prints rm 'sub' as the first line unless used with the -f option. The "rm 'sub'" line is exactly the same output the user gets when using "git rm sub" (because that command is used with the --dry-run option under the hood to det