[Sugar-devel] Check for binaries, not config files, when deciding if a tool is installed

[Sascha Silbe]

On Thu, Sep 10, 2009 at 02:41:15PM +0200, Jonas Smedegaard wrote:

> Not sure, but it seems from src/sugar/activity/activityfactory.py  
> (approx. line 250) of recent sugar-toolkit that rainbow is used if the 
>   path /etc/olpc-security exists on the system.
The intention is to be able to de/activate Rainbow use inside Sugar 
without uninstalling Rainbow. Maybe we should parse the file contents 
(i.e. store something in it) instead of just checking for existence.

> If that is correct, then I recommend changing that to instead test for 
>   the existence of some binary, as the Debian packaging system (and 
> most  probably other packaging systems as well) preserve configuration 
> files  when removing a package.
An additional check for existence of the binary might be a good idea.

CU Sascha

-- 
http://sascha.silbe.org/
http://www.infra-silbe.de/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 489 bytes
Desc: Digital signature
Url : http://lists.sugarlabs.org/archive/sugar-devel/attachments/20090910/10a3b6a6/attachment.pgp