Skip to content

Commit a0a1739

Browse files
committed
fs: transfer: don't return on first exception
Regression from #180
1 parent ebc1e3e commit a0a1739

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

src/dvc_objects/fs/generic.py

+3-2
Original file line numberDiff line numberDiff line change
@@ -293,7 +293,7 @@ def transfer(
293293
# copy, the remaining copy operations will be batched.
294294
for i, (from_p, to_p) in enumerate(zip(from_path, to_path)):
295295
if links[0] == "copy":
296-
return copy(
296+
copy(
297297
from_fs,
298298
from_path[i:],
299299
to_fs,
@@ -302,6 +302,7 @@ def transfer(
302302
batch_size=batch_size,
303303
on_error=on_error,
304304
)
305+
continue
305306
try:
306307
_try_links(
307308
links,
@@ -325,7 +326,7 @@ def transfer(
325326
and isinstance(exc.__context__, FileExistsError)
326327
):
327328
logger.debug("'%s' file already exists, skipping", to_path)
328-
return None
329+
continue
329330

330331
if on_error is not None:
331332
on_error(from_p, to_p, exc)

0 commit comments

Comments
 (0)