Commit Graph

69 Commits

Author SHA1 Message Date
JoeLametta
69bac864ab Raise exception if template has invalid variables
If the template string contains variables which aren't valid for whipper, raise ValueError exception listing all the included unrecognized variables.
I've also corrected the example template configuration lines in the README.

Fixes #279.
2018-11-02 08:05:00 +00:00
JoeLametta
24a0dceb71 Capitalize whipper's team name in README 2018-10-31 08:00:00 +00:00
JoeLametta
f9cb126270 Update README's Docker part with Docker Hub info
Now Docker adopters can use whipper trough Docker Hub's automatically built images.
Thanks to user "anarcat" for proposing and thoroughly explaining how to reach this goal.

Closes #301.
2018-10-27 19:56:26 +02:00
JoeLametta
d25b995e28 Push whipper v0.7.1 release 2018-10-23 08:30:00 +00:00
JoeLametta
a6d2e122e3 Add a requirements.txt file
Fixes #221.
2018-10-23 08:00:00 +00:00
JoeLametta
669b356daa Update README.md
Updated links from JoeLametta to whipper-team
2018-10-21 10:14:01 +02:00
JoeLametta
528b5a18e9 Update README.md
- Updated logger plugins information
- Updated links section (added Matrix)
2018-10-21 08:02:27 +00:00
anarcat
6d7e54f8b2 Fix Docker invocation (#300)
* fix syntax error in Dockerfile

* fix Docker invocation

It makes no sense to pass the `-t` argument twice and we need to tell
Docker where to find the Dockerfile. Also, it makes sense to write the
config file in a standard location. In my tests, I also had to create
the directory by hand otherwise I got this error:

    cdparanoia can defeat the audio cache on this drive.
    Adding drive cache behaviour to configuration file.
    Traceback (most recent call last):
      File "/usr/local/bin/whipper", line 11, in <module>
        load_entry_point('whipper==0.7.0', 'console_scripts', 'whipper')()
      File "/usr/local/lib/python2.7/dist-packages/whipper-0.7.0-py2.7.egg/whipper/command/main.py", line 36, in main
        ret = cmd.do()
      File "/usr/local/lib/python2.7/dist-packages/whipper-0.7.0-py2.7.egg/whipper/command/basecommand.py", line 139, in do
        return self.cmd.do()
      File "/usr/local/lib/python2.7/dist-packages/whipper-0.7.0-py2.7.egg/whipper/command/basecommand.py", line 139, in do
        return self.cmd.do()
      File "/usr/local/lib/python2.7/dist-packages/whipper-0.7.0-py2.7.egg/whipper/command/drive.py", line 63, in do
        info[0], info[1], info[2], t.defeatsCache)
      File "/usr/local/lib/python2.7/dist-packages/whipper-0.7.0-py2.7.egg/whipper/common/config.py", line 115, in setDefeatsCache
        section = self._findOrCreateDriveSection(vendor, model, release)
      File "/usr/local/lib/python2.7/dist-packages/whipper-0.7.0-py2.7.egg/whipper/common/config.py", line 164, in _findOrCreateDriveSection
        self.write()
      File "/usr/local/lib/python2.7/dist-packages/whipper-0.7.0-py2.7.egg/whipper/common/config.py", line 58, in write
        shutil.move(path, self._path)
      File "/usr/lib/python2.7/shutil.py", line 316, in move
        copy2(src, real_dst)
      File "/usr/lib/python2.7/shutil.py", line 144, in copy2
        copyfile(src, dst)
      File "/usr/lib/python2.7/shutil.py", line 97, in copyfile
        with open(dst, 'wb') as fdst:
    IOError: [Errno 13] Permission denied: u'/home/worker/.config/whipper/whipper.conf'

* remove needless tag version

:latest is implicit so it's useless noise as well.

* Update README.md
2018-10-03 15:43:58 +00:00
Thomas McWork
cfcbfd5623 Document Docker usage in the README (#297)
Document Docker usage in the README
2018-10-03 08:07:47 +00:00
Andreas Oberritter
542e071443 switch CDDB implementation to freedb.py from python-audio-tools (#276)
* freedb: Import from python-audio-tools 660ee2c

License: GPL-2.0+

* freedb: Remove unused code and set client name to whipper.

The removed functions depend on other classes of python-audio-tools,
but aren't needed here.

* cddb-py: replace with newer freedb implementation

The last release of cddb-py was 15 years ago. The freedb code taken
from python-audio-tools speaks protocol version 6 (Unicode) and is
compatible to both Python 2 and 3.

* freedb: Don't allow the pedantic CI test to fail
2018-06-08 14:40:56 +00:00
Joe
5db7656ef8 Clarify pre-emphasis reporting implementation
Currently whipper only reports about the pre-emphasis flag value as stored in the TOC.

Related to #275.
2018-05-24 17:29:51 +02:00
Andreas Oberritter
145403705d Switch to PyGObject by default (#271)
* cd/offset: remove unused call to gobject.thread_init()

There are no threads involved.

* task: switch to PyGObject

Python-gobject-2 is deprecated.

* travis: update dependencies for PyGObject and GLib-2.0

* README: Add GObject Introspection
2018-05-24 17:11:37 +02:00
JoeLametta
99bfe41c0a Update Travis CI URL 2018-05-08 08:00:00 +00:00
Joe
1cece638cd Misc README changes
Fixes #229.
2018-04-09 10:31:04 +02:00
Samantha Baldwin
1ba0132948 Small readme cleanups (#250)
* README cleanups

	- remove backwards incompatible changes section
	- add note about adding and running tests for devs
	- remove mention of incomplete man page
	- remove mention of force-pushed non-master branches

* README: one line per shield

* README: table of content updates

* README: remove Credits

* Revert "README: remove Credits"

This reverts commit f2b126c737595e08f4500ea284c8e69b457550cb.
2018-03-22 14:28:32 +01:00
Samantha Baldwin
fd2c56053b read command parameters from config sections (#240)
* read command parameters from config sections

* README: update config file documentation

* README: config example type INI, add comma

* README: clearer and better spelled wording

* README: fix 'delineated' typo
2018-03-07 14:39:29 +01:00
Joe
f6169ae3cb State that whipper isn't just a fork of morituri
First step for #79.
2018-03-03 11:37:08 +01:00
Matthew Ruszczyk
3a7e17f388 Remove references to Copr repository
Whipper is now available from the official Fedora repositories so mentions to the Copr are no longer required.
2018-03-02 14:18:14 -05:00
Samantha Baldwin
c7e8a0f21d credit four major developers by line count 2018-03-01 20:48:29 -05:00
Calum Chisholm
e8fdd868d2 Removed reference to unused "profile = flac" config option (see issue #99) 2018-02-12 10:50:22 +08:00
JoeLametta
8179c3f79e Update README. Fix missing link in CHANGELOG file 2018-02-06 10:00:00 +00:00
JoeLametta
64a0cf4327 Push whipper v0.6.0 release 2018-02-02 18:00:00 +00:00
Volker Mische
9208c06a16 whipper can now be run without build step 2018-01-29 03:40:43 +01:00
Joe
a34942e38d Changed cdparanoia dependency to cd-paranoia
Now whipper requires libcdio's `cd-paranoia` instead of Xiph's `cdparanoia`.
2018-01-26 09:18:16 +01:00
Eshan Singh
e2e8676e4d Enable connecting to a custom MusicBrainz server
Under the new [musicbrainz] section in the config,
you can set a server to connect to.

Closes #172.
2018-01-05 17:29:28 +05:30
JoeLametta
a3e92606c5 Merge pull request #187 from RecursiveForest/rewrite-accuraterip
AccurateRip V2 support
2017-09-15 23:12:40 +02:00
Clément Bœsch
984dd524ef README: document developer installation as user
If you're looking into running uninstalled, you likely don't want to
setup as root.
2017-09-08 21:14:00 +02:00
JoeLametta
47842c2d2a Update README.md 2017-09-08 14:11:00 +02:00
Samantha Baldwin
a124047af3 add requests dependency to README 2017-09-04 22:07:17 -04:00
JoeLametta
893139f683 Update README.md
Fixes #152
Fixes #165
2017-06-19 14:19:56 +02:00
JoeLametta
3dad52df42 Update whipper release number in init.py
I'm going to tag this as a new amended version 0.5.1: no changes were done apart from the version number bugfix.
I've also included a code coverage report.

Fixes #147
2017-04-24 16:22:51 +02:00
JoeLametta
1abc54c59f Polish everything after PR #130
Updated README.md, CHANGELOG.md, .travis.yml
2017-04-24 11:54:37 +02:00
Merlijn Wajer
3fbe7f4adf Remove dependency on gst from the README. 2017-04-23 23:07:52 +02:00
JoeLametta
490d84380a Amend previous tagged release (#107) 2017-01-08 20:04:19 +01:00
JoeLametta
adf4b79a20 Bump whipper version number to 0.4.1
Fixes #105
2017-01-07 23:53:47 +01:00
Frederik “Freso” S. Olesen
23a43b10cb Update links to Arch Linux AUR packages in README.
There's now a "stable" `whipper` package, which is likely the main one
we want to link to. I kept in the link to the `whipper-git` package at
the end, but also added a link to the non-git `whipper` package.
2017-01-06 20:50:41 +01:00
Samantha Baldwin
d1ed80d62a argparse & logging (#92)
* introduce logcommand.Lager, Whipper(); use argparse for whipper image commands, stub logging

* update Lager docstring to mention config.Config()

* make incorrect subcommand and --version work on toplevel command

* migrate accurip show, expand Lager, do not attempt to return from Lager.__init__.

* migrate offset find, add Lager.error

* correct offset find drive symlink handling

* migrate drive

* change Lager.__init__(prog) to arg from kwarg

* but actually

* remove Whipper.usage

* add and use Lager.device_option() context manager

* help I married an axe murderer

* use unified options namespace for entire command tree

* migrate whipper cd without comprehensive config loading

* switch to logging module

- use logging instead of flog for non-extern modules
- use WHIPPER_DEBUG and WHIPPER_LOGFILE env variables

* convert self.log calls to logger.debug

* convert self.error calls to logger.error

* remove log.Loggable, use logger not logging

* Logging conversion continues

- Convert log.* calls to logger.*
- Remove morituri.common.log imports

* remove morituri.common.log from tests

* remove extern/flog, bare minimum Debug conversion

* update README for logging changes

* update soxi to use logging

* refactor Lager for more declarative subcommands

* Refactor Lager.device_option:

- inline into __init__
- throw IOError instead of Exception for missing drives
- remove CommandError checking in rip/main

* rename rip to whipper in rip.main

* convert rip.debug commands

* Rename logcommand.Lager to command.BaseCommand

- remove command.CommandError occurrences
- remove python-command external module

* remove submodules from README, update rclog formatter

* update minor ambiguity in readme for command invocation

* update version number to match setup.py

* remove gitmodules

* update version number in tests as well (boo)

* convert logger.error to logger.critical

* Change morituri.rip to morituri.command

- mv common.command to command.basecommand
- move TEMPLATES used only by rip.cd out of rip.common
- update entry point for command to command.main

* update basecommand documentation

* go pyflaking: import fixing

* replace self.stdout with sys.stdout

* remove BaseCommand.config, alphabetise imports

* convert self.stdXXX leftovers

* convert last getRootCommand to config.Config

* convert last getExceptionMessage's to str

* change musicbrainz useragent to whipper
2016-12-20 23:11:30 +01:00
Pierre
b3f0fbc59d Update README.md
Added clearer info on required GStreamer-related packages.

I got a `no module named gst` while testing on Ubuntu 16.04 and figured it would be clearer to name the package to install to avoid that.
2016-12-17 23:25:00 +08:00
JoeLametta
c647b4cfb9 Fixed README broken links and added better changelog 2016-12-17 12:52:02 +01:00
Takeshi Suzuki
71335e1449 Fix spelling mistake in README.md (#86) 2016-12-11 15:29:19 +01:00
JoeLametta
8fdb452aa6 Fix README mistake 2016-11-10 23:16:38 +01:00
JoeLametta
2fc90e0789 Update information for release v0.4.0 2016-11-08 23:04:25 +01:00
JoeLametta
2d7692cb22 Invoke whipper by its name + Readme rewrite (#70)
* Whipper is now invoked by its name ("whipper") instead of "rip"
* README has been rewritten to provide a better experience and updated information
2016-11-08 22:31:13 +01:00
chrysn
926df4d7aa README.md: drop executable flag
The flag seems to have been set accidentally in dceb10f4.
2016-10-26 18:14:55 +02:00
Samantha Baldwin
8721ba1caf standardise program/sox.py formatting, add test case, docstring (#53)
* standardise program/sox.py formatting, add test case
* add program/sox.peak_level() docstring
* update .travis.yml to include sox as a test dependency
* move sox from test dependency to normal dependency
* update README to mention sox, remove small autoconf mention
* use .splitlines() instead of .split('n')
2016-10-21 23:54:13 +02:00
JoeLametta
4636c55233 Update CI tests
* slightly updated the README file
* updated morituri version number in cue tests
* commented out a portion of testAccurateRipChecksum (it should be updated/fixed later on)
2016-10-20 09:57:05 +02:00
Samantha Baldwin
063876b575 use setuptools, remove autohell, use raw make for src/ 2016-10-19 17:54:01 -04:00
JoeLametta
3a2e3021c2 Update README.md
Added information about the new dependencies
2016-10-15 21:20:54 +02:00
JoeLametta
746dc6599d Issue24 (#42)
* Address issue #24
* Remove getReadCaches() & replace it with getCache()

Now whipper always follows XDG specifications. All changes were documented into the README file.

The changes introduced here aren't backward compatible so, after updating whipper, you may need to configure it again (old config file are retained, though).
2016-10-09 17:26:06 +02:00
JoeLametta
67ea220604 Make pycdio dependency not overridable 2016-09-05 17:14:11 +02:00