Free the sound device when the startup when the sound has been played.

Factored out to a separate sound module from cjb patch.

bin/sugar-shell

@@ -25,10 +25,10 @@ import pygtk
 pygtk.require('2.0')
 import gtk
 import gobject
-import gst
 
 from sugar import env
 from sugar import logger
+from sugar import sound
 from sugar.profile import get_profile
 
 sys.path.insert(0, env.get_shell_path())
@@ -89,9 +89,7 @@ def _shell_started_cb():
 
     startup_sound = os.path.join(env.get_data_path('startup.flac'))
     if os.path.exists(startup_sound):
-        player = gst.element_factory_make("playbin", "player")
+        sound.play(startup_sound)
-        player.set_property("uri", "file://" + startup_sound)
-        player.set_state(gst.STATE_PLAYING)
 
 def main():
     gobject.idle_add(_shell_started_cb)

lib/sugar/Makefile.am

@@ -5,6 +5,7 @@ sugar_PYTHON =		\
 	env.py		\
         network.py	\
 	profile.py	\
+	sound.py	\
 	util.py		\
 	wm.py
 

lib/sugar/sound.py

@@ -0,0 +1,44 @@
+# Copyright (C) 2007 One Laptop Per Child
+#
+# This library is free software; you can redistribute it and/or
+# modify it under the terms of the GNU Lesser General Public
+# License as published by the Free Software Foundation; either
+# version 2 of the License, or (at your option) any later version.
+#
+# This library is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+# Lesser General Public License for more details.
+#
+# You should have received a copy of the GNU Lesser General Public
+# License along with this library; if not, write to the
+# Free Software Foundation, Inc., 59 Temple Place - Suite 330,
+# Boston, MA 02111-1307, USA.
+
+import logging
+
+import gst
+
+class Player(object):
+    def __init__(self):
+        self._player = gst.element_factory_make("playbin", "player")
+
+        bus = self._player.get_bus()
+        bus.add_signal_watch()
+        bus.connect("message::eos", self.__eos_cb)
+        bus.connect("message::error", self.__error_cb)
+
+    def play(self, sound_path):
+        self._player.set_property("uri", "file://" + sound_path)
+        self._player.set_state(gst.STATE_PLAYING)
+
+    def __eos_cb(self, bus, message):
+        self._player.set_state(gst.STATE_NULL)
+
+    def __error_cb(self, bus, message):
+        logging.error(message.parse_error())
+
+_player = Player()
+
+def play(sound_path):
+    _player.play(sound_path)