Commit Graph

4802 Commits

Author SHA1 Message Date
Thomas Adamcik
bcce865ce7 avahi: Convert MPD and HTTP to use cleaned up versions
- Move imports to top as they should be.
- Report state based on publish return value.
- Remove overly broad except clauses.
- Unpublish before stopping servers.
2013-11-11 21:15:56 +01:00
Thomas Adamcik
50cfe42ce0 avahi: Improve error handling for dbus errors.
- Handle dbus not being installed.
- Handle not finding the system bus.
- Handle not finding the avahi service.
- Return if publish succeeded.
2013-11-11 21:15:47 +01:00
Thomas Adamcik
cf5589b4eb avahi: Style and code cleanups in zeroconf module.
- Prepare for handling missing dbus directly in module.
- Constants should always be all caps.
- Extracted helpers from class to convey intent via their naming.
- Moved imports out of class, imports should always be on the top.
- Made sure calling publish mutiple times does not re-convert text.
- Made sure calling unpublish without publish does not break.
2013-11-11 21:15:35 +01:00
Stein Magnus Jodal
484efab28e utils: Remove Python 2.6 workaround 2013-11-10 21:37:53 +01:00
Thomas Adamcik
851cf65328 docs: More review fixes 2013-11-10 13:11:01 +01:00
Thomas Adamcik
d839a0d1ba local: Fix flake mistakes in scan sub-command. 2013-11-10 13:03:53 +01:00
Thomas Adamcik
4e6ebbe955 local/docs: Update based on review comments
- Bunch of typos and wording improvements from review.
- Fixed mopidy.backends.local.scan botched merge.
- Document and enforce that sub-command name needs to be bytes.
2013-11-09 18:20:22 +01:00
Thomas Adamcik
df953ea1e6 Merge branch 'develop' into feature/subcommands
Conflicts:
	mopidy/scanner.py
2013-11-09 13:40:47 +01:00
Thomas Adamcik
5cd0938e0d docs: Update changelog with sub-commands work 2013-11-09 13:38:09 +01:00
Thomas Adamcik
3945e437de docs: Fix minor doc issues found during review
- Adds autodoc for mopidy.audio.scan
- Updates ``file://`` to ``local:``
- Minor language fix.
2013-11-09 13:30:27 +01:00
Thomas Adamcik
67c028c31e docs: Update docs with respect to sub-commands 2013-11-09 13:23:52 +01:00
Lasse Bigum
deb442c54a Update documentation 2013-11-09 03:10:20 +01:00
Lasse Bigum
2bd1f043ce Updated code with new multiple artist handling and added tests 2013-11-09 03:01:53 +01:00
Lasse Bigum
8b7621c3e3 Merge branch 'develop' of https://github.com/mopidy/mopidy into feature/extra_tags 2013-11-09 02:37:44 +01:00
Lasse Bigum
c776565f65 Another round of review comments 2013-11-09 01:22:27 +01:00
Thomas Adamcik
51d1e22655 docs: Update changelog for PR #566 and #567 2013-11-08 23:51:45 +01:00
Thomas Adamcik
1a3ff456f9 scanner: Remove old scanner in favour of sub-commands 2013-11-08 23:21:26 +01:00
Thomas Adamcik
b5f8480eea local: Add 'mopidy scan' command via extension sub-commands. 2013-11-08 23:20:19 +01:00
Thomas Adamcik
2581062ae0 ext/backends: Add extensions to sub commands run 2013-11-08 23:18:26 +01:00
Thomas Adamcik
ef10c2e178 main: Wire in actual execution of sub-commands. 2013-11-08 22:04:26 +01:00
Thomas Adamcik
518cac5eab ext/backends: Add get_sub_commands and BaseSubCommandProvider 2013-11-08 22:01:18 +01:00
Thomas Adamcik
7144876dc5 main: Move default subparsers into commands module
Also switches to using dest for storing the chosen sub-parser.
2013-11-08 22:01:18 +01:00
Lasse Bigum
093c0400a9 Fix review comments 2013-11-08 21:43:31 +01:00
Lasse Bigum
03750a8bf2 Fix scan with multiple track artists and add tests 2013-11-08 21:25:21 +01:00
Thomas Adamcik
25fedc7700 loggin: Add DelayedHandler to root logger.
The delayed handler:
- Accepts and buffers logs until they are released.
- Upon release the logs are re-posted to the root logger.
- After release log records are ignored.

This allows us to avoid the silly tricks we've been doing with parsing args and
config early for the sake of bootstraping logging. Now we can just start
logging and once the logging has been setup the messages are released and
handled according to the correct settings.
2013-11-08 19:53:00 +01:00
Thomas Adamcik
4f03677675 main: Improve main bootstrapping sequence
- Parses args in two pases to allow for setup of logging well before doing
  extension sub-commands.
2013-11-08 19:53:00 +01:00
Thomas Adamcik
9539c2ac35 main: Switch to subcommands
- show-deps replaced with 'mopidy deps'
- show-config replaced with 'mopidy config'
- Just running mopidy now displays help, run 'mopidy run' to start server.
2013-11-08 19:53:00 +01:00
Thomas Adamcik
f49973304c main: Start unifying command handling
- Removes show_deps and show_config from commands module. These
  are now handled directly in the main() method pending subcommands.
- Unifies show_config with general main() config handling.
- Sets default verbosity level to zero.
- Reduce verbosity when --show-config or --show-deps is called.
- Update console logging to consider verbosity < 0 quiet/
2013-11-08 01:17:52 +01:00
Thomas Adamcik
51b778fcd6 main: Create helper for logging boostraping 2013-11-08 00:59:05 +01:00
Thomas Adamcik
bdba83b1be main: Move argparse building to a function. 2013-11-08 00:09:17 +01:00
Stein Magnus Jodal
e608f9f213 Merge pull request #567 from adamcik/feature/local-uri-translation
local/scanner: Make checking mtime and skipping known files work again
2013-11-07 15:00:17 -08:00
Thomas Adamcik
93918cb1e0 local: flake8 fixes 2013-11-07 23:25:55 +01:00
Stein Magnus Jodal
5b78ff94ff docs: Remove count from MPD limitations list 2013-11-07 19:33:46 +01:00
Javier Domingo Cansino
71bae709ef core: filter() to accept also tuples 2013-11-07 11:05:33 +01:00
Javier Domingo Cansino
45a38cdaf1 core: Changing input to accept also sets, as might be also used. 2013-11-07 10:51:29 +01:00
Javier Domingo Cansino
6721a59b26 tests: Fixing self confusion mistake
docs: Documenting tracklist's new filter() feature
2013-11-07 09:46:34 +01:00
Javier Domingo Cansino
8b5f30e5ff Adding test case 2013-11-07 09:34:40 +01:00
Thomas Adamcik
e7dd7e2677 local/scanner: Make checking mtime and skipping known files work again
This change just patches over the worst of the inconsistencies in how the
scanner mixes local and file uris to get us to a working state again. Ideally,
this still needs a real cleanup when we finish the plugable library providers
and/or json library work.
2013-11-07 00:03:00 +01:00
Thomas Adamcik
f8dedc0b84 local: Add local_to_file_uri translator 2013-11-06 21:46:41 +01:00
Stein Magnus Jodal
71eff1bf45 Merge pull request #566 from adamcik/feature/scanner
Switch back to custom scanner
2013-11-06 12:31:02 -08:00
Thomas Adamcik
86926e8011 scanner: Remove unused argument for progress helper. 2013-11-06 21:14:11 +01:00
Thomas Adamcik
dc3cf427b6 scanner/mpd: Add TODOs for later and fix flake8 warning 2013-11-06 19:19:17 +01:00
Thomas Adamcik
e6381a1a0a audio: Add mtime to scanner results for file: uris 2013-11-06 19:10:49 +01:00
Thomas Adamcik
469f414c4c audio: Don't do duration conversion to ms in scanner 2013-11-06 19:10:49 +01:00
Thomas Adamcik
16ac5277f6 audio: Make min duration in scanner configurable. 2013-11-06 19:10:49 +01:00
Thomas Adamcik
20b0602842 audio/scanner: Move translator into audio.
Tries to move more of the gst bits and pieces we are leaking into audio.
2013-11-06 19:10:49 +01:00
Thomas Adamcik
4cadba0ac7 scanner: Add progress tracking and process sorted uris 2013-11-06 19:10:49 +01:00
Thomas Adamcik
0a2d74eff1 scanner: Update to use new mopidy.audio.scan
Also adds the check less than 100ms check back to the scanner.
2013-11-06 19:10:49 +01:00
Thomas Adamcik
d6ab78a86c audio: Re-add improved version of python based scanner.
- Unlike the old python version we do not wait for the first audio handoff, we
  only progress until the PAUSED state. This ensure we don't block on empty files.
- Instead of using the signal watch and running the main loop we simply poll
  the messages from the bus directly allowing for a synchronous code flow.
- Between each file the pipeline is always returned to NULL, this is done as we
  found that gst 0.10 will slow down as the uribin does not cleanup the
  children it creates for handling each file. This issue is not present in 1.0.
- This also works around a segfault that was likely caused by a race condition
  that seems to trigger in the 0.10 version of the pbutils discoverer.
- This version of the scanner also fixes the per track slow down, and works out
  to be considerably faster than even the built in discoverer from 1.0.
- Finally this removes the WMA hack as I kan no longer find any evidence of it
  being needed.
2013-11-06 19:10:49 +01:00
Lasse Bigum
af6225538d Add more tests 2013-11-05 23:17:15 +01:00