Merge pull request #2840 from DiegoAsterio/bugfix/issue-2750

Issue-2750 - Remove log pollution
MycroftAI · Mar 3, 2021 · 17dbaa2 · 17dbaa2
2 parents 492bd22 + a00fe86
commit 17dbaa2
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/mycroft/skills/skill_manager.py b/mycroft/skills/skill_manager.py
@@ -132,6 +132,8 @@ def __init__(self, bus, watchdog=None):
         self.num_install_retries = 0
         self.settings_downloader = SkillSettingsDownloader(self.bus)
 
+        self.empty_skill_dirs = set()  # Save a record of empty skill dirs.
+
         # Statuses
         self._alive_status = False  # True after priority skills has loaded
         self._loaded_status = False  # True after all skills has loaded
@@ -314,8 +316,13 @@ def _get_skill_directories(self):
             # check if folder is a skill (must have __init__.py)
             if SKILL_MAIN_MODULE in os.listdir(skill_dir):
                 skill_directories.append(skill_dir.rstrip('/'))
+                if skill_dir in self.empty_skill_dirs:
+                    self.empty_skill_dirs.discard(skill_dir)
             else:
-                LOG.debug('Found skills directory with no skill: ' + skill_dir)
+                if skill_dir not in self.empty_skill_dirs:
+                    self.empty_skill_dirs.add(skill_dir)
+                    LOG.debug('Found skills directory with no skill: ' +
+                              skill_dir)
 
         return skill_directories