[Sugar-devel] [Dextrose] [PATCH sugar-toolkit] Insert activity root path in front of the reset of sys.path

[Martin Langhoff]

On Mon, Feb 7, 2011 at 4:40 AM, Sascha Silbe
<sascha-ml-reply-to-2011-2 at silbe.org> wrote:
>> Otherwise it is possible to include, eg, system modules before local ones.
>
> Which is how Python 2 usually works. Your patch would make us deviate
> from upstream, making it harder to debug.

I've done extensive programming with Python, Perl, PHP and similar
scripting languages over the last, um, 14 years. Local modules/libs
always override system ones.

To wit, note the first entry at

$ python
Python 2.7 (r27:82500, Sep 16 2010, 18:03:06)
[GCC 4.5.1 20100907 (Red Hat 4.5.1-3)] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import sys
>>> sys.path
['', '/usr/lib/python27.zip', '/usr/lib/python2.7',
'/usr/lib/python2.7/plat-linux2', '/usr/lib/python2.7/lib-tk',
'/usr/lib/python2.7/lib-old', '/usr/lib/python2.7/lib-dynload',
'/usr/lib/python2.7/site-packages',
'/usr/lib/python2.7/site-packages/PIL',
'/usr/lib/python2.7/site-packages/gst-0.10',
'/usr/lib/python2.7/site-packages/gtk-2.0',
'/usr/lib/python2.7/site-packages/setuptools-0.6c11-py2.7.egg-info',
'/usr/lib/python2.7/site-packages/webkit-1.0']

The library the developer meant to include is a lot easier to debug
(to the developer involved) than "system library changed API
subtly/silently/in a relnote encoded with a reference to a bug number.

The behaviour sought by Aleksey's patch makes sense. I am surprised
that CWD isn't set to SUGAR_BUNDLE_PATH, maybe that needs to get fixed
instead.

cheers,


m
--
 martin.langhoff at gmail.com
 martin at laptop.org -- Software Architect - OLPC
 - ask interesting questions
 - don't get distracted with shiny stuff  - working code first
 - http://wiki.laptop.org/go/User:Martinlanghoff