Recursive get (sync) broken for empty folders.
is wrong as zero results will be returned for an empty folder and getting the folder will not compare correctly to a File.
Also why aren't we using synapseutils.walk here instead of replicating the code.
Yes, I don't think we should not warn if we overwrite files in the cache. The cache should not be touched directly by the user.
Running `syncFromSynapse` on a project that contains an empty folder:
Verified that the problem exists in the Python client 1.8.1.
Error from calling `syncFromSynapse()` on an empty folder:
For non-empty folder:
Note that the warning is very confusing:
Does anyone else think that this is confusing?