dgorissen / coursera-dl

A script for downloading course material (video's, pdfs, quizzes, etc) from coursera.org
http://dirkgorissen.com/2012/09/07/coursera-dl-a-coursera-download-script/
GNU General Public License v3.0
1.74k stars 299 forks source link

Add an option to download only given extensions #160

Closed SebastianLopienski closed 9 years ago

SebastianLopienski commented 9 years ago

I'm usually interested in downloading only PDF files. Now I have to use -n option to ignore other files - but I have to list (and know) all other file extensions.

To address this use case, why don't we have an additional option, e.g.

  -i DOWNLOADFILES      comma-separated list of file extensions to download, e.g. "ppt,pdf"

... that would take precedence over the existing -n option:

  -n IGNOREFILES        comma-separated list of file extensions to skip, e.g. "ppt,srt,pdf"

What do you think? I can implement it if @dgorissen and others find it useful (=will merge :-)

dgorissen commented 9 years ago

Sure :)

SebastianLopienski commented 9 years ago

Here you go: #161 :-)

SebastianLopienski commented 9 years ago

I saw you already merged it; do you know when this update will be available via pip?

dgorissen commented 9 years ago

Sorry, Im swamped. If you submit PR with updated version & changelog files I will merge and update pip


Dr. Dirk Gorissen Research - Tech4Good - Flying Robots Twitter : https://twitter.com/dirkgor LinkedIn: http://www.linkedin.com/in/dirkgorissen

On Tue, Nov 25, 2014 at 7:45 AM, SebastianLopienski < notifications@github.com> wrote:

I saw you already merged it; do you know when this update will be available via pip?

— Reply to this email directly or view it on GitHub https://github.com/dgorissen/coursera-dl/issues/160#issuecomment-64321209 .

SebastianLopienski commented 9 years ago

Here you go again: #162 :-)

dgorissen commented 9 years ago

done