[Chuck Esterbrook <chuck.esterbrook@gm...>]

On 10/4/05, Christoph Zwerschke <cito@...> wrote:
> jose@... wrote:
>  > I'm on windows and I like to turn autoreload on when I'm
>  > developing (I leave it off during production) so i would from the
>  > command line run
>  >  Appserver.bat --AppServer.AutoReload=3D1
>
> I haven't looked into it yet but I'm pretty sure it's  because the new
> Launch.py script uses getopt() to read the options which stops when
> option-like arguments are passed after a normal argument, and thus does
> not pass this argument to the AppServer. It passes only non-option-like
> arguments.
>
> Since the AppServer uses non-option-like arguments otherwise ("stop",
> "start", "workdir=3D...") I thought this was no problem, but I forgot to
> allow for the parameters allowing to modify the configuration.
>
> There are two ways to fix this:
>
> 1) Change Launch.py to also pass option-like arguments to the app server
> (this should be possible using gnu_getopt() instead of getopt())
>
> 2) Change ThreadedAppServer.py to use non-option like parameters for
> configuration changes, i.e.
>
> Appserver.bat AppServer.AutoReload=3D1
>
> instead of
>
> Appserver.bat --AppServer.AutoReload=3D1
>
> I prefer 2) since it would be also more consistent with Appserver
> expecting "workdir=3D..." instead of "--workdir=3D...".
>
> But I would like to hear some opinions which is better before fixing it.
>
> -- Christoph

In supporting this form:

Appserver.bat AppServer.AutoReload=3D1

it should be easy to strip '--' off the name in the event that the
user passed this:

Appserver.bat --AppServer.AutoReload=3D1

... if name.startswith('--'): name =3D name[2:] ...

and then we'll be compatible with prior versions of Webware.

-Chuck