I've additionally verified that, at least on projects such as projectile
itself,
git ls-tree -zr --name-only HEAD
and
git ls-files -zco --exclude-standard

have the same output.  Actually, what I didn't realize is that even in
subdirectories, the output appears to be the same.  I had thought that git
ls-files would only list files off of the root, not the current working
directory, but that doesn't appear to be the case.

So, unless I discover some other reason, I think the current way the code
is, is fine.

Thanks for fixing my code, and I'm sorry about the oversight.




On Fri, Apr 12, 2013 at 5:07 PM, Andrew Hyatt <ahyatt@gmail.com> wrote:

> Looks like you can use the "-r" option with git ls-tree.  I'd recommend
> putting it back with that option, otherwise the new functionality will not
> work properly, and the repo files will not be restricted to the specified
> subdirectories.
>
>
> On Fri, Apr 12, 2013 at 5:04 PM, Andrew Hyatt <ahyatt@gmail.com> wrote:
>
>> If I recall correctly, the original git command needed to be changed,
>> since it only was supposed to be run from the root.  I needed a command
>> that can work off of subtrees from the root.  Let me investigate if
>> ls-files can be made to work recursively.
>>
>>
>> On Fri, Apr 12, 2013 at 5:58 AM, Bozhidar Batsov <
>> notifications@github.com> wrote:
>>
>>> In projectile.el:
>>>
>>> > @@ -281,12 +284,12 @@ PROJECT-ROOT.")
>>> >              (when (and (projectile-project-p) projectile-enable-caching)
>>> >                (projectile-cache-current-file))))
>>> >
>>> > -(defcustom projectile-git-command "git ls-files -zco --exclude-standard"
>>> > +(defcustom projectile-git-command "git ls-tree -z --name-only HEAD"
>>>
>>> This command does not give you a recursive listing of directory
>>> contents, so I've replaced it with the original one.
>>>
>>> —
>>> Reply to this email directly or view it on GitHub<https://github.com/bbatsov/projectile/pull/88/files#r3769584>
>>> .
>>>
>>
>>
>