Re: --headless broken with --enable-headless

2012-12-14 Thread Riccardo Magliocchetti
Hi Michael, Il 12/12/2012 22:36, Michael Meeks ha scritto: With 6a10f561d9e9cde17534bad7bf0497c9f74ece9e , master still crashes for me when running unoconv test suite: http://paste.ubuntu.com/1428111/ Odd indeed - I guess you'd need to valgrind it to work out what FMR/W is happening

Re: --headless broken with --enable-headless

2012-12-12 Thread Stephan Bergmann
On 10/01/2012 07:40 PM, Riccardo Magliocchetti wrote: Il 01/10/2012 18:11, Michael Meeks ha scritto: On Fri, 2012-09-28 at 20:26 +0200, Riccardo Magliocchetti wrote: Here you go, you'll find the valgrind output sans all the zlib / python related errors. Thanks a lot! update to git

Re: --headless broken with --enable-headless

2012-12-12 Thread Michael Meeks
Hi Stephan, On Wed, 2012-12-12 at 13:12 +0100, Stephan Bergmann wrote: On 10/01/2012 07:40 PM, Riccardo Magliocchetti wrote: nAcquireCount is always 0 and the patch above let me run two unoconv testsuite runs in a row without valgrind logs. It indeed looks like Application::Yield shall be

Re: --headless broken with --enable-headless

2012-12-12 Thread Stephan Bergmann
On 12/12/2012 01:12 PM, Stephan Bergmann wrote: Now, the interesting thing is that the Linux-specific CreateSalInstance (called from ImplSVMain - InitVCL before calling the application's Main in ImplSVMain) explicitly locks the solar mutex thanks to

Re: --headless broken with --enable-headless

2012-12-12 Thread Michael Meeks
On Wed, 2012-12-12 at 15:44 +0100, Stephan Bergmann wrote: Now, on yet closer inspection, it is only the versions of CreateSalInstance for Android and --enable-headless (vcl/android/androidinst.cxx, vcl/headless/headlessinst.cxx, both based on SvpSalInstance) that do not lock the solar

Re: --headless broken with --enable-headless

2012-12-12 Thread Stephan Bergmann
On 12/12/2012 04:50 PM, Michael Meeks wrote: On Wed, 2012-12-12 at 15:44 +0100, Stephan Bergmann wrote: Now, on yet closer inspection, it is only the versions of CreateSalInstance for Android and --enable-headless (vcl/android/androidinst.cxx, vcl/headless/headlessinst.cxx, both based on

Re: --headless broken with --enable-headless

2012-12-12 Thread Riccardo Magliocchetti
Hello Stephen, thanks a lot for debugging this issue Il 12/12/2012 15:44, Stephan Bergmann ha scritto: [snip] So http://cgit.freedesktop.org/libreoffice/core/commit/?id=5613bcfc6dc0ea45f8baeef9a8469a77c31365f5 Consistently acquire solar mutex in all CreateSalInstance variants appears to be

Re: --headless broken with --enable-headless

2012-12-12 Thread Tor Lillqvist
Now, on yet closer inspection, it is only the versions of CreateSalInstance for Android and --enable-headless (vcl/android/androidinst.cxx, vcl/headless/headlessinst.cxx, both based on SvpSalInstance) that do not lock the solar mutex -- while curiously enough both release it in

Re: --headless broken with --enable-headless

2012-12-12 Thread Michael Meeks
On Wed, 2012-12-12 at 19:55 +0100, Riccardo Magliocchetti wrote: thanks a lot for debugging this issue Thanks for helping chase this down :-) With 6a10f561d9e9cde17534bad7bf0497c9f74ece9e , master still crashes for me when running unoconv test suite:

Re: --headless broken with --enable-headless

2012-12-12 Thread Michael Meeks
On Wed, 2012-12-12 at 21:19 +0200, Tor Lillqvist wrote: Well, firstly I am not sure if this SalInstance code gets involved at all currently in our Android app(s) (all GUI is done using Android Java code, no VCL event loop etc is running) Event loop or not - you don't want two threads

Re: --headless broken with --enable-headless

2012-10-01 Thread Michael Meeks
Hi Riccardo, On Fri, 2012-09-28 at 20:26 +0200, Riccardo Magliocchetti wrote: Here you go, you'll find the valgrind output sans all the zlib / python related errors. Thanks a lot! update to git 7a3a8a254363801bf6893b23d67f86a7461f8f3b and more or less same valgrind errors. See attached

Re: --headless broken with --enable-headless

2012-09-17 Thread Dag Wieers
On Fri, 7 Sep 2012, Riccardo Magliocchetti wrote: i have libreoffice crashing early when running unoconv [1] test suite (cd tests make). Attached gdbtrace.log Libreoffice from yesterday master configured: ./autogen.sh --without-java --enable-headless and started:

Re: --headless broken with --enable-headless

2012-09-14 Thread Michael Meeks
Hi Riccardo, On Fri, 2012-09-07 at 20:33 +0200, Riccardo Magliocchetti wrote: i have libreoffice crashing early when running unoconv [1] test suite (cd tests make). Attached gdbtrace.log Nasty :-) glad there is a test suite though :-) Does anyone have a clue what can be broken?

Re: --headless broken with --enable-headless

2012-09-13 Thread Riccardo Magliocchetti
Hello Dag, Il 12/09/2012 21:59, Dag Wieers ha scritto: On Fri, 7 Sep 2012, Riccardo Magliocchetti wrote: i have libreoffice crashing early when running unoconv [1] test suite (cd tests make). Attached gdbtrace.log [snip] I would recommend using the master branch (which was recently

--headless broken with --enable-headless

2012-09-07 Thread Riccardo Magliocchetti
Hello, i have libreoffice crashing early when running unoconv [1] test suite (cd tests make). Attached gdbtrace.log Libreoffice from yesterday master configured: ./autogen.sh --without-java --enable-headless and started: /usr/local/lib/libreoffice/program/soffice --backtrace

Re: --headless broken with --enable-headless

2012-09-07 Thread John Smith
On Fri, Sep 7, 2012 at 8:33 PM, Riccardo Magliocchetti riccardo.magliocche...@gmail.com wrote: Does anyone can give it a try with a build without --enable-headless please? Urm. Do *what* exactly, without --enable-headless ? - John Smith. ___

Re: --headless broken with --enable-headless

2012-09-07 Thread Joop Kiefte
Run that command I would guess... 2012/9/7 John Smith lbalba...@gmail.com: On Fri, Sep 7, 2012 at 8:33 PM, Riccardo Magliocchetti riccardo.magliocche...@gmail.com wrote: Does anyone can give it a try with a build without --enable-headless please? Urm. Do *what* exactly, without