[prev in list] [next in list] [prev in thread] [next in thread]
List: autoconf
Subject: Re: Will autoconf work with -Werror=implicit-int and -Werror=implicit-function-declaration ?
From: Frederic Berat <fberat () redhat ! com>
Date: 2023-12-13 16:24:31
Message-ID: CAObJKZp=k+4ea4p1R7_MuUH6+Y6OuWteDr-qojizFiCaGVPhpg () mail ! gmail ! com
[Download RAW message or body]
On Wed, Dec 13, 2023 at 3:30 PM Florian Weimer <fweimer@redhat.com> wrote:
> * Zack Weinberg:
>
> > Paul Eggert made some changes back in May that attempt to address this:
> > commits 028526149ee804617a302ccef22cc6adbda681b0 and
> > 33c26d2700f927432c756ccf7a4fc89403d35b95. Do you have a minimized
> > test case for the problem (both the original problem and any remaining
> > issues you're aware of are useful to me)?
>
> The latest installment of this issue is here:
>
> configure.ac: Define _DEFAULT_SOURCE along with _XOPEN_SOURCE
> <https://github.com/samtools/htslib/pull/1711>
>
> I wasn't aware of this change from
> 33c26d2700f927432c756ccf7a4fc89403d35b95 ("Fix port of AC_FUNC_MMAP"),
> or maybe I just forgot about it:
>
> +#ifndef getpagesize
> +# ifdef _SC_PAGESIZE
> +# define getpagesize() sysconf (_SC_PAGESIZE)
> +# elif defined _SC_PAGE_SIZE
> +# define getpagesize() sysconf (_SC_PAGE_SIZE)
> +# elif HAVE_GETPAGESIZE
> +int getpagesize ();
> +# else
>
> This should indeed isolate this test from the outcome of the getpagesize
> check because it now prefers sysconf.
>
> I can't test this with htslib unfortunately because I'm not able to
> build autoconf from sources for some reason.
>
You can use the SRPM available here, that's plain autoconf 2.72d:
https://copr.fedorainfracloud.org/coprs/fberat/autoconf.2.72d-1/build/6710467/
> Thanks,
> Florian
>
>
>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic