diff options
| author | Josh Gao <jmgao@google.com> | 2015-11-11 15:10:53 -0800 |
|---|---|---|
| committer | Josh Gao <jmgao@google.com> | 2015-11-13 15:17:02 -0800 |
| commit | f96dc73b9f218f464640a9cdd80c389723665d5c (patch) | |
| tree | 28db6d65ec5c47c27c44152e595dcb28ec02077b /adb/file_sync_client.cpp | |
| parent | f131772a79f75656d9e7593268e5bdecf40ebec0 (diff) | |
adb: don't pull symlinks.
Pulling symlinks was broken for directories, and it doesn't seem like
there's a good way to make it not broken, given that the protocol doesn't
have readlink (and we don't want to create symlinks on Windows, anyway).
The behavior for files doesn't seem to be especially useful, either.
Bug: http://b/25601283
Change-Id: Ie1d27e93dd09cbc0c178623b390041d1cb11f726
Diffstat (limited to 'adb/file_sync_client.cpp')
| -rw-r--r-- | adb/file_sync_client.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/adb/file_sync_client.cpp b/adb/file_sync_client.cpp index dcf0b26e18..a2d2a66bfa 100644 --- a/adb/file_sync_client.cpp +++ b/adb/file_sync_client.cpp @@ -887,8 +887,7 @@ bool do_sync_pull(const std::vector<const char*>& srcs, const char* dst, continue; } - if (S_ISREG(src_mode) || S_ISLNK(src_mode)) { - // TODO(b/25601283): symlinks shouldn't be handled as files. + if (S_ISREG(src_mode)) { std::string path_holder; if (dst_isdir) { // If we're copying a remote file to a local directory, we |