Re: svnservice wrapper update

Stefan Küng wrote:
> Simon Large wrote:
>> SvnService.exe calls CreateProcess() to start svnserve.exe. Does it
>> really need the same CRT to do that? I thought you could start a
>> completely unrelated app that way.
>
> I'm not that familiar with the code. But if it uses CreateProcess(),
> then of course svnserve.exe is started as a separate process and it
> completely independent, which means you can use whatever runtime you want.
> But that also means it won't notice if svnserve.exe dies or does
> something unexpected - but that's maybe why it's called "wrapper" :)

     bItWorked = CreateProcess(
         szApp,
         szCommandLine,
         0, // default proc security
         0, // default thread security
         FALSE, // don't inherit handles
         CREATE_NEW_PROCESS_GROUP,
         0, // default environment
         0, // default working directory
         &startupInfo,
         &processInfo );

[snip error handling]

     eventHandles[0] = processInfo.hProcess;
     eventHandles[1] = g_quitEvent;
     err = WaitForMultipleObjects(
             2,
             eventHandles,
             FALSE, // don't wait for all events
             INFINITE );

Looks like it waits on the service manager and the process itself. It
should know that if svnserve died even if it doesn't get invited to the
autopsy.

Simon

-- 
        ___
   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.tigris.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tortoisesvn.tigris.org
For additional commands, e-mail: dev-help@tortoisesvn.tigris.org