Launch Server command requires xterm

Bug #32820 reported by Emmet Hikory
4
Affects Status Importance Assigned to Milestone
teg (Ubuntu)
Fix Released
Medium
Motu Games Team

Bug Description

When an internal server is launched from the TEG GUI, it is launched in an xterm, and will not launch if the xterm package is not installed.

Either teg should Recommend / Suggest xterm, or client/misc.c should be modified to use x-terminal-emulator rather than xterm when launching an local server.

As xterm is not in the default install, the following diff should probably be applied.

--- misc.c.orig 2006-02-25 19:49:00.000000000 +0900
+++ misc.c 2006-02-25 19:49:54.000000000 +0900
@@ -200,7 +200,7 @@

                char buffer[100];
                close_descriptors();
- args[0] = "xterm";
+ args[0] = "x-terminal-emulator";
                args[1] = "-e";
                args[2] = BINDIR"/tegserver";
                args[3] = "--port";

Revision history for this message
Emmet Hikory (persia) wrote : Debdiff for fixed version

This patch worked for me on AMD64.

Revision history for this message
Emmet Hikory (persia) wrote : Patch including debian sync

Debian also has an update to TEG, including gconf fixes that should probably be applied. Attached is a debdiff of both the described change and the sync with debian.

Emmet Hikory (persia)
Changed in teg:
assignee: nobody → motugames
Revision history for this message
Zak B. Elep (zakame) wrote :

Hey, thanks for the bug report! =)

I think this ought to be better fixed by requesting an UVF exception; see https://lists.ubuntu.com/archives/ubuntu-motu/2006-February/000545.html .

Thanks,

Changed in teg:
status: Unconfirmed → Needs Info
Revision history for this message
Emmet Hikory (persia) wrote :

This bug is not fixed by any upstream version, so no UVF would help. I have attached the diff for the newer debian version for the convenience of any maintainer, did they want to include the upstream version. WIthout changing misc.c, the package depends on xterm for no good reason (and without the appropriate control Depends: entry).

Changed in teg:
status: Needs Info → Confirmed
Revision history for this message
Barry deFreese (bddebian) wrote :

Uploaded thanks Emmett. Please watch for this and close this bug if appropriate.

Changed in teg:
status: Confirmed → Fix Committed
Revision history for this message
Emmet Hikory (persia) wrote :

Built on all architectures.

Changed in teg:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.