Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 27 Nov 2020 12:08:19 +0000
From:      "Bjoern A. Zeeb" <bzeeb-lists@lists.zabbadoz.net>
To:        "Alan Somers" <asomers@freebsd.org>
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:  <1E846E83-998A-4ACC-9491-D0411899C1A7@lists.zabbadoz.net>
In-Reply-To: <CAOtMX2jqWbVgKHrCWgRfeJsEKkbNTZ4qJSBdSbKcNi%2BYc5pZEw@mail.gmail.com>
References:  <202011260429.0AQ4TV2B023801@repo.freebsd.org> <D6607BB7-4DFD-4C9A-9351-C9D36B3A65B2@lists.zabbadoz.net> <CAOtMX2jqWbVgKHrCWgRfeJsEKkbNTZ4qJSBdSbKcNi%2BYc5pZEw@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
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án Sučan <sucanjan@gmail.com>
>>>   Sponsored by:       Google LLC (Google Summer of Code 2019)
>>>   MFC after:  Never
>>>   Differential Revision:      https://reviews.freebsd.org/D21377
>>
>> I don’t have IPv4 anymore.
>> I don’t 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’s a “dual-stack handling” binary and those we’ve 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’t know how hard it is to do this now.  I’ll be happy to have a 
look and help if it’s 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





Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1E846E83-998A-4ACC-9491-D0411899C1A7>