Opened 16 years ago

Last modified 16 years ago

#1208 enhancement closed fixed (fixed)

Remove --psyco option from Trial command line

Reported by: Jonathan Lange Owned by:
Priority: normal Milestone:
Component: trial Keywords:
Cc: radix, Jean-Paul Calderone, spiv, Jonathan Lange Branch:
Author:

Description


Change History (7)

comment:1 Changed 16 years ago by Jonathan Lange 

a) I don't think we should support this
b) I don't think anyone uses this

Might be safe to remove without deprecating

comment:2 Changed 16 years ago by Jonathan Lange 

What do you guys think? Can I remove it now?

comment:3 Changed 16 years ago by spiv 

Psyco-ising code isn't really trial's responsibility.  I'm happy for this to be
removed, but then I've never used it :)

That said, it really should be deprecated before removal.  I'm a bit fussy like
that.

comment:4 Changed 16 years ago by Jonathan Lange 

Mind if I deprecate it and make it a no-op at the same time?

comment:5 Changed 16 years ago by spiv 

Sounds reasonable.  Turning it into a no-op shouldn't break anything, just harm
(or help!) performance.

comment:6 Changed 16 years ago by Jonathan Lange 

Now a no-op and spews a DeprecationWarning.  Resolving this issue.  Someone
remind me to remove it after 2.1 is released.

comment:7 Changed 11 years ago by <automation>

Owner: Jonathan Lange deleted
Note: See TracTickets for help on using tickets.