[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 3/7] use gnulib to print --help and --version
From: |
Sami Kerola |
Subject: |
Re: [PATCH 3/7] use gnulib to print --help and --version |
Date: |
Tue, 31 Dec 2013 23:24:29 +0000 |
On 31 December 2013 19:57, Reuben Thomas <address@hidden> wrote:
> On 30 December 2013 11:58, Sami Kerola <address@hidden> wrote:
>> diff --git a/src/hello.c b/src/hello.c
>> #include "progname.h"
>> +#include "progname.h"
>
> You end up with two includes of progname.h here!
Good catch. Fixed in my repo.
>> +/* The official name of this program (e.g., no `g' prefix). */
>> +#define PROGRAM_NAME "hello"
>
> Since in our case it's the same as the package name, can't you use that
> here, and avoid duplicating the string "hello"?
I have feeling that might give wrong impression as an example.
>> +#define AUTHORS \
>> + proper_name ("The King"), \
>> + proper_name ("others")
>
> This looks bizarre. Is "The King" a known joke in hello? I can only find the
> reference in AUTHORS. If we can find a better original author, can we use
> that, and if not, maybe just leave AUTHORS blank (however best one does
> that)?
That was one of the more difficult considerations I had. The King
seems feels more correct than listing all people from AUTHORS file,
git history, etc. I did think writing Brian Kernighan & Dennish
Ritchie, but quite frankly they only invented The Hello algorithm and
have about nothing to do with the GNU Hello. Perhaps it would be more
appropriate to write
#define AUTHORS \
proper_name ("Original Author"),
proper_name ("Second Major Contributor"),
proper_name ("Firstname Lastname")
--
Sami Kerola
http://www.iki.fi/kerolasa/
[PATCH 5/7] demonstrate how to add information to a manual page, Sami Kerola, 2013/12/30
[PATCH 4/7] use standard error printing facilities, Sami Kerola, 2013/12/30
[PATCH 7/7] update NLS file sources, Sami Kerola, 2013/12/30
[PATCH 6/7] update gnulib, Sami Kerola, 2013/12/30