only log warning if old OCP is loaded, not if legacy bus api is enabled

OpenVoiceOS · May 9, 2024 · 36cfd43 · 36cfd43
1 parent ad77c1e
commit 36cfd43
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/ovos_audio/service.py b/ovos_audio/service.py
@@ -99,6 +99,7 @@ def __init__(self, ready_hook=on_ready, error_hook=on_error,
         self.audio_enabled = self.config.get("enable_old_audioservice", True)  # TODO default to False soon
         if disable_ocp is None:
             disable_ocp = self.config.get("disable_ocp", False)  # TODO default to True soon
+        self.disable_ocp = disable_ocp
         if self.audio_enabled:
             try:
                 self.audio = AudioService(self.bus, disable_ocp=disable_ocp, validate_source=validate_source)
@@ -245,8 +246,10 @@ def handle_opm_audio_query(self, message):
     def run(self):
         self.status.set_alive()
         if self.audio_enabled:
-            LOG.warning("audio service has moved to ovos-media, if you already migrated to ovos-media "
-                        'set "enable_old_audioservice": false in mycroft.conf')
+            LOG.info("Legacy AudioService enabled")
+            if not self.disable_ocp:
+                LOG.warning("OCP has moved to ovos-media, if you already migrated to ovos-media "
+                            'set "disable_ocp": true in mycroft.conf')
             if self.audio.wait_for_load():
                 if len(self.audio.service) == 0:
                     LOG.warning('No audio backends loaded! '