On 10/4/05, Christoph Zwerschke <[EMAIL PROTECTED]> wrote:
> [EMAIL PROTECTED] 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=1
>
> 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=...") 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=1
>
> instead of
>
> Appserver.bat --AppServer.AutoReload=1
>
> I prefer 2) since it would be also more consistent with Appserver
> expecting "workdir=..." instead of "--workdir=...".
>
> But I would like to hear some opinions which is better before fixing it.
>
> -- Christoph

In supporting this form:

Appserver.bat AppServer.AutoReload=1

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

Appserver.bat --AppServer.AutoReload=1

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

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

-Chuck


-------------------------------------------------------
This SF.Net email is sponsored by:
Power Architecture Resource Center: Free content, downloads, discussions,
and more. http://solutions.newsforge.com/ibmarch.tmpl
_______________________________________________
Webware-discuss mailing list
Webware-discuss@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/webware-discuss

Reply via email to