From 36cfd43c73c836a1b86fde22b94cc84328d58426 Mon Sep 17 00:00:00 2001 From: miro Date: Thu, 9 May 2024 04:34:02 +0100 Subject: [PATCH] only log warning if old OCP is loaded, not if legacy bus api is enabled --- ovos_audio/service.py | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/ovos_audio/service.py b/ovos_audio/service.py index 40e0dbb..a941bdd 100644 --- 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! '