Summary: | >=app-office/libreoffice-4.0.0.2: fails to build with dev-libs/boost-1.53.0_beta1 | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Diego Elio Pettenò (RETIRED) <flameeyes> |
Component: | New packages | Assignee: | Gentoo Office Team <office> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | cJ-gentoo, dlan, polynomial-c, robink, tom, vityokster |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
URL: | https://tinderboxlogs.s3.amazonaws.com/tbamd64.excelsior.flameeyes.eu/app-office%3Alibreoffice-4.0.0.2%3A20130126-221405.html | ||
Whiteboard: | Fixed-in: 4.0.4 | ||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 454100 | ||
Attachments: |
build.log.xz
libreoffice-4.0.0.3-boost-1.43.patch patch upstreamed earlier with 1 additional hunk for svgdata.cxx emerge --info for libreoffice-4.0.0.3 built ok with gcc 4.7.2, boost 1.53.0 using patch upstreamed earlier with 1 additional hunk for svgdata.cxx build.log.xz Patch for libreoffice 4.0.1.2 with boost 1.53.0 build.log.xz |
Description
Diego Elio Pettenò (RETIRED)
2013-01-27 12:09:56 UTC
I submitted my patch for Bug 456162 - app-office/libreoffice-3.4.6.3: fails to build with dev-libs/boost-1.53.0 upstream: https://gerrit.libreoffice.org/2064 Upstream merged my patch. However the patch merged by upstream has not yet been tested to fix the build problem with libreoffice-4.0.0.3 with boost-1.53.0 - there is the possibility that the patch may be incomplete - it may need more hunks added if there was code changes between libreoffice-3.6.4.3 and 4.0.0.3 that need tweaks to build with boost-1.53.0. I will test it when I can, which may take a while as it takes so long to build. Created attachment 338770 [details]
build.log.xz
Ich tried to build libreoffice-4.0.0.3 with that merged patch on top but the build still fails. See attached build.log.
/var/tmp/portage/app-office/libreoffice-4.0.0.3/work/libreoffice-4.0.0.3/vcl/source/gdi/svgdata.cxx:174:19: error: no matching function for call to ‘boost::shared_array<unsigned char>::shared_array(NULL)’ Created attachment 339008 [details, diff]
libreoffice-4.0.0.3-boost-1.43.patch
Tentative patch for libreoffice, builds and runs, but not fully tested.
Created attachment 339046 [details, diff] patch upstreamed earlier with 1 additional hunk for svgdata.cxx This is the patch I upstreamed earlier, with the addition of the hunk at the end for the compile error with svcdata.cxx Tested to compile ok with these use flags: argus ~ # fquery u app-office/libreoffice USE flags for app-office/libreoffice-4.0.0.3 +branding : << no description >> +cups : << no description >> +gtk : << no description >> +vba : << no description >> +webdav : << no description >> aqua : Include support for the Mac OS X Aqua (Carbon/Cocoa) GUI + bluetooth : Enables Bluetooth Support + dbus : Enable dbus support for anything that needs it (gpsd, gnomemeeting, etc) debug : Enable extra debug codepaths, like asserts and extra output. If you want to get meaningful backtraces see http://www.gentoo.org/proj/en/qa/backtraces.xml eds : Enables support for Evolution-Data-Server (EDS) elibc_FreeBSD : << no description >> gnome : Adds GNOME support gstreamer : Adds support for media-libs/gstreamer (Streaming media) gtk3 : Enable highly experimental gtk3 frontend java : Adds support for Java jemalloc : Use dev-libs/jemalloc for allocations + kde : Adds support for KDE (K Desktop Environment) libreoffice_extensions_nlpsolver : << no description >> + libreoffice_extensions_presenter-minimizer : << no description >> libreoffice_extensions_scripting-beanshell : << no description >> libreoffice_extensions_scripting-javascript : << no description >> libreoffice_extensions_wiki-publisher : << no description >> + mysql : Adds mySQL Database support odk : Build the Office Development Kit + opengl : Adds support for OpenGL (3D graphics) postgres : Adds support for the postgresql database + python_single_target_python2_7 : << no description >> python_single_target_python3_3 : << no description >> + python_targets_python2_7 : << no description >> + python_targets_python3_3 : << no description >> telepathy : Enable document colaboration features using telepathy communication framework. + test : Workaround to pull in packages needed to run with FEATURES=test. Portage-2.1.2 handles this internally, so don't set it in make.conf/package.use anymore argus ~ # Created attachment 339052 [details]
emerge --info for libreoffice-4.0.0.3 built ok with gcc 4.7.2, boost 1.53.0 using patch upstreamed earlier with 1 additional hunk for svgdata.cxx
(In reply to comment #5) > Created attachment 339008 [details, diff] [details, diff] > libreoffice-4.0.0.3-boost-1.43.patch > > Tentative patch for libreoffice, builds and runs, but not fully tested. Thanks for testing this, and submitting the patch. I applied my patch to portage, as that is the one I tested. My patch is missing 3 hunks from the patch that cj provided. (1) drawinglayer/source/primitive2d/textlayoutdevice.cxx - the patch says the previous line was: - if(!rStdRefDevice()) but in my copy of the source I compiled it doesn't have the () on the end. (2) editeng/source/editeng/eerdll.cxx b/editeng/source/editeng/eerdll.cxx Same (1) patch says the previous line was: - if ( !m_aStdRefDevice() ) but in my copy of the source I compiled it doesn't have the () on the end. (3) slideshow/source/engine/animationnodes/animationnodefactory.cxx This hunk applies ok. Just removing the parentheses from the previous lines in the first 2 hunks would allow them to apply of course. For all 3 hunks: Since they are already in a boolean context, the existing code compiles fine with gcc 4.7.2. However I wonder if there is some other compiler version or CXXFLAGS or USE flags that makes the changes necessary? If there are, it would be neat if you could please comment in the bug report, the compiler error messages and compiler version would be very useful. Tested your latest patch with use: -aqua bluetooth branding cups dbus -debug -eds -gnome gstreamer gtk gtk3 java -jemalloc -kde -mysql -odk opengl -postgres -telepathy vba webdav -test libreoffice_extensions: -nlpsolver presenter-minimizer -scripting-beanshell -scripting-javascript -wiki-publisher python_single_target: python2_7 -python3_3 python_targets: python2_7 -python3_3 Created attachment 341222 [details]
build.log.xz
Stillfailing with libreoffice-4.0.1.2
Created attachment 342322 [details, diff]
Patch for libreoffice 4.0.1.2 with boost 1.53.0
Compiled with gcc 4.7.2, boost 1.53.0.
It includes this hunk like cJ suggested earlier (thanks):
--- libreoffice-4.0.1.2-orig/editeng/source/editeng/eerdll.cxx 2013-02-28 01:02:37.000000000 +1100
+++ libreoffice-4.0.1.2/editeng/source/editeng/eerdll.cxx 2013-03-17 10:57:01.036699391 +1100
@@ -195,7 +195,7 @@
OutputDevice* GlobalEditData::GetStdRefDevice()
{
- if ( !m_aStdRefDevice )
+ if ( !static_cast< bool >(m_aStdRefDevice) )
{
m_aStdRefDevice.reset(new VirtualDevice);
m_aStdRefDevice->SetMapMode( MAP_TWIP );
Committed patch for libreoffice 4.0.1.2 to build with boost 1.53.0. I need to upstream it. @Mark: did you submit this patch to gerrit? Created attachment 344466 [details]
build.log.xz
Still failing with libreoffice-4.0.2.2
libreoffice-4.0.1.2-boost-1.53.0.patch still applies to libreoffice-4.0.2.2. Out of curiosity... why don't you libreoffice guys just keep this patch in future libreoffice ebuilds until it fails to apply? As long as it applies cleanly chances are very high upstream didn't do anything to fix this problem, right? I won't apply any patches except distro ones recursively for live ebuilds. The motivation is to get people upstream it. We at libo accept almost all patches, so it is just sending them to gerrit.libreoffice.org, and I simply don't have the time/urge now as the 53 is quite masked and we even didn't try full master build there. (In reply to comment #13) > @Mark: did you submit this patch to gerrit? I just went to now, patch tells me "Reversed (or previously applied) patch detected!" so it appears that it is upstreamed now (by someone, thanks to whoever did it). I also confirm that libreoffice-4.0.2.2 compiles fine and tests pass with libreoffice-4.0.1.2-boost-1.53.0.patch. Still failing with libreoffice-4.0.3.3 4.0.1.2-boost-1.53.0.patch still works! Guys this starts to become extremely annoying. You have a working patch and your only excuse is to wait for upstream who obviously is unable to apply the fix in a reasonable amount of time. All you have to do is keeping this patch until upstream finally fixed it. What's so hard about this? (In reply to comment #18) > Still failing with libreoffice-4.0.3.3 > 4.0.1.2-boost-1.53.0.patch still works! > > Guys this starts to become extremely annoying. You have a working patch and > your only excuse is to wait for upstream who obviously is unable to apply > the fix in a reasonable amount of time. > All you have to do is keeping this patch until upstream finally fixed it. > What's so hard about this? I already said tht the patch needs to be submitted. Which nobody did. I don't give square fuck about boost thats being masked, when its unmasked I will do the work myself. But until then, DIY over gerrit.libreoffice.org. Also other chance for me testing the patch properly is using new boost in fedora/opensuse, but both distros yet not provide the new boost either. Fixed in git and will be in 4.0.4 release. This release will be published in a week after 2.6. If you want it in 4.0.3 the patch is just bit different against this one so feel free to add the patch line into the ebuild. Also the 4.0.4 will be stabilisation candidate. |