mitsuhiko / platter

A useful helper for wheel deployments.
Other
335 stars 52 forks source link

changing os.rename to shutil.move on --format=dir parameter #28

Open dutradda opened 9 years ago

dutradda commented 9 years ago

on os.rename I found two errors: when dest directory is not empty raises: 'OSError: [Errno 39] Directory not empty' but with shutil.move it overwrite the directory.

when dest dir is not in the same hd partition of /tmp raises: 'OSError: [Errno 18] Invalid cross-device link' but with shutil.move it dont occurs.

by Diogo Dutra dutradda@gmail.com

dutradda commented 9 years ago

you already take a look on my fix? @mitsuhiko

dutradda commented 8 years ago

@mitsuhiko can you merge my pull request?