search

kevinrvaz / m3u8-dl

A CLI program to download a video played using a m3u8 playlist. Download it here - https://pypi.org/project/m3u8dl/0.5.2/
MIT License
72 stars 20 forks source link

Add processes flag #47

Closed OliverVermeulen closed 4 years ago

OliverVermeulen commented 4 years ago

What kind of change does this PR introduce? (Bug fix, feature, docs update, etc.)?

It allows users to customise how many processes are used for downloading

What is the current state prior to this change?

It currently defaults to the number of available cores on Linux or 4 on other platforms.

Additional clarity on why this change is necessary:

Closes https://github.com/excalibur-kvrv/m3u8-dl/issues/37

OliverVermeulen commented 4 years ago

https://github.com/excalibur-kvrv/m3u8-dl/pull/46 and this PR are gonna create a bunch of conflicts for whichever is merged second, but I think it makes sense for them to live as separate commits / reviews.

I did think about moving some logic from the download_process.py file to where the file arguments are loaded for both PRs though.