bug-gnulib
[Top][All Lists]
Advanced

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

Re: new module 'iconv_open'


From: Eric Blake
Subject: Re: new module 'iconv_open'
Date: Mon, 2 Apr 2007 14:46:04 +0000 (UTC)
User-agent: Loom/3.14 (http://gmane.org/)

Bruno Haible <bruno <at> clisp.org> writes:

> 
> Some proprietary iconv() implementations (IRIX, OSF/1, Solaris) are
> actually usable if
>   1. one is willing to do an indirect conversion (through UTF-8) if a direct
>      conversion between two encodings doesn't exist,
>   2. one maps the standardized encoding names to the implementation specific
>      ones (not needed on Solaris).

This change breaks compiling the latest git coreutils on at least cygwin:

> 
> GPERF = gperf
> iconv_open-aix.h: iconv_open-aix.gperf
>       $(GPERF) -m 10 iconv_open-aix.gperf > address@hidden
>       mv address@hidden $@

make[1]: Entering directory `/home/eblake/coreutils/lib'
gperf -m 10 iconv_open-aix.gperf > iconv_open-aix.h-t
gperf: invalid option -- m
Usage: gperf [-cCdDef[num]
F<initializers>GhH<hashname>i<init>Ijk<keys>K<keyname>lL<language>nN<function 
name>ors<size>S<switches>tTvW<wordlistname>Z<class name>7] [input-file]
Try `gperf --help' for more information.
make[1]: *** [iconv_open-aix.h] Error 1

gperf --version
GNU gperf 2.7.2

gperf is not on the list of widely available programs, so it must not be 
invoked by a normal user when compiling a tarball, but requiring maintainers to 
have gperf available might be reasonable if you could rework this to use a 
command line that will work with commonly-installed gperf versions.  Which 
version did you test with?

-- 
Eric Blake







reply via email to

[Prev in Thread] Current Thread [Next in Thread]