Re: Fix for 687162, CVS HEAD doesn't compile with Cygwin/gcc
Ralph Giles <[email protected]>
| Newsgroups | gmane.comp.printing.ghostscript.patches |
|---|---|
| Message-ID | <[email protected]> |
On Mon, Nov 24, 2003 at 04:30:03PM -0500, Alex Cherepanov wrote: > Improve C99 type support: recignize systems that have inttypes.h and > deal with Cygwin specially. Do we know the header called 'inttypes.h' provides the stdint fixed-size typedefs on all systems? > 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 ? I think it would be better at this point to give in and add support for an autoconf-generated header that can supplement or replace what genarch does for that build. > 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. I spoke to raph briefly about this on IRC; apparently there was some discussion during the support call and he said he would commit a minimal patch based on this. In general I'm in favor of comitting something like this for 8.12 and then working on a better fix. Also, re Igor's comment, I'm told #elif has always been in ANSI C (and fairly well supported) so it should be safe given we've dropped K&R support. That should be a separate decision (and clean up patch) however. -r