Bug#303360: bittornado: please enable psyco support in the package

2005-04-07 Thread Micah Anderson
I've actually decided to disable psyco support. I spoke with the
upstream developers of Bittorrent and Bittornado and they strongly
suggested *not* enabling psyco support automatically just because it
is installed on people's systems. They said that it increases memory
use and can cause reliability issues.

Since this version I am wanting to be stablized for Sarge, I would
prefer to err on the side of reliability. They said that users should
just enable psyco support themselves if they wish to have it running
(either in the PSYCO.py file, or the individual modules).

Perhaps after Sarge releases we can have a version with psyco support
so it can be tested out. 

I will leave python-psyco as a Suggests entry however.

Micah

On Thu, 07 Apr 2005, Pierre Habouzit wrote:

 Le Jeu 7 Avril 2005 05:04, vous avez écrit :
  tag 303360 +pending
  thanks
 
  I'll add a patch that enables psyco support if it exists, and change
  the package to Suggest: python-psyco. I wont Depend on it because
  psyco is i386 only, and until there are architecture specific Depends
  support in dpkg, I would rather bittornado be available for all
  architectures, rather than just i386.
 
  On Wed, 06 Apr 2005, Pierre Habouzit wrote:
 maybe /usr/lib/python2.3/site-packages/BitTornado/PSYCO.py could
   e.g. be :
  
   try
 import psyco
 psyco = 1
   except:
 psyco = 0
 
  This is probably what I'll do, except that the try requires a colon
  after it.
 
 lol ;)
 
 or sth more subtle like :
  
 from BittornadoConf import enable_psycho
 psyco = enable_psycho
  
 with BittornadoConf beeing a module in /etc/bittornado (see how
   mailman does this with mm_cfg.py)
 
  If you submit a patch that does this, I'll consider it.
 ok
 
 thank you for the quick fix anyway !
 -- 
 ·O·  Pierre Habouzit
 ··O
 OOOhttp://www.madism.org




signature.asc
Description: Digital signature


Bug#303360: bittornado: please enable psyco support in the package

2005-04-06 Thread Pierre Habouzit
Package: bittornado
Version: 0.3.10-3
Severity: wishlist


  bittornado has psyco support (see /usr/share/doc/bittornado/README-Psyco.txt)

  but editting that file manually is not very satisfactory.  

  maybe /usr/lib/python2.3/site-packages/BitTornado/PSYCO.py could e.g.
be :

try
  import psyco
  psyco = 1
except:
  psyco = 0


  or sth more subtle like :

  from BittornadoConf import enable_psycho
  psyco = enable_psycho

  with BittornadoConf beeing a module in /etc/bittornado (see how
mailman does this with mm_cfg.py)


  because enabling Psyco makes bittornado use 40% of the CPU to 7% here,
with the same torrents, and same conditions.


-- System Information:
Debian Release: 3.1
  APT prefers testing
  APT policy: (500, 'testing')
Architecture: i386 (i686)
Kernel: Linux 2.6.11.6
Locale: [EMAIL PROTECTED], [EMAIL PROTECTED] (charmap=ISO-8859-15)

Versions of packages bittornado depends on:
ii  python2.3.5-1An interactive high-level object-o

-- no debconf information


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#303360: bittornado: please enable psyco support in the package

2005-04-06 Thread Micah Anderson
tag 303360 +pending
thanks

I'll add a patch that enables psyco support if it exists, and change
the package to Suggest: python-psyco. I wont Depend on it because
psyco is i386 only, and until there are architecture specific Depends
support in dpkg, I would rather bittornado be available for all
architectures, rather than just i386.

On Wed, 06 Apr 2005, Pierre Habouzit wrote:

   maybe /usr/lib/python2.3/site-packages/BitTornado/PSYCO.py could e.g.
 be :
 
 try
   import psyco
   psyco = 1
 except:
   psyco = 0

This is probably what I'll do, except that the try requires a colon
after it.

   or sth more subtle like :
 
   from BittornadoConf import enable_psycho
   psyco = enable_psycho
 
   with BittornadoConf beeing a module in /etc/bittornado (see how
 mailman does this with mm_cfg.py)

If you submit a patch that does this, I'll consider it.

Micah


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]