local: Replace local/data_dir with core/data_dir

Partly fixes #1259

docs/changelog.rst

@@ -23,6 +23,12 @@ Bug fix release.
   :file:`/var/lib/mopidy/.local` and :file:`/var/lib/mopidy/.cache`. (Fixes:
   :issue:`1259`)
 
+- Local: Deprecate :confval:`local/data_dir` and respect
+  :confval:`core/data_dir` instead. This does not change the defaults for
+  desktop users, only system services installed from packages that properly set
+  :confval:`core/data_dir`, like the Debian and Arch packages. (Fixes:
+  :issue:`1259`)
+
 - Stream: If "file" is present in the :confval:`stream/protocols` config value
   and the :ref:`ext-file` extension is enabled, we exited with an error because
   two extensions claimed the same URI scheme. We now log a warning recommending

docs/ext/local.rst

@@ -47,8 +47,8 @@ active at a time.
 To create a new library provider you must create class that implements the
 :class:`mopidy.local.Library` interface and install it in the extension
 registry under ``local:library``. Any data that the library needs to store on
-disc should be stored in :confval:`local/data_dir` using the library name as
-part of the filename or directory to avoid any conflicts.
+disc should be stored in the extension's data dir, as returned by
+:meth:`~mopidy.ext.Extension.get_data_dir`.
 
 
 Configuration

mopidy/local/__init__.py

@@ -23,7 +23,7 @@ class Extension(ext.Extension):
         schema = super(Extension, self).get_config_schema()
         schema['library'] = config.String()
         schema['media_dir'] = config.Path()
-        schema['data_dir'] = config.Path()
+        schema['data_dir'] = config.Deprecated()
         schema['playlists_dir'] = config.Deprecated()
         schema['tag_cache_file'] = config.Deprecated()
         schema['scan_timeout'] = config.Integer(

mopidy/local/ext.conf

@@ -2,7 +2,6 @@
 enabled = true
 library = json
 media_dir = $XDG_MUSIC_DIR
-data_dir = $XDG_DATA_DIR/mopidy/local
 scan_timeout = 1000
 scan_flush_threshold = 1000
 scan_follow_symlinks = false

mopidy/local/json.py

@@ -12,7 +12,7 @@ import tempfile
 import mopidy
 from mopidy import compat, local, models
 from mopidy.internal import encoding, timer
-from mopidy.local import search, storage, translator
+from mopidy.local import Extension, search, storage, translator
 
 logger = logging.getLogger(__name__)
 
@@ -116,7 +116,7 @@ class JsonLibrary(local.Library):
         self._browse_cache = None
         self._media_dir = config['local']['media_dir']
         self._json_file = os.path.join(
-            config['local']['data_dir'], b'library.json.gz')
+            Extension().get_data_dir(config), b'library.json.gz')
 
         storage.check_dirs_and_files(config)
 

mopidy/local/storage.py

@@ -3,8 +3,6 @@ from __future__ import absolute_import, unicode_literals
 import logging
 import os
 
-from mopidy.internal import encoding, path
-
 logger = logging.getLogger(__name__)
 
 
@@ -13,10 +11,3 @@ def check_dirs_and_files(config):
         logger.warning(
             'Local media dir %s does not exist.' %
             config['local']['media_dir'])
-
-    try:
-        path.get_or_create_dir(config['local']['data_dir'])
-    except EnvironmentError as error:
-        logger.warning(
-            'Could not create local data dir: %s',
-            encoding.locale_decode(error))

tests/local/test_json.py

@@ -45,10 +45,11 @@ class BrowseCacheTest(unittest.TestCase):
 class JsonLibraryTest(unittest.TestCase):
 
     config = {
+        'core': {
+            'data_dir': path_to_data_dir(''),
+        },
         'local': {
             'media_dir': path_to_data_dir(''),
-            'data_dir': path_to_data_dir(''),
-            'playlists_dir': b'',
             'library': 'json',
         },
     }

tests/local/test_library.py

@@ -65,10 +65,11 @@ class LocalLibraryProviderTest(unittest.TestCase):
     ]
 
     config = {
+        'core': {
+            'data_dir': path_to_data_dir(''),
+        },
         'local': {
             'media_dir': path_to_data_dir(''),
-            'data_dir': path_to_data_dir(''),
-            'playlists_dir': b'',
             'library': 'json',
         },
     }
@@ -105,11 +106,15 @@ class LocalLibraryProviderTest(unittest.TestCase):
 
         tmpdir = tempfile.mkdtemp()
         try:
-            tmplib = os.path.join(tmpdir, 'library.json.gz')
-            shutil.copy(path_to_data_dir('library.json.gz'), tmplib)
+            tmpdir_local = os.path.join(tmpdir, 'local')
+            shutil.copytree(path_to_data_dir('local'), tmpdir_local)
 
-            config = {'local': self.config['local'].copy()}
-            config['local']['data_dir'] = tmpdir
+            config = {
+                'core': {
+                    'data_dir': tmpdir,
+                },
+                'local': self.config['local'],
+            }
             backend = actor.LocalBackend(config=config, audio=None)
 
             # Sanity check that value is in the library
@@ -117,6 +122,7 @@ class LocalLibraryProviderTest(unittest.TestCase):
             self.assertEqual(result, self.tracks[0:1])
 
             # Clear and refresh.
+            tmplib = os.path.join(tmpdir_local, 'library.json.gz')
             open(tmplib, 'w').close()
             backend.library.refresh()
 

tests/local/test_playback.py

@@ -23,12 +23,11 @@ from tests.local import generate_song, populate_tracklist
 class LocalPlaybackProviderTest(unittest.TestCase):
     config = {
         'core': {
+            'data_dir': path_to_data_dir(''),
             'max_tracklist_length': 10000,
         },
         'local': {
             'media_dir': path_to_data_dir(''),
-            'data_dir': path_to_data_dir(''),
-            'playlists_dir': b'',
             'library': 'json',
         }
     }

tests/local/test_tracklist.py

@@ -18,11 +18,11 @@ from tests.local import generate_song, populate_tracklist
 class LocalTracklistProviderTest(unittest.TestCase):
     config = {
         'core': {
+            'data_dir': path_to_data_dir(''),
             'max_tracklist_length': 10000
         },
         'local': {
             'media_dir': path_to_data_dir(''),
-            'data_dir': path_to_data_dir(''),
             'playlists_dir': b'',
             'library': 'json',
         }