[Freeciv-Dev] Re: compiling on Solaris / the continuing story of GNU gettext (was: cvs snapshots)

[Raimar Falke <hawk@xxxxxxxxxxxxxxxxxxxxxxx>]

On Tue, Mar 26, 2002 at 09:44:56PM +0100, Reinier Post wrote:
> [Cc: to freeciv-dev]
> 
> On Tue, Mar 26, 2002 at 02:12:38PM -0600, Mike Kaufman wrote:
> > ok, I see that the snapshots are indeed being created correctly and they're
> > getting up on http://www.freeciv.org/download.phtml correctly, but it seems
> > that http://www.freeciv.org/latest.html is _not_ getting updated, nor are
> > the cvs diffs getting created...
> 
> That's right.  This is because www.freeciv.org is Solaris.
> Here is what I just did to compile the current CVS tree on Solaris.
> 
> > -mike
> 
> 1) patched ./configure to use putenv() instead of setenv();
>    a patch to fix this is attached:
> 
>      autoinclude.m4.diff-u

Can you test the alternate patch I attached?

> 2) set env vars to allow ./configure to find GNU gettext:
> 
>   env MSGFMT=$HANDCOMP/gettext-0.11.1/bin/msgfmt 
> "CPPFLAGS=-I$HANDCOMP/gettext-0.11.1/include" 
> "LDFLAGS=-L$HANDCOMP/gettext-0.11.1/lib -R$HANDCOMP/gettext-0.11.1/lib" 
> $HANDCOMP/bin/cfg --with-xpm-prefix=$HANDCOMP/xpm-3.4k
> 
> Note that it would be convenient if I could just use
> 
>   --with-gettext-utils=$HANDCOMP/gettext-0.11.1
> 
> here.  Similarly, it would be nice if
> 
>   --with-zlib=$HANDCOMP/zlib=1.1.4
> 
> would work.
> 
> But it compiles, with Sun make and Forte cc, which
> gave the following warnings on this 64 bit system:
> 

> "aiunit.c", line 2203: warning: statement not reached

Can't find this.

> "autoattack.c", line 99: warning: enum type mismatch: arg #2

Adding an extra cast?

> "report.c", line 273: warning: improper pointer/integer combination: arg #3
> "report.c", line 422: warning: improper pointer/integer combination: arg #2
> "report.c", line 427: warning: improper pointer/integer combination: arg #2
> "report.c", line 1020: warning: improper pointer/integer combination: arg #3

Seems like there are problems with the PL_ macro. You have used a
plain GNU gettext-0.11.1?! I should than be able to download and test
this for myself.

> "./../include/patrol_cursor.xbm", line 6: warning: initializer does not fit 
> or is out of range: 248
> [and a whole bunch more on the same file]

I don't see a problem here.

> "plrdlg.c", line 225: warning: improper pointer/integer combination: arg #3
> "repodlgs.c", line 505: warning: improper pointer/integer combination: arg #3
> "cma_fec.c", line 279: warning: improper pointer/integer combination: arg #3
> "cma_fec.c", line 323: warning: improper pointer/integer combination: arg #3
> "citydlg_common.c", line 133: warning: improper pointer/integer combination: 
> arg #3
> "citydlg_common.c", line 183: warning: improper pointer/integer combination: 
> arg #3
> "climisc.c", line 1099: warning: improper pointer/integer combination: arg #3

PL_

> "clinet.c", line 387: warning: statement not reached

A real error. Patch attached.

> "helpdata.c", line 595: warning: improper pointer/integer combination: arg #2
> "helpdata.c", line 600: warning: improper pointer/integer combination: arg #2
> "helpdata.c", line 605: warning: improper pointer/integer combination: arg #2

PL_

> GNU msgfmt gave warnings such as:
> 
> [path to]/msgfmt: da.po: warning: Charset "ISO-8859-1" is not supported. 
> msgfmt relies on iconv(),
>                                                            and iconv() does 
> not support "ISO-8859-1".
>                                                            Installing GNU 
> libiconv and then reinstalling GNU gettext
>                                                            would fix this 
> problem.
>                                                            Continuing anyway.
> 2299 translated messages, 285 untranslated messages.
> 
> 
> But I did get a civclient and civserver, and setting $LANG works.
> 
> In the past I haven't had any luck with setting $LANG to values such
> as 'de' or 'nl', while 'de_DE' and 'nl_NL' did work.  A patch to README:
> 
>   README.diff-u
> 
> But I have to say, the values 'nl' and 'de' do work when I try them now.

> --- README.orig       Sat May 19 16:59:02 2001
> +++ README    Tue Mar 26 21:21:54 2002
> @@ -365,7 +365,7 @@
>  Freeciv supports several local languages.
>  
>  You may choose which local language to use by specifying a "locale".
> -Each locale has a standard name (e.g., 'de' for German).  If you have
> +Each locale has a standard name (e.g., 'de_de' for German).  If you have
                                              ^^ Shouldn't this be DE?

>  installed Freeciv, you may choose a locale by setting the environment
>  variable LANG to that locale's standard name before running civserver
>  and civclient.
> @@ -373,9 +373,9 @@
>  For example, assuming you wish to use the German localization, you
>  would do:
>  
> -  export LANG; LANG=de   (in the Bourne shell (sh)),
> +  export LANG; LANG=de_DE   (in the Bourne shell (sh)),
>  or
> -  setenv LANG de         (in the C shell (csh)).
> +  setenv LANG de_DE         (in the C shell (csh)).
>  
>  (You could do this in your .profile or .login file.)

        Raimar

-- 
 email: rf13@xxxxxxxxxxxxxxxxx
 "The BeOS takes the best features from the major operating systems. 
  It's got the power and flexibility of Unix, the interface and ease 
  of use of the MacOS, and Minesweeper from Windows."

setenv1.diff
Description: Text document

unqueue_mapview_update1.diff
Description: Text document