Re: [Libreoffice] Tinderbox failure, last success: 2011-12-05 19:05:50

On 01/03/2012 01:49 PM, Matúš Kukan wrote:
> On 3 January 2012 03:37, Norbert Thiebaud<nthiebaud@gmail.com>  wrote:
> > I believe that one is linked to the gbuild conversion of cppuhelper...
>
> Hm, me too, but I have no idea how
> cppu::WeakImplHelper1<class
> com::sun::star::io::XOutputStream>::~WeakImplHelper1<class
> com::sun::star::io::XOutputStream>(void)
> could be defined in comphelpMSC.dll.

This is apparently yet another incarnation of a well-known mis-feature 
of the MS C++ compiler.  Mathias Bauer once explained it quite nicely on 
now-defunct GullFOSS, but still archived at 
<http://web.archive.org/web/20100504161204/http://blogs.sun.com/GullFOSS/entry/why_some_compilers_suck_more> 
"Why some compilers suck more than others."

<http://cgit.freedesktop.org/libreoffice/core/commit/?id=a90d4d5f036ab7785ba72038936c6c8427b74ba8> 
"Do not export whole class to avoid MS C++ implicitly exporting base 
template." should hopefully fix it.

Stephan

> > On Mon, Jan 2, 2012 at 8:12 PM, nthiebaud@gmail.com<nthiebaud@gmail.com>  wrote:
> > > Hi folks,
> > >
> > > One of you broke the build of LibreOffice with your commit :-(
> > > Please commit and push a fix ASAP!
> > >
> > > Full log available at http://tinderbox.libreoffice.org/MASTER/status.html
> > >
> > > Tinderbox info:
> > >
> > >   Box name: Win-x86@9-Voreppe Win32
> > >   Machine: CYGWIN_NT-6.1-WOW64 VOREPPELIBO 1.7.9(0.237/5/3) 2011-03-29 10:10 i686 Cygwin
> > >   Configured with: --with-distro=LibreOfficeWin32
> > > --disable-build-mozilla
> > > --with-mozilla-build=c:/mozilla-build
> > > --with-ant-home=c:/apache-ant-1.8.2
> > > --without-junit
> > > --with-lang=en-US fr de
> > > --without-helppack-integration
> > > --enable-binfilter
> > > --with-max-jobs=2
> > >
> > > Commits since the last success:
> > >
> > > ==== core ====
> > >   5436f57  gbuild: Rdb: blind fix for Windows, use gb_Helper_abbreviate_dirs_native
> > >   14c0e03  Clean non English strings in the code we do translation differently.
> > >   9cb0780  Online update: Enable online update even for the Linux release builds.
> > >   adad42f  Online update: Don't check for buildid at the user's end.
> > >   7489604  pack *.default files, too fdo#43923
> > >   b052839  -Wunused-but-set-variable
> > >   ca34393  Group a common test to avoid unnecessary calculations
> > >   fe36838  Do not repeat the same test twice
> > >   85d1ce2  Fix for fdo43460 Part XXVII getLength() to isEmpty()
> > >   829db12  Fix for fdo43460 Part XXVI getLength() to isEmpty()
> > >   6af5964  Fix for fdo43460 Part XXV getLength() to isEmpty()
> > >   93d2a2e  tabs ->  spaces
> > >   e8559f0  Remove unused code
> > >   9d54084  Replace dynamic array with std::vector.
> > >   13a40f3  Get rid of manual ref counting.
> > >   c28c09a  USE_INTROSPECTION_CACHE is always defined.
> > >   9bbb628  oops use == not = in dmake conditional
> > >   4f769af  Fix thinko: ==, not =
> > >   004f9bd  We have no MAKEFILE.MK or makefile.rc files
> > >   08c7038  The Android NDK calls the CPU just "arm", not "armel"
> > >   eee474c  scp2: cppuhelper is now gbuild'ified
> > >   dee53a3  Temporary hack around cppu_detail_getCppuType variants violating ODR.
> > >   8907d15  Temporary hack to work around autodoc bug
> > >   a768436  CppunitTest_cppuhelper_qa_weak depends on module bridges
> > >   b222aa8  cppuhelper: convert to gbuild
> > >   bd46aa5  cppuhelper: add visibility symbols as first step for porting to gbuild
> >
> > [...]
> >
> > >   .o d:/master/workdir/wntmsci12.pro/CxxObject/svtools/source/uno/genericunodialog.o 
> > > d:/master/workdir/wntmsci12.pro/CxxObject/svtools/source/uno/miscservices.o 
> > > d:/master/workdir/wntmsci12.pro/CxxObject/svtools/source/uno/popupmenucontrollerbase.o 
> > > d:/master/workdir/wntmsci12.pro/CxxObject/svtools/source/uno/popupwindowcontroller.o 
> > > d:/master/workdir/wntmsci12.pro/CxxObject/svtools/source/uno/statusbarcontroller.o 
> > > d:/master/workdir/wntmsci12.pro/CxxObject/svtools/source/uno/svtxgridcontrol.o 
> > > d:/master/workdir/wntmsci12.pro/CxxObject/svtools/source/uno/toolboxcontroller.o 
> > > d:/master/workdir/wntmsci12.pro/CxxObject/svtools/source/uno/treecontrolpeer.o 
> > > d:/master/workdir/wntmsci12.pro/CxxObject/svtools/source/uno/unocontroltablemodel.o 
> > > d:/master/workdir/wntmsci12.pro/CxxObject/svtools/source/uno/unogridcolumnfacade.o 
> > > d:/master/workdir/wntmsci12.pro/CxxObject/svtools/source/uno/unoevent.o 
> > > d:/master/workdir/wntmsci12.pro/CxxObject/svtools/source/uno/unoiface.o d:/master/workdir/wn
tmsci
> > >   12.pro/CxxObject/svtools/source/uno/unoimap.o 
> > > d:/master/workdir/wntmsci12.pro/CxxObject/svtools/source/uno/wizard/unowizard.o 
> > > d:/master/workdir/wntmsci12.pro/CxxObject/svtools/source/uno/wizard/wizardpagecontroller.o 
> > > d:/master/workdir/wntmsci12.pro/CxxObject/svtools/source/uno/wizard/wizardshell.o 
> > > d:/master/workdir/wntmsci12.pro/CxxObject/svtools/source/urlobj/inetimg.o 
> > > d:/master/workdir/wntmsci12.pro/CObject/svtools/source/filter/jpeg/jpegc.o
> > > [ build RES ] svtde
> > > icomphelp.lib(comphelpMSC.dll) : error LNK2005: "public: virtual __thiscall cppu::WeakImplHelper1<class 
> > > com::sun::star::io::XOutputStream>::~WeakImplHelper1<class com::sun::star::io::XOutputStream>(void)" 
> > > (??1?$WeakImplHelper1@VXOutputStream@io@star@sun@com@@@cppu@@UAE@XZ) already defined in filter.o
> > >    Creating library d:/master/workdir/wntmsci12.pro/LinkTarget/Library/svtool.lib and object 
> > > d:/master/workdir/wntmsci12.pro/LinkTarget/Library/svtool.exp
> > > LINK : warning LNK4098: defaultlib 'LIBCMT' conflicts with use of other libs; use 
> > > /NODEFAULTLIB:library
> > > d:/master/workdir/wntmsci12.pro/LinkTarget/Library/svtlo.dll : fatal error LNK1169: one or more 
> > > multiply defined symbols found
> > > make[2]: *** [/cygdrive/d/master/workdir/wntmsci12.pro/LinkTarget/Library/svtool.lib] Error 145
> > > make[2]: *** Waiting for unfinished jobs....
> > > dmake:  Error code 2, while making 'all'
> > > ======
> > > ERROR: error 65280 occurred while making /cygdrive/d/master/svtools/prj
> > > ERROR: error 65280 occurred while making /cygdrive/d/master/postgresql