Re: [PATCH 2/2] Removed sc_prohibit_strcmp from the syntax-check

[Eric Blake]

On 05/21/2014 12:00 PM, Kieran Colford wrote:
> With the removal of STREQ and STRNEQ from gnulib, this syntax test is
> no longer useful and simply generates a nuisance error message that
> can't be fixed by gnulib.

STREQ and STRNEQ have never been provided by a generic .h file in
gnulib, to my knowledge, so this commit message is misleading.  However,
several .c files define them rather consistently, as well as
tests/macros.h (although that header is not installed for use outside of
tests), and it is easy to copy that definition into your code as well.

> ---
>  top/maint.mk |    8 --------
>  1 file changed, 8 deletions(-)

I'd rather not remove this rule - at least 2 projects I work on
(coreutils and libvirt) would be negatively impacted if the rule
disappears.  Instead, you can customize your cfg.mk to skip this rule
(by adding it to local-checks-to-skip).

For that matter, maybe we should add a new gnulib module that can be
used to provide these macros as part of config.h rather than forcing
downstream packages to rewrite it themselves.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

signature.asc
Description: OpenPGP digital signature