Re: [BUILD] Linux32 snapshot configure
Hi, On 14.08.2013 19:29, Andrew Rist wrote: Here is the current configure statement for the Linux32 bot - is this correct, and what changes would make it better? ./configure \ --with-jdk-home=/usr/lib/jvm/java-6-openjdk \ --with-epm-url=http://www.msweet.org/files/project2/epm-3.7-source.tar.gz \ --with-dmake-url=http://dmake.apache-extras.org.codespot.com/files/dmake-4.12.tar.bz2; \ --enable-verbose \ --without-stlport \ --enable-category-b \ --enable-opengl \ --enable-dbus \ --enable-gstreamer \ --with-package-format=installed rpm deb \ --enable-bundled-dictionaries \ --with-lang=ast cs de el en-GB en-US es fi fr gd gl hu it ja km ko nl pl pt pt-BR ru sk sl ta zh-CN zh-TW ca eu he hi id lt sv th tr \ --with-vendor=Apache OpenOffice buildbot \ --with-build-version=%(today)s-Rev.%(got_revision)s \ These configure options correspond to the ones which I am using on my Ubuntu 11.10 32bit VM. I will perform a clean build on my VM in order to check, if I can reproduce the build breaker from our Linux 32bit build bot. Stay tuned for my results. Best regards, Oliver. - To unsubscribe, e-mail: dev-unsubscr...@openoffice.apache.org For additional commands, e-mail: dev-h...@openoffice.apache.org
Re: [BUILD] Linux32 snapshot configure
Hi Andrew, On 14.08.2013 19:29, Andrew Rist wrote: Here is the current configure statement for the Linux32 bot - is this correct, and what changes would make it better? [...] --with-package-format=installed rpm deb \ I'd suggest to change that line to --with-package-format=archive rpm deb The installed target is great for developers who want to run a build directly, but for buildbots it doesn't make sense. The archive target is great for quickly testing a build. Herbert - To unsubscribe, e-mail: dev-unsubscr...@openoffice.apache.org For additional commands, e-mail: dev-h...@openoffice.apache.org
Re: [BUILD] Linux32 snapshot configure
Hi, On 15.08.2013 09:00, Oliver-Rainer Wittmann wrote: Hi, On 14.08.2013 19:29, Andrew Rist wrote: Here is the current configure statement for the Linux32 bot - is this correct, and what changes would make it better? ./configure \ --with-jdk-home=/usr/lib/jvm/java-6-openjdk \ --with-epm-url=http://www.msweet.org/files/project2/epm-3.7-source.tar.gz \ --with-dmake-url=http://dmake.apache-extras.org.codespot.com/files/dmake-4.12.tar.bz2; \ --enable-verbose \ --without-stlport \ --enable-category-b \ --enable-opengl \ --enable-dbus \ --enable-gstreamer \ --with-package-format=installed rpm deb \ --enable-bundled-dictionaries \ --with-lang=ast cs de el en-GB en-US es fi fr gd gl hu it ja km ko nl pl pt pt-BR ru sk sl ta zh-CN zh-TW ca eu he hi id lt sv th tr \ --with-vendor=Apache OpenOffice buildbot \ --with-build-version=%(today)s-Rev.%(got_revision)s \ These configure options correspond to the ones which I am using on my Ubuntu 11.10 32bit VM. I will perform a clean build on my VM in order to check, if I can reproduce the build breaker from our Linux 32bit build bot. Stay tuned for my results. Herbert, Andre and myself had a closer look on the error log of the recent Linux 32bit build bot build (#456) and at the corresponding code. It seems that there is a programming error in method SvgSvgNode::getCurrentViewPort(). I am working on it. Best regards, Oliver. - To unsubscribe, e-mail: dev-unsubscr...@openoffice.apache.org For additional commands, e-mail: dev-h...@openoffice.apache.org
Re: [BUILD] Linux32 snapshot configure
Hi Oliver, Oliver-Rainer Wittmann schrieb: Hi, Herbert, Andre and myself had a closer look on the error log of the recent Linux 32bit build bot build (#456) and at the corresponding code. It seems that there is a programming error in method SvgSvgNode::getCurrentViewPort(). I am working on it. Please tell me, what is the problem there. I wrote it. Kind regards Regina - To unsubscribe, e-mail: dev-unsubscr...@openoffice.apache.org For additional commands, e-mail: dev-h...@openoffice.apache.org
Re: [BUILD] Linux32 snapshot configure
Hi, On 15.08.2013 10:10, Regina Henschel wrote: Hi Oliver, Oliver-Rainer Wittmann schrieb: Hi, Herbert, Andre and myself had a closer look on the error log of the recent Linux 32bit build bot build (#456) and at the corresponding code. It seems that there is a programming error in method SvgSvgNode::getCurrentViewPort(). I am working on it. Please tell me, what is the problem there. I wrote it. What Herbert, Andre and myself have figured out is the following: Error message from Linux 32bit build bot says: /home/buildslave20/slave20/openoffice-linux32-nightly/build/main/svgio/source/svgreader/svgsvgnode.cxx: In member function 'virtual const basegfx::B2DRange* svgio::svgreader::SvgSvgNode::getCurrentViewPort() const': /home/buildslave20/slave20/openoffice-linux32-nightly/build/main/svgio/source/svgreader/svgsvgnode.cxx:636:71: error: taking address of temporary [-fpermissive] /home/buildslave20/slave20/openoffice-linux32-nightly/build/main/svgio/source/svgreader/svgsvgnode.cxx:679:75: error: taking address of temporary [-fpermissive] /home/buildslave20/slave20/openoffice-linux32-nightly/build/main/svgio/source/svgreader/svgsvgnode.cxx:698:67: error: taking address of temporary [-fpermissive] make: *** [/home/buildslave20/slave20/openoffice-linux32-nightly/build/main/solver/400/unxlngi6.pro/workdir/CxxObject/svgio/source/svgreader/svgsvgnode.o] Error 1 The corresponding code line are: - 636: return basegfx::B2DRange(fX, fY, fX+fW, fY+fH); - 679: return basegfx::B2DRange(fX, fY, fX+fW, fY+fH); - 698: return basegfx::B2DRange(0.0, 0.0, fW, fH); Thus, a pointer to a temporary created object is returned here and the return type of this method is basegfx::B2DRange*. The life time of such an object ends when the method ends. Thus, accessing the returned pointer outside the method means accessing 'dead' memory. Best regards, Oliver. - To unsubscribe, e-mail: dev-unsubscr...@openoffice.apache.org For additional commands, e-mail: dev-h...@openoffice.apache.org
Re: [BUILD] Linux32 snapshot configure
On 15.08.2013 10:27, Oliver-Rainer Wittmann wrote: Error message from Linux 32bit build bot says: /home/buildslave20/slave20/openoffice-linux32-nightly/build/main/svgio/source/svgreader/svgsvgnode.cxx: In member function 'virtual const basegfx::B2DRange* svgio::svgreader::SvgSvgNode::getCurrentViewPort() const': /home/buildslave20/slave20/openoffice-linux32-nightly/build/main/svgio/source/svgreader/svgsvgnode.cxx:636:71: error: taking address of temporary [-fpermissive] This bug was only found by the buildbot because it has a recent version of the compiler. This proves yet again that buildbots as code quality testers are very valuable. There is an interesting discrepancy: - builds for a wide test audience are best created for the greatest common denominator target which on Linux requires that they are built on very old systems - compile testing should best be done on very up-to-date buildbots but the usability of their build output is usually quite limited Maybe we should reserve some buildbots to be always up-to-date whose purpose is just for compile testing. Only their build reports with warnings and errors should be kept and the builds themselves be should be discarded. Herbert - To unsubscribe, e-mail: dev-unsubscr...@openoffice.apache.org For additional commands, e-mail: dev-h...@openoffice.apache.org
Re: [BUILD] Linux32 snapshot configure
On Aug 15, 2013 11:00 AM, Herbert Duerr h...@apache.org wrote: On 15.08.2013 10:27, Oliver-Rainer Wittmann wrote: Error message from Linux 32bit build bot says: /home/buildslave20/slave20/openoffice-linux32-nightly/build/main/svgio/source/svgreader/svgsvgnode.cxx: In member function 'virtual const basegfx::B2DRange* svgio::svgreader::SvgSvgNode::getCurrentViewPort() const': /home/buildslave20/slave20/openoffice-linux32-nightly/build/main/svgio/source/svgreader/svgsvgnode.cxx:636:71: error: taking address of temporary [-fpermissive] This bug was only found by the buildbot because it has a recent version of the compiler. This proves yet again that buildbots as code quality testers are very valuable. There is an interesting discrepancy: - builds for a wide test audience are best created for the greatest common denominator target which on Linux requires that they are built on very old systems - compile testing should best be done on very up-to-date buildbots but the usability of their build output is usually quite limited Maybe we should reserve some buildbots to be always up-to-date whose purpose is just for compile testing. Only their build reports with warnings and errors should be kept and the builds themselves be should be discarded. If we make such a setup correct on the centOS box, the footprint of the vm wille be smalll, and can even be tarted on demand. I have a similar setup, the vm only contains build environment, no svn files, they are attached as local network drives (very fast). Starting/stopping can be done by making a buildbot job for the centos. that way we can have multiple test env. rgds jan i Herbert - To unsubscribe, e-mail: dev-unsubscr...@openoffice.apache.org For additional commands, e-mail: dev-h...@openoffice.apache.org
Re: [BUILD] Linux32 snapshot configure
Hi Oliver, Oliver-Rainer Wittmann schrieb: Hi, On 15.08.2013 10:10, Regina Henschel wrote: Hi Oliver, Oliver-Rainer Wittmann schrieb: Hi, Herbert, Andre and myself had a closer look on the error log of the recent Linux 32bit build bot build (#456) and at the corresponding code. It seems that there is a programming error in method SvgSvgNode::getCurrentViewPort(). I am working on it. Please tell me, what is the problem there. I wrote it. What Herbert, Andre and myself have figured out is the following: Error message from Linux 32bit build bot says: /home/buildslave20/slave20/openoffice-linux32-nightly/build/main/svgio/source/svgreader/svgsvgnode.cxx: In member function 'virtual const basegfx::B2DRange* svgio::svgreader::SvgSvgNode::getCurrentViewPort() const': /home/buildslave20/slave20/openoffice-linux32-nightly/build/main/svgio/source/svgreader/svgsvgnode.cxx:636:71: error: taking address of temporary [-fpermissive] /home/buildslave20/slave20/openoffice-linux32-nightly/build/main/svgio/source/svgreader/svgsvgnode.cxx:679:75: error: taking address of temporary [-fpermissive] /home/buildslave20/slave20/openoffice-linux32-nightly/build/main/svgio/source/svgreader/svgsvgnode.cxx:698:67: error: taking address of temporary [-fpermissive] make: *** [/home/buildslave20/slave20/openoffice-linux32-nightly/build/main/solver/400/unxlngi6.pro/workdir/CxxObject/svgio/source/svgreader/svgsvgnode.o] Error 1 The corresponding code line are: - 636: return basegfx::B2DRange(fX, fY, fX+fW, fY+fH); - 679: return basegfx::B2DRange(fX, fY, fX+fW, fY+fH); - 698: return basegfx::B2DRange(0.0, 0.0, fW, fH); Thus, a pointer to a temporary created object is returned here and the return type of this method is basegfx::B2DRange*. The life time of such an object ends when the method ends. Thus, accessing the returned pointer outside the method means accessing 'dead' memory. Sorry, MS compiler not even gave a warning. I'm not experienced and still struggling with C++. Therefore I had ask for review. Sadly it slipped through. If you knew a solution, I would be happy. I suggest, you replace the method getCurrentViewport with the previous version of it. The crash has happened in method decomposeSvgNode and the solution there is independent of method getCurrentViewport. In content the method getCurrentViewport has to return 4 double values, which are calculated inside the method. Changing it that way, would result in a change in signature with deep impact on the whole class. I think, a solutions needs to be discussed with Armin. But as far as I know, Armin is now in vacation. Kind regards Regina - To unsubscribe, e-mail: dev-unsubscr...@openoffice.apache.org For additional commands, e-mail: dev-h...@openoffice.apache.org
Re: [BUILD] Linux32 snapshot configure
Hi Regina, On 15.08.2013 11:26, Regina Henschel wrote: Hi Oliver, Oliver-Rainer Wittmann schrieb: Hi, On 15.08.2013 10:10, Regina Henschel wrote: Hi Oliver, Oliver-Rainer Wittmann schrieb: Hi, Herbert, Andre and myself had a closer look on the error log of the recent Linux 32bit build bot build (#456) and at the corresponding code. It seems that there is a programming error in method SvgSvgNode::getCurrentViewPort(). I am working on it. Please tell me, what is the problem there. I wrote it. What Herbert, Andre and myself have figured out is the following: Error message from Linux 32bit build bot says: /home/buildslave20/slave20/openoffice-linux32-nightly/build/main/svgio/source/svgreader/svgsvgnode.cxx: In member function 'virtual const basegfx::B2DRange* svgio::svgreader::SvgSvgNode::getCurrentViewPort() const': /home/buildslave20/slave20/openoffice-linux32-nightly/build/main/svgio/source/svgreader/svgsvgnode.cxx:636:71: error: taking address of temporary [-fpermissive] /home/buildslave20/slave20/openoffice-linux32-nightly/build/main/svgio/source/svgreader/svgsvgnode.cxx:679:75: error: taking address of temporary [-fpermissive] /home/buildslave20/slave20/openoffice-linux32-nightly/build/main/svgio/source/svgreader/svgsvgnode.cxx:698:67: error: taking address of temporary [-fpermissive] make: *** [/home/buildslave20/slave20/openoffice-linux32-nightly/build/main/solver/400/unxlngi6.pro/workdir/CxxObject/svgio/source/svgreader/svgsvgnode.o] Error 1 The corresponding code line are: - 636: return basegfx::B2DRange(fX, fY, fX+fW, fY+fH); - 679: return basegfx::B2DRange(fX, fY, fX+fW, fY+fH); - 698: return basegfx::B2DRange(0.0, 0.0, fW, fH); Thus, a pointer to a temporary created object is returned here and the return type of this method is basegfx::B2DRange*. The life time of such an object ends when the method ends. Thus, accessing the returned pointer outside the method means accessing 'dead' memory. Sorry, MS compiler not even gave a warning. I'm not experienced and still struggling with C++. Therefore I had ask for review. Sadly it slipped through. If you knew a solution, I would be happy. I suggest, you replace the method getCurrentViewport with the previous version of it. The crash has happened in method decomposeSvgNode and the solution there is independent of method getCurrentViewport. In content the method getCurrentViewport has to return 4 double values, which are calculated inside the method. Changing it that way, would result in a change in signature with deep impact on the whole class. I think, a solutions needs to be discussed with Armin. But as far as I know, Armin is now in vacation. Please have a look at the corresponding issue 122600 and its mail thread - I have attached a solution. Best regards, Oliver. - To unsubscribe, e-mail: dev-unsubscr...@openoffice.apache.org For additional commands, e-mail: dev-h...@openoffice.apache.org
[BUILD] Linux32 snapshot configure
Here is the current configure statement for the Linux32 bot - is this correct, and what changes would make it better? ./configure \ --with-jdk-home=/usr/lib/jvm/java-6-openjdk \ --with-epm-url=http://www.msweet.org/files/project2/epm-3.7-source.tar.gz \ --with-dmake-url=http://dmake.apache-extras.org.codespot.com/files/dmake-4.12.tar.bz2; \ --enable-verbose \ --without-stlport \ --enable-category-b \ --enable-opengl \ --enable-dbus \ --enable-gstreamer \ --with-package-format=installed rpm deb \ --enable-bundled-dictionaries \ --with-lang=ast cs de el en-GB en-US es fi fr gd gl hu it ja km ko nl pl pt pt-BR ru sk sl ta zh-CN zh-TW ca eu he hi id lt sv th tr \ --with-vendor=Apache OpenOffice buildbot \ --with-build-version=%(today)s-Rev.%(got_revision)s \
Re: [BUILD] Linux32 snapshot configure
On Wed, Aug 14, 2013 at 10:29 AM, Andrew Rist andrew.r...@oracle.comwrote: Here is the current configure statement for the Linux32 bot - is this correct, and what changes would make it better? ./configure \ --with-jdk-home=/usr/lib/jvm/**java-6-openjdk \ --with-epm-url=http://www.**msweet.org/files/project2/epm-** 3.7-source.tar.gzhttp://www.msweet.org/files/project2/epm-3.7-source.tar.gz \ --with-dmake-url=http://**dmake.apache-extras.org.** codespot.com/files/dmake-4.12.**tar.bz2http://dmake.apache-extras.org.codespot.com/files/dmake-4.12.tar.bz2 \ --enable-verbose \ --without-stlport \ --enable-category-b \ --enable-opengl \ --enable-dbus \ --enable-gstreamer \ --with-package-format=**installed rpm deb \ --enable-bundled-dictionaries \ --with-lang=ast cs de el en-GB en-US es fi fr gd gl hu it ja km ko nl pl pt pt-BR ru sk sl ta zh-CN zh-TW ca eu he hi id lt sv th tr \ --with-vendor=Apache OpenOffice buildbot \ --with-build-version=%(today)**s-Rev.%(got_revision)s \ OK, I'm not exactly sure why you're asking this -- I have meant to take a look at the build logs for the current instance since it has been failing, but no time yet -- but are you referring to the existing Linux 32 bot with your question or the newly proposed one? We have discussed using java 7 instead of 6. My recent build went fine but of course not thoroughly tested many aspects. Please -- --enable-verbose -- - MzK When in doubt, cop an attitude. -- Cat laws