From nobody Sun Jun 11 10:12:44 2023 X-Original-To: dev-commits-ports-main@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4Qf9cH5HSlz4ZJSx; Sun, 11 Jun 2023 10:12:47 +0000 (UTC) (envelope-from grembo@freebsd.org) Received: from mail.evolve.de (mail.evolve.de [213.239.217.29]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA512 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mail.evolve.de", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4Qf9cH1KyTz48Wn; Sun, 11 Jun 2023 10:12:47 +0000 (UTC) (envelope-from grembo@freebsd.org) Authentication-Results: mx1.freebsd.org; none Received: by mail.evolve.de (OpenSMTPD) with ESMTP id fc47b37f; Sun, 11 Jun 2023 10:12:45 +0000 (UTC) Received: by mail.evolve.de (OpenSMTPD) with ESMTPSA id d6f81f5f (TLSv1.3:TLS_AES_256_GCM_SHA384:256:NO); Sun, 11 Jun 2023 10:12:45 +0000 (UTC) Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable List-Id: Commits to the main branch of the FreeBSD ports repository List-Archive: https://lists.freebsd.org/archives/dev-commits-ports-main List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-ports-main@freebsd.org X-BeenThere: dev-commits-ports-main@freebsd.org Mime-Version: 1.0 (1.0) Subject: Re: git: 637c0bace261 - main - sysutils/pftop: Fix build on 14.0-CURRENT From: Michael Gmelin In-Reply-To: <87wn0afhdx.wl-herbert@gojira.at> Cc: ports-committers@freebsd.org, dev-commits-ports-all@freebsd.org, dev-commits-ports-main@freebsd.org Date: Sun, 11 Jun 2023 12:12:44 +0200 Message-Id: <25969E38-1EA2-4F9A-8644-AF378AC2E857@freebsd.org> References: <87wn0afhdx.wl-herbert@gojira.at> To: "Herbert J. Skuhra" X-Mailer: iPhone Mail (20D67) X-Rspamd-Queue-Id: 4Qf9cH1KyTz48Wn X-Spamd-Bar: ---- X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[]; ASN(0.00)[asn:24940, ipnet:213.239.192.0/18, country:DE] X-Rspamd-Pre-Result: action=no action; module=replies; Message is reply to one we originated X-ThisMailContainsUnwantedMimeParts: N > On 11. Jun 2023, at 12:00, Herbert J. Skuhra wrote: > =EF=BB=BFOn Sun, 11 Jun 2023 11:25:45 +0200, Michael Gmelin wrote: >>=20 >>=20 >>=20 >>> On 11. Jun 2023, at 10:27, Herbert J. Skuhra wrote: >>> On Sat, 10 Jun 2023 12:06:09 +0200, >>> Michael Gmelin wrote: >>>> The branch main has been updated by grembo: >>>> URL: https://cgit.FreeBSD.org/ports/commit/?id=3D637c0bace26138529a3623= 2e948549ad59342ba9 >>>> commit 637c0bace26138529a36232e948549ad59342ba9 >>>> Author: Michael Gmelin >>>> AuthorDate: 2023-06-10 10:03:39 +0000 >>>> Commit: Michael Gmelin >>>> CommitDate: 2023-06-10 10:03:39 +0000 >>>> sysutils/pftop: Fix build on 14.0-CURRENT >>>> --- >>>> sysutils/pftop/Makefile | 10 ++++++++-- >>>> sysutils/pftop/files/extra-patch-config.h | 6 +++++- >>>> 2 files changed, 13 insertions(+), 3 deletions(-) >>>> diff --git a/sysutils/pftop/Makefile b/sysutils/pftop/Makefile >>>> index f3c6d879f637..cba2ecd65aeb 100644 >>>> --- a/sysutils/pftop/Makefile >>>> +++ b/sysutils/pftop/Makefile >>>> @@ -1,6 +1,6 @@ >>>> PORTNAME=3D pftop >>>> PORTVERSION=3D 0.8 >>>> -PORTREVISION=3D 2 >>>> +PORTREVISION=3D 3 >>>> CATEGORIES=3D sysutils net >>>> MAINTAINER=3D grembo@FreeBSD.org >>>> @@ -22,7 +22,13 @@ EXTRA_PATCHES+=3D ${FILESDIR}/extra-patch-bpf_dum= p.c \ >>>> ${FILESDIR}/extra-patch-sf-gencode.h >>>> MAKE_ARGS=3D LOCALBASE=3D"${PREFIX}" \ >>>> - OSLEVEL=3D45 >>>> + >>>> +.if ${OPSYS} =3D=3D FreeBSD && ${OSVERSION} >=3D 1400090 >>>> +MAKE_ARGS+=3D OSLEVEL=3D46 >>>> +.else >>>> +MAKE_ARGS+=3D OSLEVEL=3D45 >>>> +.endif >>>> + >>>> CFLAGS+=3D -DHAVE_SNPRINTF=3D1 -DHAVE_VSNPRINTF=3D1 \ >>>> -DHAVE_FINE_GRAINED_LOCKING=3D1 >>>> diff --git a/sysutils/pftop/files/extra-patch-config.h b/sysutils/pftop= /files/extra-patch-config.h >>>> index 6d2873c42ab1..d24f88179718 100644 >>>> --- a/sysutils/pftop/files/extra-patch-config.h >>>> +++ b/sysutils/pftop/files/extra-patch-config.h >>>> @@ -1,7 +1,7 @@ >>>> $OpenBSD: patch-config_h,v 1.4 2008/12/20 04:36:11 canacar Exp $ >>>> --- config.h.orig Tue Nov 6 22:34:18 2007 >>>> +++ config.h Fri Dec 19 20:28:01 2008 >>>> -@@ -74,11 +74,20 @@ >>>> +@@ -74,11 +74,24 @@ >>>> #define HAVE_PFSYNC_STATE >>>> #endif >>>> @@ -11,7 +11,11 @@ $OpenBSD: patch-config_h,v 1.4 2008/12/20 04:36:11 c= anacar Exp $ >>>> +#endif >>>> + >>>> #ifdef HAVE_PFSYNC_STATE >>>> ++#if OS_LEVEL > 45 >>>> ++typedef struct pfsync_state_1400 pf_state_t; >>> Are you sure that this is correct? >>> If I replace pfsync_state_1400 with pfsync_state_1301 the port builds >>> and the output looks sane. >>=20 >> Hi, thanks for reporting, could you please add some details (like, >> how the output differs)? >=20 > With your change: >=20 > sctp Out (null)[13715] (null)[0] 0:255 = 0 * * * > 237 In (null)[0] (null)[0] NO_TRAFFIC:NO_TRAFF= IC 0 0 * * > ip In (null)[0] (null)[0] NO_TRAFFIC:NO_TRAFF= IC 0 9324h * * > ip In (null)[7185] (null)[512] NO_TRAFFIC:NO_TRAFF= IC 0 53 * 2048G > ip In (null)[4097] (null)[49408] NO_TRAFFIC:NO_TRAFF= IC 0 0 0 1 > ip In (null)[9732] (null)[4992] 9:0 = 0 0 42 73728M > cpnx In (null)[0] (null)[0] 255:0 = 0 * * * > ipenca In (null)[0] (null)[0] NO_TRAFFIC:NO_TRAFF= IC 0 0 * * > ip In (null)[0] (null)[0] 0:9 = * * * * >=20 > Garbage? >=20 > With attached patch I see ipv[46] addresses and port numbers again. :-) >=20 > I am running main-n263493-4e8d558c9d1c. >=20 The question is if pftop shouldn=E2=80=99t use pfsync_state_1400, or if ther= e is a kernel problem with that struct. I=E2=80=99ll look into it. Thanks