Re: [Webware-devel] [UPDATE] Autoreload feature in CVS

[Chuck E. <Chu...@ya...>]

On Sunday 10 November 2002 11:26 am, Ian Bicking wrote:
> On Sun, 2002-11-10 at 02:27, Chuck Esterbrook wrote:
> > On Friday 25 October 2002 08:48 am, Jason Hildebrand wrote:
> > > Well right now ImportSpy hooks itself in regardless of whether
> > > AutoReload is activated or not.  I would have preferred to have
> > > AutoReloadingAppServer install the ImportSpy hook, but I also
> > > wanted ImportSpy to get hooked in as soon as possible, so that
> > > most modules get monitored.  Do you see a better way of doing
> > > this?
> >
> > Just a thought: Could the top of Launch.py import the sys module
> > and check the command line args to determine if ImportSpy should be
> > loaded?
> >
> > This would obviously require a command line arg either to AppServer
> > and perhaps a convenience script named AutoReloadingAppServer
> > script.
> >
> > I'm not sure if ImportSpy has any performance impact on
> > non-autoreloading servers (e.g., your non-autoreloading production
> > box).
>
> This has been changed since then -- now the hook is only put in when
> auto-reloading is on (so there's no overhead).

Ah. I saw "import ImportSpy" at the top of Launch.py which made me 
think it was required and doing something. I then went reading and 
found the above message.

Launch.py seems to work fine without it, even when AutoReload is on, so 
I have removed it.

-Chuck