Subject: Re: wrt: various platforms
To: Blair Sadewitz <blair.sadewitz@gmail.com>
From: Julio M. Merino Vidal <jmmv84@gmail.com>
List: tech-pkg
Date: 09/16/2007 23:32:24
On Sep 16, 2007, at 11:26 PM, Blair Sadewitz wrote:

> Now and then, I come across a package which uses configure to test for
> the presence/usability of a certain header file, yet mysteriously does
> not use the definition in config.h in the source itself.
>
> For instance, X.org's pixman library tests for inttypes.h, but in
> pixman.h, they only include it if __OpenBSD__ is defined.
>
> Are there cases where this is the right thing to do?  Not using
> HAVE_INTTYPES_H when configure tests for it seems so peculiar to me I
> can't help but suspect I'm missing something.

They have probably overlooked something, and I think you should  
report this as a bug.  Checking for __*__ macros directly from the  
code is generally bad, as you cannot predict how systems will be  
modified in the future (whereas well-coded configure tests can do a  
good job at this) and it is extremely hard to track all their  
previous versions and variants straight from preprocessor conditionals.

-- 
Julio M. Merino Vidal <jmmv84@gmail.com>