[jds-review] Review : gnome-games : glchess not functional on Solaris

[Stephen Browne]

You are removing the menu entry but we will still ship the binary that
doesn't work?

On Thu, 2007-04-05 at 13:17 +0100, Matt Keenan wrote:
> glchess is not functional on Solaris, and from what I can see this is 
> due to forkpty, missing from
> Solaris land.
> 
> As gnome-games are not supported, I've logged a bug in bugzilla, as an 
> interm solution I am removing
> glchess from our menu's so that all users don't see the option to run 
> it, very easy to add it back
> in when/if this get's resolved.
> 
> Just requires a change to gnome-games.spec (svn diff below)
> 
> bugster :  http://monaco.sfbay/detail.jsf?cr=6537569
> bugzilla : http://bugzilla.gnome.org/show_bug.cgi?id=426538
> 
> Cheers
> 
> Matt
> 
> 
> Index: base-specs/gnome-games.spec
> ===================================================================
> --- base-specs/gnome-games.spec (revision 11040)
> +++ base-specs/gnome-games.spec (working copy)
> @@ -77,8 +77,8 @@
>  ## things we just don't want in the package
>  rm -rf $RPM_BUILD_ROOT/var/lib/scrollkeeper
>  find $RPM_BUILD_ROOT -type f -name "*.pyo" -exec rm -f {} ';'
> +rm -f $RPM_BUILD_ROOT/usr/share/applications/glchess.desktop
>  
> -
>  ## install desktop files
>  
>  %find_lang %{name}
> @@ -137,6 +137,9 @@
>  %config %{_sysconfdir}/sound/events/*
>  
>  %changelog
> +* Thu Apr 05 2007 - matt.keenan at sun.com
> +- Remove glchess.desktop, until glchess is made functional on Solaris
> +- Bugzilla : 426538
>  * Wed Mar 21 2007 - damien.carbery at sun.com
>  - Bump to 2.18.0.1.
>  * Tue Mar 13 2007 - damien.carbery at sun.com
>