GNUAspell / aspell

http://aspell.net
GNU Lesser General Public License v2.1
243 stars 53 forks source link

solaris 10 / 11 #371

Open aspell-helper opened 18 years ago

aspell-helper commented 18 years ago

Niki W. Waibel nikiwaibel\@sf created a bug report on 2006-06-12 03:19:39 UTC (Orig. from https://sourceforge.net/p/aspell/bugs/190)

hi, solaris >= 10 cares much more about the _XOPEN_SOURCE* definitions. compilation fails if you simply set _XOPEN_SOURCE_EXTENDED to 1 (some wide stuff is not "seen" anymore, because it assumes _XPG4_2, even if you set _XOPEN_SOURCE to 500)! (simple) solution:

diff -r -u aspell-0.60.4.orig/prog/check_funs.cpp aspell-0.60.4/prog/check_funs.cpp --- aspell-0.60.4.orig/prog/check_funs.cpp
2005-06-19 14:25:01.000000000 +0200 +++ aspell-0.60.4/prog/check_funs.cpp 2006-06-12 04:54:14.843145200 +0200 @@ -19,7 +19,7 @@ #include "settings.h"

#ifdef DEFINE_XOPEN_SOURCE_EXTENDED -# define _XOPEN_SOURCE_EXTENDED 1 +# define _XOPEN_SOURCE_EXTENDED #endif

#ifdef CURSES_NON_POSIX

another problem is gcc-4.1 (i guess this has been reported already). anyway, here's my fix:

--- aspell-0.60.4.orig/modules/filter/nroff.cpp 2005-02-20 22:49:18.000000000 +0100 +++ aspell-0.60.4/modules/filter/nroff.cpp
2006-06-11 00:45:30.000000000 +0200 @@ -73,7 +73,7 @@ return false; }

- bool NroffFilter::process_char (FilterChar::Chr c); + bool process_char (FilterChar::Chr c);

public:

===

using this 2 fixes i can compile successfully using gcc-4.1.1/binutils-2.17.50.0.2 on linux/x86, linux/x86_64, solaris8/9/10/sparc.

rds, niki

aspell-helper commented 18 years ago

Niki W. Waibel nikiwaibel\@sf commented on 2006-06-12 03:19:40 UTC

aspell-0.60.4.nww.patch

aspell-helper commented 18 years ago

Kevin Atkinson kevina\@sf updated the issue on 2006-11-13 06:33:56 UTC

aspell-helper commented 18 years ago

Kevin Atkinson kevina\@sf updated the issue on 2006-11-13 06:44:05 UTC

aspell-helper commented 18 years ago

Kevin Atkinson kevina\@sf commented on 2006-11-18 10:25:12 UTC

Logged In: YES user_id=6591 Originator: NO

The _XOPEN_SOURCE_EXTENDED is there for a reason. I am afraid that changing it may break some things.

Will things work okay for you if you avoid defining XOPEN_SOURCE_EXTENDED at all?

The gcc fix is already applied.

aspell-helper commented 18 years ago

Kevin Atkinson kevina\@sf updated the issue on 2006-11-18 10:25:12 UTC

aspell-helper commented 18 years ago

Kevin Atkinson kevina\@sf updated the issue on 2006-11-18 10:25:29 UTC

aspell-helper commented 14 years ago

Kevin Atkinson kevina\@sf updated the issue on 2010-03-04 01:40:06 UTC