Re: [PATCH] Fix compilation of librumpuser on Linux and Hurd

[Etienne Brateau <etienne.brateau%gmail.com@localhost>]

Yes, it is.

I just wanted to add the HAVE_SYS_EVCNT, it’s just that my autoconf version is newer than the one that generated the previous version of the configure script.

I regenerated the configure because it’s asked in the configure.ac file.

If you want to avoid the changes of configure, I can resubmit the patch without the regeneration, or you can update my patch with the configure regenerated with the same tool version than previously.

Etienne

Le jeu. 22 sept. 2022 à 21:14, Valery Ushakov <uwe%stderr.spb.ru@localhost> a écrit :

On Thu, Sep 22, 2022 at 20:16:36 +0200, Etienne Brateau wrote:

> On linux and Hurd, the header sys/evcnt does not exists. So this patch
> add conditional compilation to don?t include that header and the related
> code if it?s unsupported.
> ---
>  lib/librumpuser/configure            | 3525 ++++++++++++++------------
>  lib/librumpuser/configure.ac         |    2 +-
>  lib/librumpuser/rumpuser_config.h.in |   18 +-
>  lib/librumpuser/rumpuser_dl.c        |    4 +
>  lib/librumpuser/rumpuser_port.h      |    2 +-
>  5 files changed, 1878 insertions(+), 1673 deletions(-)

About 99% of this diff is from using a different version of autoconf.
This kind of changes must not be mixed in one commit.


> diff --git a/lib/librumpuser/rumpuser_config.h.in b/lib/librumpuser/rumpuser_config.h.in
> index b008c0621b42..7e5f1336ab6f 100644
> --- a/lib/librumpuser/rumpuser_config.h.in
> +++ b/lib/librumpuser/rumpuser_config.h.in
> @@ -48,9 +48,6 @@
>  /* Define to 1 if you have the `memalign' function. */
>  #undef HAVE_MEMALIGN
> 
> -/* Define to 1 if you have the <memory.h> header file. */
> -#undef HAVE_MEMORY_H
> -
[...]
> +/* Define to 1 if you have the <stdio.h> header file. */
> +#undef HAVE_STDIO_H
> +

Is this also from the new autoconf?


-uwe