Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 4 May 2024 20:42:24 +0100
From:      Jessica Clarke <jrtc27@freebsd.org>
To:        Lexi Winter <lexi@le-fay.org>
Cc:        arch@freebsd.org, net@freebsd.org
Subject:   Re: review request: changing the default ifconfig(8) address format to CIDR
Message-ID:  <2A85E653-1228-4635-83A4-BED72AEE9420@freebsd.org>
In-Reply-To: <ZjZVaxzdMehqqci_@ilythia.eden.le-fay.org>
References:  <ZjZVaxzdMehqqci_@ilythia.eden.le-fay.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On 4 May 2024, at 16:34, Lexi Winter <lexi@le-fay.org> wrote:

> hi,
>=20
> i've just submitted this PR:
>=20
> https://github.com/freebsd/freebsd-src/pull/1216
>=20
> which contains this commit:
>=20
> commit 57d273c90ee1c17446236aba25ed0bd291c4f126 (HEAD -> lf/main, =
hemlock/lf/main)
> Author: Lexi Winter <lexi@le-Fay.ORG>
> Date:   Sat May 4 16:11:21 2024 +0100
>=20
>    ifconfig(8): change default IP address format to 'cidr'
>=20
>    'netmasks' haven't been used in IP networking for decades.  Change =
the
>    default address format for both IPv4 and IPv6 addreses in =
ifconfig(8) to
>    'cidr', which prints addreses in the format most users will be more
>    familiar with.
>=20
>    The previous format is still available using -finet:hex or =
-finet6:numeric.
>=20
> imp@ suggested i should ask arch@ and net@ about this, so here i am!
>=20
> i understand there might be some backward-compat concerns with =
scripting
> here, but it's well past time this change was made, and anyone who
> really can't update their scripts can use ifconfig -f or
> $IFCONFIG_FORMAT to retain the old behaviour.

Do we need to care about supporting (/ do we currently support)
historical non-contiguous netmasks? At a glance the CIDR code doesn=E2=80=99=
t
handle that and will stop at the first 0, so changing to that by
default would break such setups.

Jess




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?2A85E653-1228-4635-83A4-BED72AEE9420>