Merge pull request #437 from canonical/fix/quick-sound-workaround

fix: Don't crash if `canberra-gtk-play` doesn't exist
canonical · Feb 16, 2024 · 4553552 · 4553552
2 parents 64bcffa + 8feccc0
commit 4553552
Showing 1 changed file with 12 additions and 1 deletion.
diff --git a/packages/ubuntu_provision/lib/src/services/sound_service.dart b/packages/ubuntu_provision/lib/src/services/sound_service.dart
@@ -1,7 +1,18 @@
 import 'dart:io';
 
+import 'package:ubuntu_logger/ubuntu_logger.dart';
+
+final _log = Logger('sound_service');
+
+/// A service for playing sounds.
 class SoundService {
   Future<void> play(String id) async {
-    await Process.run('canberra-gtk-play', ['--id=$id']);
+    try {
+      await Process.run('canberra-gtk-play', ['--id=$id']);
+    } on ProcessException catch (_) {
+      _log.error(
+        'Error playing sound with id: $id (most likely missing canberra-gtk-play)',
+      );
+    }
   }
 }