[GRASS-dev] DBMI trouble compiling on Win32

[Glynn Clements]

Moritz Lennert wrote:

> > This fixed things in part. However, now I have a problem with the DBMI
> > lib:
> > 
> > cd /src/grass6/lib/db/dbmi_client
> > make
> > 
> > gives me:
> > 
> > gcc -I/src/grass6/dist.i686-pc-mingw32/include -I/include
> > -I/local/include -g -O2  -I/include -I/local/include
> > -DPACKAGE=\""grasslibs"\"   -I../dbmi_base    -DPACKAGE=\""grasslibs"\"
> >  -I/src/grass6/dist.i686-pc-mingw32/include \
> >         -o OBJ.i686-pc-mingw32/start.o -c start.c
> > start.c: In function `db_start_driver':
> > start.c:157: error: `have_stdin' undeclared (first use in this function)
> > start.c:157: error: (Each undeclared identifier is reported only once
> > start.c:157: error: for each function it appears in.)
> > start.c:157: error: `have_stdout' undeclared (first use in this function)
> > start.c:162: error: `stdin_fd' undeclared (first use in this function)
> > start.c:168: error: `stdin_orig' undeclared (first use in this function)
> > start.c:185: error: `stdout_fd' undeclared (first use in this function)
> > start.c:191: error: `stdout_orig' undeclared (first use in this function)
> > make[1]: *** [OBJ.i686-pc-mingw32/start.o] Error 1
> > make[1]: Leaving directory `/src/grass6/lib/db/dbmi_client'
> > 
> > 
> > Maybe something needs to be fixed in the configure script for Win32?
> 
> This seems to come from Brad's latest change to start.c:
> 
> 
> @@ -33,9 +38,6 @@
>       int stat;
>       dbConnection connection;
>       char ebuf[5];
> -    int stdin_orig, stdout_orig;
> -    int have_stdin, have_stdout;
> -    int stdin_fd, stdout_fd;
> 
> Brad, any special reason for this, or did you just not see that these 
> are used in the #ifdef __MINGW32__ which starts at line 117 ?

Almost certainly the latter. I've restored the variables,
conditionalised upon "#ifdef __MINGW32__" to prevent "unused variable"
warnings on other platforms (which is probably the reason that they
were removed).

-- 
Glynn Clements <glynn at gclements.plus.com>