dsd patch to record problem (OLPC #12686)v42

3 files changed, 11 insertions, 8 deletions

diff --git a/NEWS b/NEWS
index 64362f1..c19eb54 100644
--- a/NEWS
+++ b/NEWS
@@ -1,3 +1,8 @@
+42
+
+BUG FIX:
+* Fixed record problem (tip of the hat to dsd)
+
 41
 
 ENHANCEMENT:
diff --git a/activity/activity.info b/activity/activity.info
index d669e4f..3a842e6 100644
--- a/activity/activity.info
+++ b/activity/activity.info
@@ -1,6 +1,6 @@
 [Activity]
 name = Portfolio
-activity_version = 41
+activity_version = 42
 license = GPLv3
 bundle_id = org.sugarlabs.PortfolioActivity
 exec = sugar-activity PortfolioActivity.PortfolioActivity
diff --git a/grecord.py b/grecord.py
index 92a4a75..8a4b252 100644
--- a/grecord.py
+++ b/grecord.py
@@ -212,20 +212,18 @@ filesink name=audioFilesink'
         return True
 
     def _query_position(self, pipe):
-        try:
-            position, format = pipe.query_position(Gst.Format.TIME)
-        except:
+        result, position = pipe.query_position(Gst.Format.TIME)
+        if not result:
             position = Gst.CLOCK_TIME_NONE
 
-        try:
-            duration, format = pipe.query_duration(Gst.Format.TIME)
-        except:
+        result, duration = pipe.query_duration(Gst.Format.TIME)
+        if not result:
             duration = Gst.CLOCK_TIME_NONE
 
         return (position, duration)
 
     def _onMuxedAudioMessageCb(self, bus, message, pipe):
-        _logger.debug(message.type)
+        # _logger.debug(message.type)
         if message.type != Gst.MessageType.EOS:
             return True
         self._clean_up_transcoding_pipeline(pipe)