UCI engines are not able to communicate

[Enrique Argones]

Binary package hint: gnome-games

Glaurung, Toga II, Fruit, ... None of the UCI engines are able to start a new match.

ProblemType: Bug
Architecture: amd64
CheckboxSubmission: bf7e46e2f13cb713d8d2a1da3a817a77
CheckboxSystem: edda5d4f616ca792bf437989cb597002
Date: Mon Jan 25 17:36:21 2010
DistroRelease: Ubuntu 9.10
InstallationMedia: Ubuntu 9.10 "Karmic Koala" - Release amd64 (20091027)
Package: gnome-games 1:2.28.0-0ubuntu1
PackageArchitecture: all
ProcEnviron:
 LANG=es_ES.UTF-8
 SHELL=/bin/bash
ProcVersionSignature: Ubuntu 2.6.31-17.54-generic
SourcePackage: gnome-games
Uname: Linux 2.6.31-17-generic x86_64

[Robert Ancell (robert-ancell)]

Proposing for backport to Karmic:
- Bug stops glchess from being able to use any UCI based engines (regressing from Jaunty)
- Fixed in upstream and Lucid in 2.29.6
- Patch is:
 Index: gnome-games-2.28.0/glchess/src/lib/uci.py
===================================================================
--- gnome-games-2.28.0.orig/glchess/src/lib/uci.py 2010-01-27 13:13:19.586447023 +1100
+++ gnome-games-2.28.0/glchess/src/lib/uci.py 2010-01-27 13:13:25.474042095 +1100
@@ -17,7 +17,6 @@
         self.__readyToConfigure = False
         self.__ready = False
         self.__inCallback = False
- self.__queuedCommands = None
         self.__positionCommand = 'position startpos'

     def logText(self, text, style):
- TEST CASE:
  Install fruit: apt-get install fruit
  Open Applications>Games>Chess and select Game>New menu. Start new game with white player set to "Fruit".
- Regression potential is very low as the fix is an incorrect variable initialisation

[Martin Pitt (pitti)]

Accepted gnome-games into karmic-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

[Otus (jan-varho)]

This is not fixed in gnome-games-2.28.0-0ubuntu2 from karmic-proposed. The patch above is not included in the update. Manually removing the line in question does fix the bug.

[Robert Ancell (robert-ancell)]

The patch was missing from debian/patches/series... I've uploaded an updated package

[Martin Pitt (pitti)]

Thanks Robert, accepted into karmic-proposed.

[Otus (jan-varho)]

The current package from -proposed solves the problem and I see no regressions. Thanks!

[Martin Pitt (pitti)]

Copied to karmic-updates.