Hi,
On Wed, Aug 19, 2009 at 03:12:11PM +0200, Frank Schoenheit, Sun Microsystems
Germany wrote:
> > So we either need something like this (when did they change that?):
> > [configure patch]
>
> I'd prefer that. Making the source code dependent on different boost
> versions (even if it's only about different headers to include) is too
> prone to breakage, IMO.
The problem is for people who might want to use system-boost for their
older distros not having boost >= 1.38. But it might be possible to use
internal boost for them...
BTW, if the problem only is that header (build still running), we probably
shouldn't do my original patch but just patch the AC_CHECK_HEADER directly...
Like this:
--- configure.in (Revision 275137)
+++ configure.in (Arbeitskopie)
@@ -3934,8 +3934,8 @@
AC_LANG_PUSH([C++])
AC_CHECK_HEADER(boost/shared_ptr.hpp, [],
[AC_MSG_ERROR(boost/shared_ptr.hpp not found. install boost)], [])
- AC_CHECK_HEADER(boost/spirit/core.hpp, [],
- [AC_MSG_ERROR(boost/spirit/core.hpp not found. install boost)], [])
+ AC_CHECK_HEADER(boost/spirit/include/classic_core.hpp, [],
+ [AC_MSG_ERROR(boost/spirit/include/classic_core.hpp not found. install
boost >= 1.38)], [])
AC_LANG_POP([C++])
else
AC_MSG_RESULT([internal])
Grüße/Regards,
Rene
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@openoffice.org
For additional commands, e-mail: dev-h...@openoffice.org
