Re: 4/8 Enable the <config.h>-requiring test; fix violations

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

 



"Richard W.M. Jones" <rjones@xxxxxxxxxx> wrote:
> Jim Meyering wrote:
>> Enable the <config.h>-requiring test; fix violations
>>
>> Use <config.h>, not "config.h", per autoconf documentation.
>> * Makefile.cfg (local-checks-to-skip) [sc_require_config_h]: Enable.
>> * .x-sc_require_config_h: New file, to list exempted files.
>> * Makefile.am (EXTRA_DIST): Add .x-sc_require_config_h.
>>
>> Signed-off-by: Jim Meyering <meyering@xxxxxxxxxx>
>> ---
>>  .x-sc_require_config_h   |    5 +++++
>>  ChangeLog                |    8 ++++----
>
> We actually patch the Changelog file.  Is that a good idea?
>
> ACK to the rest.

Good catch.
I'll add an exclusion (list ChangeLog in .x-sc_require_config_h)

--
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]