Re: new module 'iconv_open'

[John Darrington]

On Tue, Apr 03, 2007 at 12:43:19AM +0200, Bruno Haible wrote:
     Simon Josefsson wrote:
     > Well, at least after I installed
     > gperf, before that I got this error during build:
     > 
     > make[2]: Entering directory `/home/jas/src/libidn/lib/gl'
     > gperf -m 10 iconv_open-aix.gperf > iconv_open-aix.h-t
     > /bin/sh: gperf: command not found
     > make[2]: *** [iconv_open-aix.h] Error 127

This was an unexpected surprise for me too.
     
     Yes, bison, flex, and gperf are the three common tools for code generation.
     
     > No warnings about missing gperf during ./configure.  I think the M4
     > code should test for a gperf tool, using AM_MISSING_PROG or
     > similar.

I agree.
     
     It could, in theory. But what should the Makefile rule do if 'gperf' was 
not
     found?
       - If the iconv_open-aix.h file does not yet exist, it simply cannot 
proceed.
         (Except on AIX... - Should I make the dependencies platform dependent?)
       - If the iconv_open-aix.h file exists but is older, what to do? Emit a
         warning and compile the old .h file?
     
I don't fully understand all the issues here, but at the very least,
./configure could show a warning if gperf is not found or is too old.

J'
     
-- 
PGP Public key ID: 1024D/2DE827B3 
fingerprint = 8797 A26D 0854 2EAB 0285  A290 8A67 719C 2DE8 27B3
See http://pgp.mit.edu or any PGP keyserver for public key.

signature.asc
Description: Digital signature