Commit Graph

1559 Commits

Author SHA1 Message Date
Peter Taylor
4fb9d99ddd Add missing 'Album Artist' tag when its value is 'Various Artists'
Fixes #518.

Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-12-25 14:30:00 +00:00
Alex Jones
e59872716e Parameterise the UID of the worker user in the docker build file. (#517)
* Parameterise the UID of the worker user in the docker build file.

Signed-off-by: Alex Jones <alex.jones@unclealex.co.uk>

* Remove spurious use of ARG uid so that it only appears after the FROM statement.

Signed-off-by: Alex Jones <alex.jones@unclealex.co.uk>
2020-11-28 21:07:43 +01:00
JoeLametta
c229c01a58 Replace 'freedb.dbpoweramp.com' CDDB server with gnudb.org
It seems gnudb.org allows submissions too while 'freedb.dbpoweramp.com' is read only.

Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-11-28 17:44:21 +00:00
JoeLametta
4181c455ca Remove mention about Exherbo from README
Now the package is included in the badge (Repology).

Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-11-28 17:18:37 +00:00
JoeLametta
660b28fd90 Travis CI: replace Python 3.9 dev with 3.9 release
Removed it from the 'allow_failures' block too.

Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-11-19 15:48:34 +00:00
JoeLametta
3e61c3dc1b Fix 'list index out of range' bug
It was introduced in commit acf942b5b6.

The error happens because 'self.itable.tracks' is a list of tracks
(zero-based numbering, HTOA excluded) so the first track can be accessed with
'self.itable.tracks[0]' but when 'number' has value 0, that matches the HTOA
(everything is shifted by one compared to 'self.itable.tracks').

It's unrelated to this bugfix but I've also moved some instructions outside
the try ... except clause.

Fixes #512.

Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-11-19 15:36:48 +00:00
JoeLametta
04ff005806 Fix bug in cdrom drive status handling
The bare if evaluated to true for return codes > 0 and that's wrong (CDS_DISC_OK = 4).

Fixes #511.

Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-10-12 12:20:53 +00:00
JoeLametta
d885e79c9f Merge pull request #510 from MasterOdin/patch-1
Fix capitalization of "Health status" in rip log
2020-09-24 09:33:18 +00:00
Matthew Peveler
a8942a8037 Update test_result_logger.log
Signed-off-by: Matthew Peveler <matt.peveler@gmail.com>
2020-09-24 03:58:32 -04:00
Matthew Peveler
1015d6e000 Fix capitalization of "Health status" in rip log
This fixes a regression from move to ruamel.yaml where all other fields in rip log have second word lowercased, except for "Health Status". This changes it to make it inline with all other fields again.

Signed-off-by: Matthew Peveler <matt.peveler@gmail.com>
2020-09-24 03:57:45 -04:00
JoeLametta
ae361b8345 Man: add 'drive-auto-close' option values
Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-09-24 07:40:26 +00:00
JoeLametta
c9e1e17175 Update Travis CI's dist to Ubuntu focal and fix setuptools_scm version string generation
Replaced cdparanoia with cd-paranoia too.

Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-09-24 07:29:38 +00:00
JoeLametta
f98c995aed Add notes to README's Docker section
Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-09-23 18:19:41 +00:00
JoeLametta
3c9e75c3bc Merge pull request #509 from whipper-team/feature/issue-320-isrc-audio-tag
Tag audio tracks with ISRCs (if available)
2020-09-23 19:56:54 +02:00
JoeLametta
84cc824f2e Merge pull request #505 from whipper-team/feature/issue-488-drive-auto-close
Allow configuring whether to auto close the drive's tray
2020-09-23 19:54:59 +02:00
JoeLametta
21185f40a9 Merge pull request #507 from whipper-team/bugfix/issue-385-missing-cd-warning
Provide better error message when there's no CD in the drive
2020-09-23 19:52:50 +02:00
JoeLametta
302bc22a66 Merge pull request #506 from whipper-team/ux/issue-495-cdparanoia-warnings
Add checks and warnings for (known) cdparanoia's upstream bugs
2020-09-23 19:51:16 +02:00
JoeLametta
8a1c0fabfc Allow configuring whether to auto close the drive's tray
Fixes #488.

Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-09-23 17:41:48 +00:00
JoeLametta
b754b2b0bf Restore getRipResult method to fix regression
The regression was introduced in commit 3acc3ffed6.
The getRipResult method has been slimmed down to its essence.

Fixes #508.

Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-09-20 13:07:14 +00:00
JoeLametta
acf942b5b6 Tag audio tracks with ISRCs (if available)
Fixes #320.

Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-09-19 20:14:22 +00:00
JoeLametta
1661e4291e Emit warning when the subcode's pre-emphasis flag of a track differs from the ToC's one
In the future I'd like to make sure this information is included in the logfile too (maybe also in the cue sheet).

Related to issue #296.

Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-09-18 11:58:18 +00:00
JoeLametta
145cc47aa8 Fix 'Resource not accessible by integration' for GitHub action
This commit should make it work for pull requests too.
2020-09-17 21:48:17 +02:00
JoeLametta
921e25bf98 Provide better error message when there's no CD in the drive
Fixes #385.

Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-09-17 19:18:27 +00:00
JoeLametta
07bd0348f3 Add checks and warnings for (known) cdparanoia's upstream bugs
Fixes #495.

Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-09-17 18:07:28 +00:00
JoeLametta
3acc3ffed6 Drop whipper caching (#336)
Whipper's caching implementation causes a few issues (#196, #230, [#321 (comment)](https://github.com/whipper-team/whipper/pull/321#issuecomment-437588821)) and complicates the code: it's better to drop this feature.

The rip resume feature doesn't work anymore: if possible it will be restored in the future.

* Remove caching item from TODO
* Delete unneeded files related to caching
* Update 'common/directory.py' & 'test/test_common_directory.py' (caching removal)
* Update 'common/accurip.py' & 'test/test_common_accurip.py' (caching removal)
* Update 'common/program.py' (caching removal)
* Update 'command/cd.py' (caching removal)

This fixes #335, fixes #196 and fixes #230.

Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-09-17 17:52:11 +02:00
JoeLametta
4b5b6e5e2b Fix mistake in .travis.yml
Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-09-14 13:29:16 +00:00
JoeLametta
ae5bb15a5e Discontinue python 3.5 support (EOL reached)
Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-09-14 13:25:03 +00:00
JoeLametta
23c3e62436 Merge pull request #504 from cclauss/patch-1
Travis CI: Add Python 3.9 release candidate 1
2020-09-10 10:12:48 +02:00
Christian Clauss
7947f2cb1f Travis CI: Add Python 3.9 release candidate 1
* Python 3.9 in allow_failures mode until release on Oct. 5th  https://www.python.org/download/pre-releases
* Now that pip has a real dependancy resolver, feed it all requirements in one command
* Fix Travis build config validation issues: os, sudo

Note: Python 3.5 goes EOL next week.
Signed-off-by: cclauss <cclauss@me.com>
2020-09-09 23:08:12 +02:00
JoeLametta
b3aae5f162 Fix Dockerfile syntactic error
Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-08-10 15:05:04 +00:00
JoeLametta
28221adf04 Remove mention about outdated unoffical snap package from README
Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-08-10 14:58:34 +00:00
JoeLametta
164b604cc9 Merge pull request #498 from MasterOdin/patch-1
Define libcdio version as environment variables in docker
2020-08-10 16:49:50 +02:00
JoeLametta
bbed92bb30 Update failing AccurateRipResponse tests
Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-08-10 16:39:55 +02:00
JoeLametta
8676e254e2 Fix CD drive permission issue with Docker (on ArchLinux)
Fixes #499.

Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-08-10 14:07:52 +00:00
Matthew Peveler
8a43568bce Define libcdio version as environment variables in docker
Signed-off-by: Matthew Peveler <matt.peveler@gmail.com>
2020-07-14 13:09:57 -04:00
JoeLametta
0ffa34bc98 Fix wrong 'rm' directory name in Dockerfile
Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-05-30 11:15:11 +00:00
JoeLametta
e685c04e97 Merge pull request #490 from baldurmen/manpages
Add man pages.
2020-05-30 12:58:05 +02:00
Louis-Philippe Véronneau
3c199f109f Document the new docutils optional dep. and link to the man pages
README

Signed-off-by: Louis-Philippe Véronneau <pollo@debian.org>
2020-05-30 10:51:26 +00:00
Louis-Philippe Véronneau
abcdd06713 Add man pages.
Fixes #73.

Signed-off-by: Louis-Philippe Véronneau <pollo@debian.org>
2020-05-30 10:49:07 +00:00
JoeLametta
ec1598e97d Replace 'master' with 'develop' branch in README links
Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-05-30 10:12:07 +00:00
JoeLametta
fa7c50d3a6 Replace 'sys.exit()' and 'exit()' instructions with 'SystemExit()' equivalents
- `SystemExit` doesn't require importing the `sys` module
- `exit()` depends on the `site` module (for this reason its usage is discouraged in production code)

Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-05-30 09:38:52 +00:00
JoeLametta
8802c5482e Improve error message for unconfigured drive offset
Fixes #478.

Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-05-29 16:24:30 +00:00
JoeLametta
0a960d991b Change docker alias in README to use '${HOME}' rather than '~'
Inlcudes another unrelated change to the README.

Fixes #482.

Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-05-29 15:58:05 +00:00
JoeLametta
2fe4292a9b Update README
- Updated bugs information about the `libcdio-utils` package
- Added missing entries to ToC

Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-05-29 14:27:00 +00:00
JoeLametta
752162a434 Fix two flake8 errors
Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-05-29 14:03:27 +00:00
JoeLametta
86fa4a3e77 Correct mistake in previous commit
The mistake caused the following error: "cd: can't cd to libcdio-paranoia-10.2+2.0.0".

Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-04-21 14:10:19 +00:00
JoeLametta
30ab61fee2 Update libcdio-paranoia's version in Dockerfile
Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-04-21 12:11:07 +00:00
JoeLametta
bbf1eba0e4 Replace 'freedb.freedb.org' CDDB server with a mirror
This is motivated by the imminent shut down of freedb.org which will happen on 2020-03-31.

More details here: https://web.archive.org/web/20200331093822/http://www.freedb.org/
And here: https://hydrogenaud.io/index.php?topic=118682

Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-03-31 13:22:21 +00:00
JoeLametta
6b7fc81189 Merge pull request #324 from whipper-team/feature/issue-313-pathfilter-questions
Rewrite PathFilter
2020-02-22 19:10:35 +01:00
JoeLametta
4dc02ec12e Rewrite PathFilter
Added filter options:
- dot (replace leading dot with _)
- posix (replace illegal chars in *nix OSes with _)
- vfat (replace illegal chars in VFAT filesystems with _)
- whitespace (replace all whitespace chars with _)
- printable (replace all non printable ASCII chars with _)

Removed filter options:
- fat (replaced with vfat)
- special

Fixes #313.

Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
2020-02-22 18:01:20 +00:00