After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 559583 - Swift synthesis driver causes Python client to get "TypeError: could not demarshal return value" on call to getAllVoices
Swift synthesis driver causes Python client to get "TypeError: could not dema...
Status: RESOLVED FIXED
Product: gnome-speech
Classification: Deprecated
Component: drivers
unspecified
Other Linux
: Normal major
: ---
Assigned To: Willie Walker
GNOME Speech Maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2008-11-06 12:29 UTC by Willie Walker
Modified: 2008-11-16 16:34 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Simple app to reproduct (616 bytes, text/plain)
2008-11-06 12:34 UTC, Willie Walker 		  Details
Patch to remedy the problem (1.28 KB, patch)
2008-11-06 16:16 UTC, Willie Walker 		committed Details | Review

Description Willie Walker 2008-11-06 12:29:33 UTC 
Python clients connecting to the gnome-speech driver for the Cepstral Swift 5.1.0 and 4.2.1 engines end up getting a "TypeError: could not demarshal return value" on a call to the driver's getAllVoices method.  The same call in from C clients does not result in the same problem.
Comment 1 Willie Walker 2008-11-06 12:34:38 UTC 
Created attachment 122092 [details]
Simple app to reproduct

This Python app reproduces the problem.  To do the same with C, use the gnome-speech test-speech application.
Comment 2 Willie Walker 2008-11-06 12:47:34 UTC 
> Simple app to reproduct

I'm pretty sure I wrote "Simple app to reproduce the problem", but I guess I didn't. :-(
Comment 3 Willie Walker 2008-11-06 16:16:38 UTC 
Created attachment 122115 [details] [review]
Patch to remedy the problem

The problem boils down to the Cepstral voices using varying upper/lower cases for the gender string and the fact that the driver was setting the gender to -1, which is an illegal value.  This patch changes the swift gender string to lower case before doing a string compare and also defaults the value to male if the gender string is neither "male" nor "female".

Mental note to self - make a new release before the next GNOME 2.24.x deadline.
Comment 4 Willie Walker 2008-11-16 16:34:23 UTC 
New gnome-speech 0.4.22 release made.  Closing this as FIXED.