Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#6672 closed defect (fixed)

Allow gnuchess gcompris activity to be installed in /home/olpc

Reported by: cscott Owned by: bdoin
Priority: blocker Milestone: Update.1
Component: gcompris-activities Version:
Keywords: release? Cc: mstone, walter
Blocked By: Blocking:
Deployments affected: Action Needed:
Verified: no

Description

The chess gcompris activity has required gnuchess in /usr/games. A recent patch to the upstream sources removed this requirement: http://svn.gnome.org/viewvc/gcompris/trunk/src/boards/chess.c?r1=3286&r2=3285&pathrev=3286

I believe that the .xo activity now just needs the Fedora gnuchess binary included in a 'bin' directory of activity, assuming that the activities CWD is its activity directory. Michael, is that correct? If not, then the gcompris activity (or sugar?) should be patched to add the activity directory to the local PATH; that should allow a binary included in the .xo file to be properly invoked.

Milestone set to update.1 because Peru wants chess in their builds.

Change History (5)

comment:1 Changed 7 years ago by mstone

First, the gnuchess rpm also installs a 3.6M file called /var/lib/games/gnuchess/book.dat that looks like it might be significant.

Second, sugar commit 177ee7220ac5cb4b6d9b8367ef10f2fa83103a49, from Oct. 9 caused sugar to add $BUNDLE/bin to the PATH variable used to launch activities.

Third, source code inspection suggests to me that activities CWD is equal to $SUGAR_BUNDLE_PATH.

Finally: be aware that support for $BUNDLE/lib in #5415 still open pending a stylistic change. This is not necessary for gnuchess, which has no interesting library dependencies, but may be an issue for other software.

comment:2 Changed 7 years ago by cscott

Is the Oct 9 sugar commit currently in update.1?

The book.dat is a library of 180,000 chess games used to compute gnuchess' opening book. It is not necessary, although it does make gnuchess play better. You can also compile smaller opening books which may be more appropriate for the XO.

gnuchess will read a book.dat file from its CWD; you can also modify 'bookbin' in gnuchess/src/book.h if you want to add additional search locations.

comment:3 Changed 7 years ago by bdoin

  • Status changed from new to assigned

could you please test these updated activities and tell me if this suits your expectations:
http://gcompris.net/incoming/chess_computer.activity.xo
http://gcompris.net/incoming/chess_movelearn.activity.xo
http://gcompris.net/incoming/chess_partyend.activity.xo

I now ship my gnuchess in the bundle bin directory.

BTW, I did the same with gnucap and our electric activity, please have a test:
http://gcompris.net/incoming/electric.activity.xo

comment:4 Changed 7 years ago by bdoin

  • Resolution set to fixed
  • Status changed from assigned to closed

Published a new release with the fix. Get it on http://wiki.laptop.org/go/GCompris

comment:5 Changed 7 years ago by Blaketh

  • Keywords release? added
Note: See TracTickets for help on using tickets.