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 th

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 thread

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: > http://paste.ubuntu.com/1428

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 > DestroySal

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 "Consistently acquire solar mutex in all CreateSalInstance variants" appears to be

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 SvpSalI

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 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
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 sh

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 7a3a8a25436

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

2012-10-01 Thread Riccardo Magliocchetti
Hi Michael, Il 01/10/2012 18:11, Michael Meeks ha scritto: 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 7a3a8a254363801bf6893b23d67f86a7461f8f3

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 atta

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: /usr/local/lib/libreoffic

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 rele

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

2012-09-07 Thread Joop Kiefte
Run that command I would guess... 2012/9/7 John Smith : > On Fri, Sep 7, 2012 at 8:33 PM, Riccardo Magliocchetti > 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 John Smith
On Fri, Sep 7, 2012 at 8:33 PM, Riccardo Magliocchetti wrote: > Does anyone can give it a try with a build without --enable-headless please? Urm. Do *what* exactly, without --enable-headless ? - John Smith. ___ LibreOffice mailing list LibreOffice@lists

--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 "--accept=so