Re: git: daf481f3d508 - main - rescue: include ping iff at least one of INET & INET6 is enabled

On Sun, Jan 2, 2022 at 10:17 AM Gleb Smirnoff <glebius@freebsd.org> wrote:
>
> On Fri, Dec 31, 2021 at 06:18:53PM +0000, Ed Maste wrote:
> E> The branch main has been updated by emaste:
> E>
> E> URL: https://cgit.FreeBSD.org/src/commit/?id=daf481f3d508e7497b456e8df7db17c90b623ee0
> E>
> E> commit daf481f3d508e7497b456e8df7db17c90b623ee0
> E> Author:     Ed Maste <emaste@FreeBSD.org>
> E> AuthorDate: 2021-12-31 01:11:31 +0000
> E> Commit:     Ed Maste <emaste@FreeBSD.org>
> E> CommitDate: 2021-12-31 18:16:30 +0000
> E>
> E>     rescue: include ping iff at least one of INET & INET6 is enabled
> E>
> E>     Ping does not build (and serves no purpose) if neither is enabled
> E>     (i.e., building WITHOUT_INET and WITHOUT_INET6).
> E>
> E>     This is equvalent to a4ef9e58bc0c but for rescue/.
> E>
> E>     PR:             260082
> E>     Fixes:          a4ef9e58bc0c ("sbin: build ping if at least one of...")
> E>     Sponsored by:   The FreeBSD Foundation
>
> Shouldn't be ping built only with INET and ping6 with INET6?
> Is there any use of ping on machine that doesn't have IPv4?
>
> --
> Gleb Smirnoff

In 14.0-CURRENT, ping and ping6 are the same binary.  However, parts
of that binary are conditionally compiled by INET and INET6.