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 157786 - No speech for contents from command line
No speech for contents from command line
Status: RESOLVED FIXED
Product: gnopernicus
Classification: Deprecated
Component: speech
unspecified
Other Linux
: High major
: ---
Assigned To: remus draica
remus draica
AP1
Depends on:
Blocks:
 
 
Reported: 2004-11-09 21:06 UTC by tvuong
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: 2.7/2.8


Attachments
proposed patch (2.16 KB, patch)
2004-11-16 14:20 UTC, remus draica
none Details | Review
reworked patch (with Bill's suggestion) (3.50 KB, patch)
2004-11-26 13:37 UTC, remus draica
accepted-commit_now Details | Review
proposed patch for speech issue (888 bytes, patch)
2004-11-26 14:32 UTC, remus draica
accepted-commit_now Details | Review

Description tvuong 2004-11-09 21:06:46 UTC
When type in Linux Command line (i.e pwd, etc.) from gnome-terminal, speech 
engine does not announce the content of command line ....

When typed in "pwd" .. it does not annouce the the path.

Steps:

1. Bring up Gnopernicus
2. Enable speech
3. Bring up Gnome terminal
4. Listen for speech
5. Type in "pwd"
6. Listen for path speech, resulting from command "pwd"

No speech about the path at all ...

Setups:
Gnopernicus : 0.9.16
Gnome Desktop : 2.8.1
Fedora Core 2
Comment 1 padraig.obriain 2004-11-10 09:11:07 UTC
t oh I do not think that this is a gnopernicus bug.

We are using vte 0.11.11 with patches for bugs #113590, #153405 and #156161 and
then this proble does not occur.
Comment 2 Dana Ormenisan 2004-11-11 10:46:15 UTC
I tried to reproduce this bug in Cinnabar 21 using gnopernicus 0.9.16 and vte
0.11.11 (with and without the patches for bugs #113590, #153405 and #156161 -
i.e. the behavior was the same).

In most situations the result of 'pwd' was spoken by gnopernicus, followed by
the presentation of the current prompt. Sometimes, only the presentation of the
current prompt was spoken. In both situation the result of 'pwd' was sent to speech.

In my oppinion, this is a duplicate of bug #142772.
Comment 3 Alexandra Telescu 2004-11-15 11:13:44 UTC
I am using Cinnabar 22 with gnopernicus from CVS HEAD November 15 and I observed
the same behavior described in comment 2. 
Comment 4 korn 2004-11-15 16:45:40 UTC
So we really have two solutions: either always concatenate all speech output
from vte, relying on the user to press Shut-Up; or use XEvIE to see the user
keystrokes and always shut up any time the user presses a
non-Gnopernicus-command key.
Comment 5 remus draica 2004-11-16 14:20:28 UTC
Created attachment 33837 [details] [review]
proposed patch


This patch is not the best solution. A new event handling algorithm in
gnopernicus is the best one.
Comment 6 bill.haneman 2004-11-22 17:15:42 UTC
I think the patch could be reduced to one line by using the conditional operator
( a?b:c ) for the last param to speech_send_chunk.
Comment 7 remus draica 2004-11-26 13:37:35 UTC
Created attachment 34152 [details] [review]
reworked patch (with Bill's suggestion)
Comment 8 remus draica 2004-11-26 14:31:52 UTC
In case of presing shut up key, at next text-insert event, gnopernicus speaks
older information. This is because when shut up function is called list with
items waiting to be spoken is not freed. This bug is obvious when many items
with same priority but with non interruptible are sent to speech one after
other, like in case of previous patch.
Comment 9 remus draica 2004-11-26 14:32:38 UTC
Created attachment 34155 [details] [review]
proposed patch for speech issue
Comment 10 bill.haneman 2004-12-06 13:33:42 UTC
Comment on attachment 34155 [details] [review]
proposed patch for speech issue

looks good - I presume we're agreed that shutting up speech that hasn't yet
started being spoken is OK?  Even for highest-priority speech?

We might consider a small modification to this patch which doesn't remove
queued speech with a top priority.  This means that in order to stop urgent
speech, the user might have to press ShutUp twice, but it means that the first
shutup would stop the current speech and purge any non-critical messages, but
top priority speech would only be shutup if it was the 'current' thing being
spoken.

Or is it already the case that such urgent speech would interrupt the current
utterance, so the patch would only remove queued "top priority" speech if there
were multiple top-priority items in the speech system?
Comment 11 remus draica 2004-12-08 11:01:47 UTC
Bill, the real problem of this bug is solved in patch from comment #7. Please
review it.

Comment 12 bill.haneman 2004-12-08 11:30:57 UTC
the patch in comment 7 doesn't actually use my suggestion... however it looks OK.
Comment 13 remus draica 2004-12-09 15:44:44 UTC
Comment on attachment 34152 [details] [review]
reworked patch (with Bill's suggestion)


Patch applied to cvs head and gnome-2-8 branch.
Comment 14 remus draica 2004-12-09 15:45:09 UTC
Comment on attachment 34155 [details] [review]
proposed patch for speech issue


Patch applied to cvs head and gnome-2-8 branch.