Re: [Shinken-devel] small question

[nap <nap...@gm...>]

Hi,

No rpoblem for the as, it was quickly fixed :)

For the sys.path, it's a bug I've got in centos 5, python2.4, direct launch
(no installation, just untar/launch) with the ndo module (the module do not
found the shinken lib). That's why I put this back again an the sys.path are
so hard to test :)


Jean

2011/2/3 Grégory Starck <g.s...@gm...>

> Jean,
>
> thx for the fix on my try: ... except Exception as e"   ;   I  quite often
> forget (with lot of others things  ;)) that this isn't 2.4 compatible.  (I
> hope there are not too much that I recently have introduced).
>
> but about the next commit :
>
>  I was asking myself in what situation the sys.path change was also needed
> while the "import shinken" succeeded ..
>
>      import shinken
> +    if hasattr(sys.modules['__main__'], '__file__'):
> +        my_path = os.path.abspath(sys.modules['__main__'].__file__)
> +        elts = os.path.dirname(my_path).split(os.sep)[:-1]
> +        elts.append('shinken')
> +        sys.path.append(os.sep.join(elts)
>
> on my side I don't need at all this ; and more : I don't see, when the
> "import shinken" succeed then, why it'd be needed to still change the
> sys.path to explicitly put in in it the shinken "package" directory.
>
> Or am I missing something.. :-?
>
> greg
>
>
> ------------------------------------------------------------------------------
> Special Offer-- Download ArcSight Logger for FREE (a $49 USD value)!
> Finally, a world-class log management solution at an even better
> price-free!
> Download using promo code Free_Logger_4_Dev2Dev. Offer expires
> February 28th, so secure your free ArcSight Logger TODAY!
> http://p.sf.net/sfu/arcsight-sfd2d
> _______________________________________________
> Shinken-devel mailing list
> Shi...@li...
> https://lists.sourceforge.net/lists/listinfo/shinken-devel
>
>