Re: [REVIEW] smoketest - gbuild, TestExtension.idl needed ?

2012-02-17 Thread Michael Meeks
Hi Tor, On Tue, 2012-02-14 at 19:23 +0200, Tor Lillqvist wrote: In the feature/android branch I have added the fw[eiklm], sfx, sb, svt and tk libraries to libmerged. Working on verifying that the last addition didn't break anything; I found a nice direct reference to the name tk in

Re: [REVIEW] smoketest - gbuild, TestExtension.idl needed ?

2012-02-17 Thread Michael Meeks
Hi Matus, On Wed, 2012-02-15 at 01:05 +0100, Matúš Kukan wrote: It's going quite well. With tml's patch [1] we can add more. He is doing great job, mostly in the android branch as he already said. Cool :-) yep, I saw. Current tail_build blocker is connectivity/. Peter was working on

Re: [REVIEW] smoketest - gbuild, TestExtension.idl needed ?

2012-02-15 Thread Tor Lillqvist
On 14 February 2012 19:23, Tor Lillqvist t...@iki.fi wrote: I also merged sal and sal_textenc (in the --enable-mergelibs case). Hmm, I experimented with this but apparently didn't commit and then did a git reset and lost the changes, or something. Will have a look again if I still have the

Re: [REVIEW] smoketest - gbuild, TestExtension.idl needed ?

2012-02-15 Thread Matúš Kukan
On 15 February 2012 13:35, Tor Lillqvist t...@iki.fi wrote: On 14 February 2012 19:23, Tor Lillqvist t...@iki.fi wrote: I also merged sal and sal_textenc (in the --enable-mergelibs case). Hmm, I experimented with this but apparently didn't commit and then did a git reset and lost the

Re: [REVIEW] smoketest - gbuild, TestExtension.idl needed ?

2012-02-15 Thread Tor Lillqvist
It was something more than this: ? Umm, no, that was it. Now I am confused, why don't I see that in my master checkout after merging the branch to master? Git is a mystery. --tml ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org

Re: [REVIEW] smoketest - gbuild, TestExtension.idl needed ?

2012-02-15 Thread Stephan Bergmann
On 02/14/2012 06:23 PM, Tor Lillqvist wrote: I also merged sal and sal_textenc (in the --enable-mergelibs case). I'm not sure this is not counter-productive. For Android and iOS it might be vital to get sal_textenc merged, to reduce the overall number of dynamic libraries. But for other

Re: [REVIEW] smoketest - gbuild, TestExtension.idl needed ?

2012-02-15 Thread Stephan Bergmann
On 02/15/2012 03:11 PM, Tor Lillqvist wrote: I'm not sure this is not counter-productive. For Android and iOS it might be vital to get sal_textenc merged, Well, --enable-mergelibs is probably actually intended for Android only, I think? I wasn't sure about the status of --enable-mergelibs,

Re: [REVIEW] smoketest - gbuild, TestExtension.idl needed ?

2012-02-15 Thread Tor Lillqvist
I wasn't sure about the status of --enable-mergelibs, whether it is only something Android-specific or whether it is also used to enable that general merge/LTO endeavor. I don't know either, really. But anyway, I will make the sal_textenc+sal merge Android-specific and not mergelibs-specific;)

Re: [REVIEW] smoketest - gbuild, TestExtension.idl needed ?

2012-02-14 Thread Stephan Bergmann
On 02/13/2012 11:21 PM, Matúš Kukan wrote: On 12 February 2012 23:51, Matúš Kukanmatus.ku...@gmail.com wrote: Do not push them without improvement, make check fails for me. Attached diff solves one problem but still there is something wrong. Well, I was fooled by

Re: [REVIEW] smoketest - gbuild, TestExtension.idl needed ?

2012-02-14 Thread Michael Meeks
On Mon, 2012-02-13 at 23:21 +0100, Matúš Kukan wrote: I'd like to push attached patch. Hopefully it will work also for others and there is nothing wrong with removing TestExtension.rdb from TestExtension.oxt. Nice ;-) I was wondering - how are we doing wrt. getting more of the

Re: [REVIEW] smoketest - gbuild, TestExtension.idl needed ?

2012-02-14 Thread Tor Lillqvist
I was wondering - how are we doing wrt. getting more of the libraries into tail_build and libmerged ? In the feature/android branch I have added the fw[eiklm], sfx, sb, svt and tk libraries to libmerged. Working on verifying that the last addition didn't break anything; I found a nice direct

Re: [REVIEW] smoketest - gbuild, TestExtension.idl needed ?

2012-02-14 Thread Matúš Kukan
On 14 February 2012 18:12, Michael Meeks michael.me...@suse.com wrote:        Nice ;-) I was wondering - how are we doing wrt. getting more of the libraries into tail_build and libmerged ? :-) I wonder could we get 'basic' into that ? svtools ? some of the 'helper' libraries ? :-) It's going

[REVIEW] smoketest - gbuild, TestExtension.idl needed ?

2012-02-13 Thread Matúš Kukan
On 12 February 2012 23:51, Matúš Kukan matus.ku...@gmail.com wrote: Do not push them without improvement, make check fails for me. Attached diff solves one problem but still there is something wrong. Well, I was fooled by smoketest/com/sun/star/comp/smoketest/makefile.mk There is javamaker