Re: [PATCH] #include "config.h" needed

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Rich.

Happy new year :)

"Richard W.M. Jones" <rjones@xxxxxxxxxx> wrote:
> There is one file missing:
>
>   #include "config.h"
>
> which ought to be included at the beginning of every C file.  If
> missing, it causes problems on Windows -- in particular the C99
> keyword 'restrict' won't be recognised and that causes compile errors
> all over the place.

Yes, indeed.
I like to use a "make distcheck"-time check for this.
Just grep all version-controlled .c files, with a list of exceptions.
Otherwise, it's too easy to forget.

> I've also changed a few <config.h> --> "config.h", which I think is
> more correct because config.h is a local file and should never come
> from the system include files (CC'd to Jim Meyering -- is this right
> in your opinion?)

I used to prefer "config.h", too, but after some discussion
many years ago, I switched, and now coreutils, gnulib, etc.
use <config.h> everywhere.

The "Configuration Header Files" section of the autoconf
manual explains the subtle preference for <config.h>.
All it takes is being burned once by a stray config.h
file built for another architecture or with different
options, and you'll never use "config.h" again :-)

--
Libvir-list mailing list
Libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]