Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 27 Nov 2020 07:32:28 -0700
From:      Alan Somers <asomers@freebsd.org>
To:        "Bjoern A. Zeeb" <bzeeb-lists@lists.zabbadoz.net>
Cc:        src-committers <src-committers@freebsd.org>, svn-src-all <svn-src-all@freebsd.org>,  svn-src-head <svn-src-head@freebsd.org>
Subject:   Re: svn commit: r368045 - in head: . etc/mtree rescue/rescue sbin sbin/ping sbin/ping/tests sbin/ping6 sbin/ping6/tests tools/build/mk
Message-ID:  <CAOtMX2jh98zYQeCDhOa5yJ0mvHJ5hy4sX3wAkGG_BDYcJwx1OA@mail.gmail.com>
In-Reply-To: <1E846E83-998A-4ACC-9491-D0411899C1A7@lists.zabbadoz.net>
References:  <202011260429.0AQ4TV2B023801@repo.freebsd.org> <D6607BB7-4DFD-4C9A-9351-C9D36B3A65B2@lists.zabbadoz.net> <CAOtMX2jqWbVgKHrCWgRfeJsEKkbNTZ4qJSBdSbKcNi%2BYc5pZEw@mail.gmail.com> <1E846E83-998A-4ACC-9491-D0411899C1A7@lists.zabbadoz.net>

next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, Nov 27, 2020 at 5:08 AM Bjoern A. Zeeb <
bzeeb-lists@lists.zabbadoz.net> wrote:

> On 26 Nov 2020, at 18:51, Alan Somers wrote:
>
> > On Thu, Nov 26, 2020 at 2:16 AM Bjoern A. Zeeb <
> > bzeeb-lists@lists.zabbadoz.net> wrote:
> >
> >> On 26 Nov 2020, at 4:29, Alan Somers wrote:
> >>
> >>> Author: asomers
> >>> Date: Thu Nov 26 04:29:30 2020
> >>> New Revision: 368045
> >>> URL: https://svnweb.freebsd.org/changeset/base/368045
> >>>
> >>> Log:
> >>>   Merge ping6 to ping
> >>>
> >>>   There is now a single ping binary, which chooses to use ICMP or
> >>> ICMPv4
> >>>   based on the -4 and -6 options, and the format of the address.
> >>>
> >>>   Submitted by:       J=C3=A1n Su=C4=8Dan <sucanjan@gmail.com>
> >>>   Sponsored by:       Google LLC (Google Summer of Code 2019)
> >>>   MFC after:  Never
> >>>   Differential Revision:      https://reviews.freebsd.org/D21377
> >>
> >> I don=E2=80=99t have IPv4 anymore.
> >> I don=E2=80=99t see any WITHOUT_INET or -DINET checks.
> >> How can I compile INET out now?
> >>
> >
> > I don't see any such checks before, either.  Was it ever possible to
> > exclude ping by building WITHOUT_INET?
>
> No, for various reasons including startup scripts relying on it etc.
> I think no one ever bothered to fully find it all.
>
> But it was possible to just ditch the binary (not installing it into
> custom images, or rm -f it post-install along with other things).
>
>
> Now it=E2=80=99s a =E2=80=9Cdual-stack handling=E2=80=9D binary and those=
 we=E2=80=99ve tried
> with a lot of care to make sure they grow compiling out both ways as you
> cannot throw away the binary anymore.
>
> I don=E2=80=99t know how hard it is to do this now.  I=E2=80=99ll be happ=
y to have a
> look and help if it=E2=80=99s not a 5 minute job for you knowing the code=
 and
> split up.   Would we just have to handle main.c with #ifdefs and the old
> ping[4] files from the Makefile or is ping6 also using shared code from
> the former ping[4]?
>
>
> /bz
>

Yes, that's about right.  ping.c is the old ping.c, and ping6.c is the old
ping6.c, with main() removed from both.  So it should be pretty easy to
compile out INET.
-Alan



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAOtMX2jh98zYQeCDhOa5yJ0mvHJ5hy4sX3wAkGG_BDYcJwx1OA>