Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 24 Mar 2011 08:22:32 +0200
From:      Andriy Gapon <avg@freebsd.org>
To:        Baptiste Daroussin <bapt@freebsd.org>
Cc:        freebsd-ports@freebsd.org
Subject:   libreoffice and gcc44+
Message-ID:  <4D8AE328.3020906@freebsd.org>

next in thread | raw e-mail | index | archive | help
This is a multi-part message in MIME format.
--------------080605080601010307010902
Content-Type: text/plain; charset=X-VIET-VPS
Content-Transfer-Encoding: 7bit


Baptiste,

I found it necessary to use the attached patch for libreoffice compilation to
succeed with gcc45. (patch name doesn't conform to the ports standards).
Without the patch my compilation failed because FILE declaration was not found.

I am CC-ing ports@ as this is an example of a classical base gcc vs gcc44+
problem.  The reasons for this issue are described here:
http://gcc.gnu.org/gcc-4.3/porting_to.html
See "Header dependency cleanup".

I am surprised that this kind of problem has sneaked into the release code of
libreoffice.  It could mean two things: (a) they use older gcc themselves; (b)
FILE/stdio.h inclusion was brought in by some other inclusion likely in Linux
system header files.
In any case I think that the upstream developers should have used cstdio header
inclusion and std::FILE if they really need that.
Are you a regular on their development channels?  Maybe you could forward the
report to them?

Thanks!
-- 
Andriy Gapon

--------------080605080601010307010902
Content-Type: text/plain;
 name="libreoffice-gcc45-includes.patch"
Content-Transfer-Encoding: 7bit
Content-Disposition: attachment;
 filename="libreoffice-gcc45-includes.patch"

--- build/libreoffice/filter/inc/filter/msfilter/mstoolbar.hxx.orig	2011-03-24 07:57:23.427842820 +0200
+++ build/libreoffice/filter/inc/filter/msfilter/mstoolbar.hxx	2011-03-24 07:59:12.858835904 +0200
@@ -4,6 +4,7 @@
 #include "filter/msfilter/msfilterdllapi.h"
 #include <tools/string.hxx>
 #include <sot/storage.hxx>
+#include <stdio.h>
 #include <ostream>
 #include <memory>
 #include <vector>

--------------080605080601010307010902--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4D8AE328.3020906>