mps-youtube / pafy

Python library to download YouTube content and retrieve metadata
1.39k stars 313 forks source link

Use Python built-in `os.rename` to move files #251

Closed ritiek closed 4 years ago

ritiek commented 4 years ago

Using subprocess to call mv assumes that the shell command mv exists and its operation is to move files. This move operation would fail if mv doesn't exist or has been modified to perform some other operation.

Using Python's built-in os.rename is more reliable.