From: Tim R. <ti...@pr...> - 2002-12-11 20:21:01
|
Count this as a +1 vote for restricting list posts to list members only. -- - Tim Roberts, ti...@pr... Providenza & Boekelheide, Inc. |
From: Tim R. <ti...@pr...> - 2002-12-12 17:27:02
|
I've been lusting after the auto-restart feature since I first read about it here, so I bit the bullet earlier this week and fetched from CVS instead of using the canned distributions as I have in the past. Now that I've turned on auto-restart, it doesn't appear to do what I expect. Looking into the code, it seems pretty clear why that is so. Is it really true that, as of today, auto-restart is only supported on Win32? It appears that the implementation just terminates with a known exit code (3), relying on some outer wrapper to detect that and restart the server. However, the only wrapper that does so today is the Win32-only AppServer.bat. Since I'm on Linux, batch files aren't much use, and the auto-restart feature instantly becomes a less useful auto-terminate feature. Is that right? I can certainly go implement the auto-restart (perhaps by using execve in ThreadedAppServer), but before I do so, I'd like to make sure I haven't missed something. -- - Tim Roberts, ti...@pr... Providenza & Boekelheide, Inc. |
From: Ian B. <ia...@co...> - 2002-12-12 17:36:37
|
On Thu, 2002-12-12 at 11:25, Tim Roberts wrote: > Now that I've turned on auto-restart, it doesn't appear to do what I expect. > Looking into the code, it seems pretty clear why that is so. Is it really true > that, as of today, auto-restart is only supported on Win32? It appears that > the implementation just terminates with a known exit code (3), relying on some > outer wrapper to detect that and restart the server. However, the only wrapper > that does so today is the Win32-only AppServer.bat. Since I'm on Linux, batch > files aren't much use, and the auto-restart feature instantly becomes a less > useful auto-terminate feature. No, the AppServer script does it too. However, if you used MakeAppWorkDir, the version of AppServer in your working directory won't check for the exit code. You'll have to copy a new version over. -- Ian Bicking Colorstudy Web Development ia...@co... http://www.colorstudy.com PGP: gpg --keyserver pgp.mit.edu --recv-keys 0x9B9E28B7 4869 N Talman Ave, Chicago, IL 60625 / (773) 275-7241 |
From: Tim R. <ti...@pr...> - 2002-12-12 18:06:18
|
On 12 Dec 2002 11:37:41 -0600, Ian Bicking wrote: >On Thu, 2002-12-12 at 11:25, Tim Roberts wrote: >> Now that I've turned on auto-restart, it doesn't appear to do what I expect. >> Looking into the code, it seems pretty clear why that is so. Is it really >> true that, as of today, auto-restart is only supported on Win32? > >No, the AppServer script does it too. However, if you used >MakeAppWorkDir, the version of AppServer in your working directory won't >check for the exit code. You'll have to copy a new version over. Ah, yes, I see that now. My problem was that I use the "webkit" script in /etc/rc.d/init.d, which launches "python Launch.py ThreadedAppServer" directly instead of using the AppServer wrapper. I've now changed the "webkit" script to use $WEBKIT_DIR/AppServer, and all appears to be happy again. Is that worth a patch? -- - Tim Roberts, ti...@pr... Providenza & Boekelheide, Inc. |
From: Tim R. <ti...@pr...> - 2002-12-12 18:11:43
|
OK, now I'd like to post yet another question that shows I haven't been paying attention. We now have a config switch ('OldStyleActions') to choose between two different behaviors for _action_ handling. I figured, what the heck, I'm a modern guy, let's switch over. I eliminated the second parameter from all my action methods, and while most things work fine, some don't. The difference, it turns out, is that the action handling no longer calls methodNameForAction. I have several submit buttons called _action_ where I want the visible label to be capitalized, but the method name to be all lower case, so methondNameForAction was a call to x.lower(). I presume this change was made because the _action_ labels can now come in in several different formats, making case-insensitivity more complicated. So, how are people handling this? Have you changed your method names to be mixed-case? -- - Tim Roberts, ti...@pr... Providenza & Boekelheide, Inc. |
From: Ian B. <ia...@co...> - 2002-12-12 18:22:26
|
You should change those buttons to be like: <input type="submit" name="_action_new" value="New"> i.e., name it _action_methodToBeCalled (that's the new-style actions). On Thu, 2002-12-12 at 12:10, Tim Roberts wrote: > OK, now I'd like to post yet another question that shows I haven't been paying > attention. > > We now have a config switch ('OldStyleActions') to choose between two different > behaviors for _action_ handling. I figured, what the heck, I'm a modern guy, > let's switch over. I eliminated the second parameter from all my action > methods, and while most things work fine, some don't. > > The difference, it turns out, is that the action handling no longer calls > methodNameForAction. I have several submit buttons called _action_ where I > want the visible label to be capitalized, but the method name to be all lower > case, so methondNameForAction was a call to x.lower(). > > I presume this change was made because the _action_ labels can now come in in > several different formats, making case-insensitivity more complicated. So, how > are people handling this? Have you changed your method names to be mixed-case? > > -- > - Tim Roberts, ti...@pr... > Providenza & Boekelheide, Inc. > > > > > ------------------------------------------------------- > This sf.net email is sponsored by: > With Great Power, Comes Great Responsibility > Learn to use your power at OSDN's High Performance Computing Channel > http://hpc.devchannel.org/ > _______________________________________________ > Webware-discuss mailing list > Web...@li... > https://lists.sourceforge.net/lists/listinfo/webware-discuss -- Ian Bicking <ia...@co...> |
From: Tavis R. <ta...@re...> - 2002-12-11 20:42:40
|
+2 On December 11, 2002 12:19 pm, Tim Roberts wrote: > Count this as a +1 vote for restricting list posts to list members only= =2E |