Re: [RFC] add --pidfile switch to write libo pid to a file

2013-03-23 Thread Riccardo Magliocchetti
Hi Stephan, Il 19/03/2013 11:47, Stephan Bergmann ha scritto: Setting up the listeners for the --accept arguments passed from soffice/oosplash to soffice.bin is done in Desktop::RegisterServices (desktop/source/app/appinit.cxx), so that might also be a good place to write the pid. (Setting up li

Re: [RFC] add --pidfile switch to write libo pid to a file

2013-03-19 Thread Riccardo Magliocchetti
Hi Michael, Il 19/03/2013 11:35, Michael Meeks ha scritto: Hi Riccardo, On Tue, 2013-03-19 at 10:06 +0100, Riccardo Magliocchetti wrote: soffice.bin is the one i see in netstat so it looked the right one to me, but handling this from soffice would be lot more easier. Right - that is

Re: [RFC] add --pidfile switch to write libo pid to a file

2013-03-19 Thread Stephan Bergmann
On 03/19/2013 11:35 AM, Michael Meeks wrote: I think we can speculate as to the benefits of launching pagein/oosplash in the headless case; it would be reasonably reasonable to tweak: Note that oosplash also restarts soffice.bin in the scenario of "first call to soffice for a given Use

Re: [RFC] add --pidfile switch to write libo pid to a file

2013-03-19 Thread Stephan Bergmann
On 03/19/2013 10:06 AM, Riccardo Magliocchetti wrote: Il 18/03/2013 22:18, Stephan Bergmann ha scritto: On 03/16/2013 07:55 PM, Riccardo Magliocchetti wrote: attached a patch to add a switch to have libreoffice store its pid in a file. I need that so that when libo runs in headless mode i can h

Re: [RFC] add --pidfile switch to write libo pid to a file

2013-03-19 Thread Michael Meeks
Hi Riccardo, On Tue, 2013-03-19 at 10:06 +0100, Riccardo Magliocchetti wrote: > soffice.bin is the one i see in netstat so it looked the right one to me, > but handling this from soffice would be lot more easier. Right - that is doing all the work. Of course after the 'soffice' wrapper sh

Re: [RFC] add --pidfile switch to write libo pid to a file

2013-03-19 Thread Stephan Bergmann
On 03/19/2013 07:08 AM, Tor Lillqvist wrote: .It would thus look more natural to be interested in pid files from the client-facing process rather than from soffice.bin processes (which your patch would implement). Ah, but his very point is that he is working on the --enable-headless thing, wher

Re: [RFC] add --pidfile switch to write libo pid to a file

2013-03-19 Thread Riccardo Magliocchetti
Hi Tor, Il 19/03/2013 07:08, Tor Lillqvist ha scritto: .It would thus look more natural to be interested in pid files from the client-facing process rather than from soffice.bin processes (which your patch would implement). Ah, but his very point is that he is working on the --enable-headless

Re: [RFC] add --pidfile switch to write libo pid to a file

2013-03-19 Thread Riccardo Magliocchetti
Hi Stephan, Il 18/03/2013 22:18, Stephan Bergmann ha scritto: > On 03/16/2013 07:55 PM, Riccardo Magliocchetti wrote: >> attached a patch to add a switch to have libreoffice store its pid in a >> file. I need that so that when libo runs in headless mode i can have a >> supervisor process checking

Re: [RFC] add --pidfile switch to write libo pid to a file

2013-03-18 Thread Tor Lillqvist
>.It would thus look more natural to be interested in pid files > from the client-facing process rather than from soffice.bin processes (which > your patch would implement). Ah, but his very point is that he is working on the --enable-headless thing, where (if I understand correctly) there is no "

Re: [RFC] add --pidfile switch to write libo pid to a file

2013-03-18 Thread Stephan Bergmann
On 03/16/2013 07:55 PM, Riccardo Magliocchetti wrote: attached a patch to add a switch to have libreoffice store its pid in a file. I need that so that when libo runs in headless mode i can have a supervisor process checking if libo is running and in case just restart it. The error handling is a

Re: [RFC] add --pidfile switch to write libo pid to a file

2013-03-18 Thread Riccardo Magliocchetti
Hi Michael, Il 18/03/2013 16:21, Michael Meeks ha scritto: On Mon, 2013-03-18 at 14:41 +0100, Riccardo Magliocchetti wrote: The standard practice is to use just a plain file with the pid inside, other tools won't parse an ini file just to find the pid. Seems unlikely to cause any rea

Re: [RFC] add --pidfile switch to write libo pid to a file

2013-03-18 Thread Michael Meeks
On Mon, 2013-03-18 at 14:41 +0100, Riccardo Magliocchetti wrote: > The standard practice is to use just a plain file with the pid inside, > other tools won't parse an ini file just to find the pid. Seems unlikely to cause any real harm to me :-) Out of interest, did you see the

Re: [RFC] add --pidfile switch to write libo pid to a file

2013-03-18 Thread Riccardo Magliocchetti
Hi Jan, Il 18/03/2013 14:08, Jan Holesovsky ha scritto: Hi Riccardo, Riccardo Magliocchetti píše v So 16. 03. 2013 v 19:55 +0100: attached a patch to add a switch to have libreoffice store its pid in a file. I need that so that when libo runs in headless mode i can have a supervisor process c

Re: [RFC] add --pidfile switch to write libo pid to a file

2013-03-18 Thread Jan Holesovsky
Hi Riccardo, Riccardo Magliocchetti píše v So 16. 03. 2013 v 19:55 +0100: > attached a patch to add a switch to have libreoffice store its pid in a > file. I need that so that when libo runs in headless mode i can have a > supervisor process checking if libo is running and in case just restart