Re: [PATCH] Some documentation...

On Wed, 20 Apr 2005, David Greaves wrote:

That was the plan.

I agree that "find . -type f | xargs update-cache --add --" in _theory_ is a nice thing to do. But in practice, you want to make sure that find doesn't incldue the ".git" directory and that we always use the canonical names for all files etc etc.

I could do it in the low-level tools (ie do pathname cleanup there), and indeed I did exactly that in the original code sequence. However, it very quickly became obvious that the low-level code really doesn't want to care, and that it's a lot easier to just do it at a higher level when necessary.

For example, if you have to add a sed-script or something that just removes '^./' and "^.git/", then that's trivial to do, and it leaves the core tools with a very clear agenda in life.

		Linus