Merge pull request #147 from level12/106-config-resolution-permissions

Ignore non-existent or inaccessible files in config discovery

Author: guruofgentoo

keg/cli.py

@@ -126,6 +126,11 @@ def config_command():
     for val in config.config_file_paths():
         click.echo('    {}'.format(val))
 
+    if config.config_paths_unreadable:
+        click.echo(_('Could not access the following config paths:'))
+        for path, exc in config.config_paths_unreadable:
+            click.echo('    {}: {}'.format(path, str(exc)))
+
     click.echo(_('Config objects used:'))
     for val in config.configs_found:
         click.echo('    {}'.format(val))

keg/config.py

@@ -65,12 +65,19 @@ def init_app(self, app_config_profile, app_import_name, app_root_path, use_test_
         self.dirs = appdirs.AppDirs(app_import_name, appauthor=False, multipath=True)
         self.app_import_name = app_import_name
         self.app_root_path = app_root_path
+        self.config_paths_unreadable = []
 
         if config_file_objs:
             self.config_file_objs = config_file_objs
         else:
+            self.config_file_objs = []
             possible_config_fpaths = self.config_file_paths()
-            self.config_file_objs = pymodule_fpaths_to_objects(possible_config_fpaths)
+            fpaths_to_objects = pymodule_fpaths_to_objects(possible_config_fpaths)
+            for fpath, objects, exc in fpaths_to_objects:
+                if objects is None:
+                    self.config_paths_unreadable.append((fpath, exc))
+                else:
+                    self.config_file_objs.append((fpath, objects))
 
         if self.profile is None:
             self.profile = self.determine_selected_profile()