aag / apple_trailer_downloader

A script to download HD trailers from the Apple Trailers website.
GNU General Public License v3.0
23 stars 5 forks source link

Suppress information about already downloaded trailers #10

Closed forthrin closed 8 years ago

forthrin commented 8 years ago

Please add an option so that only information the following information is printed:

This means suppressing the following information:

This will make it easier to glean what has happened since last time. It will also have the nice side effect that when running with cron, the user will only get an email when there are new trailers to check out.

aag commented 8 years ago

Hi forthrin,

I just pushed a commit that adds the output_level option. To have the script only print downloaded trailers, you can run the script with the command-line option -o downloads or put output_level = downloads in your config file.

Please give it a try and let me know if it's good enough for your needs.

forthrin commented 8 years ago

Seems to work like a charm! Maybe there should be only one line per movie to make it easier to read, but it's sort of nice to have both the URL and the filename, so...

aag commented 8 years ago

I went ahead and reduced the output to one line per file at the "downloads" output level. It now prints the movie name and type as well as the filename.

forthrin commented 8 years ago

Well, since the title and filename seem to be identical, I suggest simply:

logging.info('Downloading ' + trailerUrl['type'] + ': ' + trailerFileName)
aag commented 8 years ago

Well, since the title and filename seem to be identical, I suggest simply:

I went ahead and changed it to the format you suggested.