~spvkgn/whipper/+git/whipper:bugfix/issue-453-name-too-long

Last commit made on 2020-01-19
Get this branch:
git clone -b bugfix/issue-453-name-too-long https://git.launchpad.net/~spvkgn/whipper/+git/whipper

Branch merges

Branch information

Name:
bugfix/issue-453-name-too-long
Repository:
lp:~spvkgn/whipper/+git/whipper

Recent commits

151f6c8... by JoeLametta <email address hidden>

Truncate path length before it gets too long

Fixes #453.

Signed-off-by: JoeLametta <email address hidden>

553a6de... by JoeLametta <email address hidden>

Fix typo in README and clarify Docker instructions

Fixes #452.

Signed-off-by: JoeLametta <email address hidden>

caa2c8b... by JoeLametta <email address hidden>

Bug: whipper shouldn't abort if track rip succeeds on last allowed retry attempt

Fixes #449.

Signed-off-by: JoeLametta <email address hidden>

9e37219... by JoeLametta <email address hidden>

Merge pull request #436 from ABCbum/grab-cover-art

Grab cover art from MusicBrainz/Cover Art Archive and add it to the resulting whipper rips

e2942b0... by ABCbum <email address hidden>

Add test case to check getCoverArt's functionality

Mock two functions `getCoverArt`, `get_image_front` and use
a locally available cover art to check if the created cover
art exists.

Problems:
- How to check image's quality.
- Not sure if only this check is enough (do we need to check the
embedding part?).

Signed-off-by: ABCbum <email address hidden>

8181cac... by ABCbum <email address hidden>

Update README, dependencies and supporting files for cover art feature

Signed-off-by: ABCbum <email address hidden>
Co-authored-by: JoeLametta <email address hidden>
Signed-off-by: JoeLametta <email address hidden>

f61214a... by ABCbum <email address hidden>

Support fetching cover art images from the Cover Art Archive

Add option `--cover-art` to `whipper cd rip` command which accepts three values:
- `file`: save the downloaded cover image as standalone file in the rip folder (named `cover.jpg`)
- `embed`: embed the download cover image into all the ripped audio tracks (no standalone file will be kept)
- `complete`: save standalone cover image as standalone file and embed it into all the ripped audio tracks (`file` + `embed`)

Every cover art is fetched from the Cover Art Archive as JPEG thumbnail with a maximum dimension of 500px.
Other supported values for the thumbnails are 250, 500 and 1200 (currently only some images have a corresponding 1200px sized thumbnail).

This feature introduces an optional dependency on the `Pillow` module which is required for the decoding of the cover file (required by the `embed` and `complete` option values).

Problem:
- EmbedPicTureTask shouldn't be a task.

Signed-off-by: ABCbum <email address hidden>
Co-authored-by: JoeLametta <email address hidden>
Signed-off-by: JoeLametta <email address hidden>

6a43d7d... by JoeLametta <email address hidden>

Update copyright year in README

Misc README changes too.

Signed-off-by: JoeLametta <email address hidden>

150f0d5... by JoeLametta <email address hidden>

Move inline comment to separate line in example whipper config file

This avoids `%` character interpolation leading to `InterpolationSyntaxError`.
Added a comment explaining this too.

Fixes #443.

Signed-off-by: JoeLametta <email address hidden>

fb9fb34... by JoeLametta <email address hidden>

Move comment to the right place

The blank line after the comment was added in commit 644e67f1056024c75544500e4e48d9e80ea54f90.

Signed-off-by: JoeLametta <email address hidden>