By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.
Bug 1956 - starttde: invalid 'trap' command
Summary: starttde: invalid 'trap' command
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: tdebase (show other bugs)
Version: R14.0.0 [Trinity]
Hardware: All Linux
: P5 normal
Assignee: Francois Andriot
URL:
Depends on:
Blocks:
 
Reported: 2014-02-22 09:25 CST by Francois Andriot
Modified: 2014-06-01 14:22 CDT (History)
3 users (show)

See Also:
Compiler Version:
TDE Version String:
Application Version:
Application Name:

Attachments
tdebase: fix trap command in starttde (427 bytes, patch)
2014-04-20 12:15 CDT, Francois Andriot 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Francois Andriot 2014-02-22 09:25:30 CST 
http://git.trinitydesktop.org/cgit/tdebase/tree/starttde
Line 84

  trap '[starttde] echo GOT SIGHUP' HUP


This causes the following error message when stopping TDE:
  /opt/trinity/bin/starttde: ligne1: [starttde] : commande introuvable
(which means "command not found" in english)

The "[starttde] word should be AFTER the echo command, not before.

  trap 'echo [starttde] GOT SIGHUP' HUP
Comment 1 Darrell 2014-02-23 15:28:33 CST 
How are you invoking this? I don't see the message with a normal exit or when using Ctrl+Alt+Backspace.

I am using run level 3 and startx.
Comment 2 Francois Andriot 2014-02-23 15:49:07 CST 
I'm running nested TDE session using Xephyr.

From a terminal:
  xinit /opt/trinity/bin/starttde -- /usr/bin/Xephyr :1 -screen 1024x768

This allows  to see all TDE messages in terminal.

Note: avoid doing this from inside TDE, it causes strange behaviour in main TDE session. I personnaly run Xephyr from KDE4 session.
Comment 3 Francois Andriot 2014-04-20 12:15:34 CDT 
Created attachment 2037 [details]
tdebase: fix trap command in starttde

Patch to fix the invalid trap command.
Comment 4 Darrell 2014-06-01 14:22:57 CDT 
Patch was pushed to git in commit 8155652c 2014-05-01.