On Mon, Oct 02, 2017 at 12:18:32PM -0400, tho...@habets.se wrote:
> Should be correct now[1]. It's almost exactly what your initial patch
> was, except skipping `AC_CACHE_CHECK` for `AC_CHECK_DECL`. Hopefully
> I'm not misunderstanding docs[2] again:
> "This macro caches its result[???]"
Your understanding is correct. What I don't understand here is why you
set the very same cache variable that is already set by AC_CHECK_DECL.
The shorter form
C_CHECK_DECL([CLOCK_MONOTONIC],[],[],[#include...])
should have the same effect.
> It does. It was merely on the point of trying to resolve the issue
> swiftly, mentioned earlier.
I think this is fairly swift (compared to many other bugs). :)
> From my point of view this will be in arping-2.20, and is so far the
> only change since 2.19 released in July.
Take your time. As long as this change doesn't slip buster (i.e.
uploaded by the end of 2018), I won't be annoyed.
> [1]
> https://github.com/ThomasHabets/arping/commit/52b8d05ae62301c698488445666ad4604d9fe2b6
Looks good to me. Thanks for not giving up. :)
Helmut
