Remove the data_dir option, and replace it with XDG_DATA_HOME.

poezio/config.py

@@ -45,7 +45,6 @@ DEFAULT_CONFIG = {
         'create_gaps': False,
         'custom_host': '',
         'custom_port': '',
-        'data_dir': '',
         'default_nick': '',
         'deterministic_nick_colors': True,
         'device_id': '',
@@ -592,35 +591,12 @@ def create_global_config():
         sys.exit(1)
 
 
-def check_create_data_dir():
-    """Create the poezio data directory if it doesn't exist"""
-    global DATA_DIR
-    DATA_DIR = config.get('data_dir')
-
-    if not DATA_DIR:
-        data_home = environ.get('XDG_DATA_HOME')
-        if data_home is None or not Path(data_home).is_absolute():
-            data_home = path.join(environ.get('HOME'), '.local', 'share')
-
-        DATA_DIR = path.join(data_home, 'poezio')
-
-    DATA_DIR = path.expanduser(DATA_DIR)
-    try:
-        makedirs(DATA_DIR)
-    except:
-        pass
-
-
 def check_create_log_dir():
     "Create the poezio logging directory if it doesn’t exist"
     global LOG_DIR
+
     LOG_DIR = config.get('log_dir')
-
-    if not LOG_DIR and not DATA_DIR:
-        check_create_data_dir()
-
-    if not LOG_DIR:
-        LOG_DIR = path.join(DATA_DIR, 'logs')
+    LOG_DIR = Path(LOG_DIR).expanduser() if LOG_DIR else xdg.DATA_HOME / 'logs'
 
     try:
         makedirs(LOG_DIR)
@@ -700,8 +676,5 @@ options = None
 # delayed import from common.py
 safeJID = None
 
-# the global data dir
-DATA_DIR = ''
-
 # the global log dir
 LOG_DIR = ''

poezio/poezio.py

@@ -79,7 +79,6 @@ def main():
     from poezio import config
     config.run_cmdline_args()
     config.create_global_config()
-    config.check_create_data_dir()
     config.check_create_log_dir()
     config.setup_logging()
     config.post_logging_setup()