Re: Fix for 687162, CVS HEAD doesn't compile with Cygwin/gcc

Ralph,

I've tried the cygwin-buildfix.diff with Cygwin.
It still fails, but the error messages are now different than before :

In file included from ./src/genconf.c:19:
src/stdpre.h:307: parse error before `uint8_t'
src/stdpre.h:307: warning: data definition has no type or storage class
src/stdpre.h:308: parse error before `uint16_t'
src/stdpre.h:308: warning: data definition has no type or storage class
src/stdpre.h:309: parse error before `uint32_t'
src/stdpre.h:309: warning: data definition has no type or storage class
src/stdpre.h:310: parse error before `uint64_t'
src/stdpre.h:310: warning: data definition has no type or storage class
make[1]: *** [obj/../debugobj/genconf] Error 1
make: *** [debug] Error 2

The line 307 is :

   typedef unsigned int8_t         uint8_t;

I believe that it's not a valid C syntax :
'unsigned' can't apply to a defined type.

Igor.




----- Original Message ----- 
From: "Ralph Giles" <[email protected]>
To: "Alex Cherepanov" <[email protected]>
Cc: "gs-code-review" <[email protected]>
Sent: Sunday, November 30, 2003 10:36 PM
Subject: Re: [gs-code-review] Fix for 687162,CVS HEAD doesn't compile with
Cygwin/gcc


> On Mon, Nov 24, 2003 at 04:30:03PM -0500, Alex Cherepanov wrote:
>
> > Cygwin is not yet recognized as a target for jbig2dec inclusion. One can
> > easily add platform identification code to configure.ac but it will
> > bring in config.guess and some other files. Do we want this now ?
>
> No, but it does look like we need to move to generating a config.h from
and/or replacing genarch
> with the configure script. That's post 8.12, of course.
>
> > Re the code freeze, this patch fixes the build process, and won't
> > introduce subtle changes. It either breaks everything or significantly
> > improves portabilty. The latter seems to be more likely outcome.
>
> Can you try the attached, more minimal patch?
>
> Thanks,
>  -r
>


----------------------------------------------------------------------------
----


> _______________________________________________
> gs-code-review mailing list
> [email protected]
> http://www.ghostscript.com/mailman/listinfo/gs-code-review
>