bug-gnulib
[Top][All Lists]
Advanced

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

Re: [Bug-gnulib] feature request: argp()


From: Bruno Haible
Subject: Re: [Bug-gnulib] feature request: argp()
Date: Tue, 10 Jun 2003 13:37:12 +0200
User-agent: KMail/1.5

Hi Simon,

> * Commit the following patch.  It consists of verbatim (from CVS)
>   libc/argp/argp* (except argp-test.c) and new gnulib/m4/argp.m4,
>   gnulib/modules/argp.  This builds on my Debian Linux box.  (My
>   savannah account is "jas" if you want me to commit it.)

OK, I've committed the lib/* files for you, and also the module
description. I've tweaked the title of the description because there
is no argp() function.

With m4/argp.m4 I see two problems:

  - The module doesn't define a function argp(), therefore _FUNC_ARGP is
    inappropriate. Can you call the macro AC_ARGP or gl_ARGP ?

  - The code is compiled and used even on glibc systems, which
    unnecessarily increases executable size. Can you arrange to
    not compile the files on a glibc system?

> * Fix the portability problems in the code using followup-patches.
>   This way it is easy to see what modifications went into gnulib,
>   compared to glibc.  Some problems I'm aware of are as follows:
>
>   * argp-eexst.c includes sysexits.h for EX_USAGE.  Either we
>     workaround it and simply hard code 64 (ugly, but perhaps there is
>     some standard that says EX_USAGE really must be 64 anyway) or we
>     add a sysexits.h to gnulib.
>
>   * program_invocation_{short_,}name.  Either we workaround it similar
>     to program_name in fatal.c, or we provide a new gnulib module
>     "progname" which defines these two variables.
>
>   * Replace *_unlocked with * and depend on unlocked-io.

Sounds like a good plan.

> * There is documentation and examples in glibc, but no place for that
>   within gnulib that I could find.

Yes, this is a problem...

Bruno





reply via email to

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