Ubuntu
speech-dispatcher package

Incorrect file paths for icons played via spd-conf

Bug #314185 reported by Nolan Darilek on 2009-01-05

Affects		Status	Importance	Assigned to	Milestone
	speech-dispatcher (Ubuntu)	Fix Released	Undecided	Luke Yelavich

Bug Description

Under python-speechd 0.6.70-ubuntu2 and Intrepid, the paths for files played via spd-conf are incorrect. Looks like the directory where the files are found needs to end in a /, because the last subdirectory and filename are run together. For instance, see this transcript:

$ spd-conf -d
Speech Dispatcher configuration tool

Testing Speech Dispatcher using spd_say

Did you hear the message about Speech Dispatcher working? [yes] :

Speech Dispatcher is working

Speech Dispatcher works. Do you want to skip other tests? [yes] :

Do you want to test the Festival synthesizer now? [yes] :

Do you want to test the Espeak synthesizer now? [yes] :

Do you want to test the ALSA sound system now? [yes] :

Testing ALSA sound output

/usr/share/sounds/speech-dispatchertest.wav: No such file or directory

Can't play audio via aplay /usr/share/sounds/speech-dispatchertest.wav

Your audio doesn't seem to work, please fix audio first or choose

a different method.

/usr/share/sounds/speech-dispatchertest.wav does not exist, but /usr/share/sounds/speech-dispatcher/test.wav does.

Related branches

lp://staging/ubuntu/maverick/speech-dispatcher

Revision history for this message

Nolan Darilek (nolan-thewordnerd) wrote on 2009-03-30:

Fix Edit (562 bytes, text/plain)

This is tested and works here.

Luke Yelavich (themuso) on 2010-07-15

Changed in speech-dispatcher (Ubuntu):
assignee:	nobody → Luke Yelavich (themuso)
status:	New → In Progress

Revision history for this message

Launchpad Janitor (janitor) wrote on 2010-07-15:

This bug was fixed in the package speech-dispatcher - 0.7-5ubuntu1

---------------
speech-dispatcher (0.7-5ubuntu1) maverick; urgency=low

  * Merge from debian unstable. Remaining changes:
    - Python-speechd should depend on speech-dispatcher
    - Moved the flite output module to a separate package, and added
      it to suggests
    - Use cdbs for package build, cleaner to maintain
    - Quieten logfiles by default
    - Only enable espeak output module
    - Fix pulse driver endainness
    - Fix broken spd-conf
  * debian/speech-dispatcher.init: Don't assume that the user was previously
    running speech-dispatcher system-wide, at least in Ubuntu
  * Remove references to speechd-server-spawn and speechd-user-port.sh, since
    speech-dispatcher now has a different way of spawning, and uses sockets
    (LP: #601114)
  * Flite patch from Debian fixes FTBFs (LP: #601005)
  * debian/patches/spd-conf-test-path-fix.patch: Fix path to test.wav file
    which gets played by spd-conf (LP: #314185)

speech-dispatcher (0.7-5) unstable; urgency=low

  * Move Python build dependencies from Build-Depends-Indep to Build-Depends;
    closes: #588107.
  * Standards 3.9.0 (no real change).

speech-dispatcher (0.7-4) unstable; urgency=low

* Default AudioOutputMethod changed to "pulse" to prevent blocking audio
output; information to README.Debian added.

speech-dispatcher (0.7-3) unstable; urgency=low

* Paths in python-speechd adjusted to match Debian installation.
* Package priorities set to match override entries.

speech-dispatcher (0.7-2) unstable; urgency=low

  * Link speech-dispatcher log location to /var/log/speech-dispatcher/, if
    it doesn't exist yet.
  * Permit access to speech-dispatcher socket to `audio' group.

speech-dispatcher (0.7-1) unstable; urgency=low

  * New upstream version.
  * Switch to dpkg-source 3.0 (quilt) format.
  * Versioned build dependency on flite-dev.
  * Don't start Speech Dispatcher by default; closes: #577217.
  * Standards 3.8.4 (no real change).
-- Luke Yelavich <email address hidden> Thu, 15 Jul 2010 14:30:18 +1000

This bug was fixed in the package speech-dispatcher - 0.7-5ubuntu1

---------------
speech-dispatcher (0.7-5ubuntu1) maverick; urgency=low

* Merge from debian unstable.  Remaining changes:
    - Python-speechd should depend on speech-dispatcher
    - Moved the flite output module to a separate package, and added
      it to suggests
    - Use cdbs for package build, cleaner to maintain
    - Quieten logfiles by default
    - Only enable espeak output module
    - Fix pulse driver endainness
    - Fix broken spd-conf
  * debian/speech-dispatcher.init: Don't assume that the user was previously
    running speech-dispatcher system-wide, at least in Ubuntu
  * Remove references to speechd-server-spawn and speechd-user-port.sh, since
    speech-dispatcher now has a different way of spawning, and uses sockets
    (LP: #601114)
  * Flite patch from Debian fixes FTBFs (LP: #601005)
  * debian/patches/spd-conf-test-path-fix.patch: Fix path to test.wav file
    which gets played by spd-conf (LP: #314185)

speech-dispatcher (0.7-5) unstable; urgency=low

* Move Python build dependencies from Build-Depends-Indep to Build-Depends;
    closes: #588107.
  * Standards 3.9.0 (no real change).

speech-dispatcher (0.7-4) unstable; urgency=low

* Default AudioOutputMethod changed to "pulse" to prevent blocking audio
    output; information to README.Debian added.

speech-dispatcher (0.7-3) unstable; urgency=low

* Paths in python-speechd adjusted to match Debian installation.
  * Package priorities set to match override entries.

speech-dispatcher (0.7-2) unstable; urgency=low

* Link speech-dispatcher log location to /var/log/speech-dispatcher/, if
    it doesn't exist yet.
  * Permit access to speech-dispatcher socket to `audio' group.

speech-dispatcher (0.7-1) unstable; urgency=low

* New upstream version.
  * Switch to dpkg-source 3.0 (quilt) format.
  * Versioned build dependency on flite-dev.
  * Don't start Speech Dispatcher by default; closes: #577217.
  * Standards 3.8.4 (no real change).
 -- Luke Yelavich <themuso@ubuntu.com>   Thu, 15 Jul 2010 14:30:18 +1000

Changed in speech-dispatcher (Ubuntu):
status:	In Progress → Fix Released

Revision history for this message

Boris Dušek (dusek) wrote on 2010-07-16:

The fix released in 0.7-5ubuntu1 (the patch spd-conf-test-path-fix.patch) is:

* wrong: uses os.path.join with second argument as absolute path ("/test.wav"), which results into the final (absolute) path of just "/test.wav".

* unneeded: this was fixed upstream (commit f0ba6a27...) by using os.path.join, which supplies the correct platform-specific path separators at correct places

The fix for this is easy - do not use the patch spd-conf-test-path-fix.patch.