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

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 <[email protected]> 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
>