Re: new modules for looking up the ancestors of a process

[Bernhard Voelker]

On 8/14/19 2:05 AM, Bruno Haible wrote:
> diff --git a/modules/get_progname_of b/modules/get_progname_of
> new file mode 100644
> index 0000000..012cc59
> --- /dev/null
> +++ b/modules/get_progname_of
> @@ -0,0 +1,24 @@
> +Description:
> +Determine the program name of a given process.
> +
> +Files:
> +lib/get_progname_of.h
> +lib/get_progname_of.c
> +
> +Depends-on:
> +extensions
> +unistd
> +
> +configure.ac:
> +
> +Makefile.am:
> +lib_SOURCES += get_progname_of.h get_progname_of.c
> +
> +Include:
> +"get_progname_of.h"
> +
> +License:
> +LGPL
> +
> +Maintainer:
> +all

Nice work, thanks.

Nits:
I'd document that the caller has to free the returned memory later;
so at least here in the module description, but maybe also in the
header and - as it serves as example code - in the test.

Have a nice day,
Berny