Bug 28480 - X-Session != $TERM=xterm
X-Session != $TERM=xterm
Status: RESOLVED WONTFIX
Product: UCS
Classification: Unclassified
Component: Update - Release updates
UCS 3.0
Other Linux
: P5 normal (vote)
: ---
Assigned To: UCS maintainers
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-09-12 17:18 CEST by Philipp Hahn
Modified: 2018-04-14 13:51 CEST (History)
0 users

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional): Troubleshooting
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Philipp Hahn univentionstaff 2012-09-12 17:18:06 CEST
Im preup.sh Skript wird versucht zu überprüfen, ob der Updater aus einer laufenden X11-Sitzung gestartet wird. Das erfolgt per
  if [ "$TERM" = "xterm" ]; then

Loggt man sich von einerm xterm per ssh auf den Rechner ein, so ist auch dort TERM=xterm gesetzt, weil TERM mitgenommen wird, d.h. die Meldung ist dann falsch: Auf dem entfernten Rechner läuft dann gerade keine X11-Sitzung, die durch ein gdm-Restart beendet werden könnte.

Außerdem gibt es noch andere X11-Terminal-Emulatoren wie [u]rxvt, die dann z.B. TERM=rxvt(-unicode) setzen, was auch von dem Test nicht erfasst wird.

Sinnvollerweise scheint mir folgendes zu sein:
  if [ -n "$WINDOWID" ] && (which xprop && xprop -id "$WINDOWID") >/dev/null 2>&1
  then

WINDOWID wird von xterm, konsole, rxvt auf die ID des X11-Fensters gesetzt, die per xprop auf Gültigkeit getestet wird.
Eine screen-Session erbt diese, so daß dort dann --ignoreterm übergeben werden muß. Das wäre noch mit einem zusätzlichen „&& [ "$TERM" != screen ]” abfangbar, allerdings gibt es ja noch weitere screen-ähnliche Programme wie "tmux" <http://en.wikipedia.org/wiki/Terminal_multiplexer>, die dann ggf. auch explizit abgefangen werden müssten, von daher würde ich die besser weglassen.
Comment 1 Stefan Gohmann univentionstaff 2017-06-16 20:36:53 CEST
This issue has been filed against UCS 3. UCS 3 is out of the normal maintenance and many UCS components have vastly changed in UCS 4.

If this issue is still valid, please change the version to a newer UCS version otherwise this issue will be automatically closed in the next weeks.
Comment 2 Stefan Gohmann univentionstaff 2017-08-08 07:07:50 CEST
This issue has been filed against UCS 3.0.

UCS 3.0 is out of maintenance and many UCS components have vastly changed in later releases. Thus, this issue is now being closed.

If this issue still occurs in newer UCS versions, please use "Clone this bug" or reopen this issue. In this case please provide detailed information on how this issue is affecting you.