Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 17 Dec 2021 12:01:07 -0800
From:      John Baldwin <jhb@FreeBSD.org>
To:        Kristof Provost <kp@FreeBSD.org>, src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   Re: git: dc04fa802dc1 - main - pf: make if_pflog.h self-contained
Message-ID:  <9aefa94c-e0e7-c5a5-2aa6-6b427581dae1@FreeBSD.org>
In-Reply-To: <202112171139.1BHBd2jw073888@gitrepo.freebsd.org>
References:  <202112171139.1BHBd2jw073888@gitrepo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On 12/17/21 3:39 AM, Kristof Provost wrote:
> The branch main has been updated by kp:
> 
> URL: https://cgit.FreeBSD.org/src/commit/?id=dc04fa802dc17ec663dffb3de99db2690b393ca1
> 
> commit dc04fa802dc17ec663dffb3de99db2690b393ca1
> Author:     Kristof Provost <kp@FreeBSD.org>
> AuthorDate: 2021-12-15 19:54:19 +0000
> Commit:     Kristof Provost <kp@FreeBSD.org>
> CommitDate: 2021-12-17 11:38:35 +0000
> 
>      pf: make if_pflog.h self-contained
>      
>      Reviewed by:    imp
>      Sponsored by:   Rubicon Communications, LLC ("Netgate")
>      Differential Revision:  https://reviews.freebsd.org/D33503
> ---
>   sys/net/if_pflog.h | 3 +++
>   1 file changed, 3 insertions(+)
> 
> diff --git a/sys/net/if_pflog.h b/sys/net/if_pflog.h
> index 93a69a2bb3a5..0406f78474a8 100644
> --- a/sys/net/if_pflog.h
> +++ b/sys/net/if_pflog.h
> @@ -31,7 +31,10 @@
>   #ifndef _NET_IF_PFLOG_H_
>   #define	_NET_IF_PFLOG_H_
>   
> +#include <sys/types.h>
> +
>   #include <net/bpf.h>
> +#include <net/if.h>
>   
>   #define	PFLOGIFS_MAX	16

FYI, as a counter point, net/libcap doesn't build on main (thus meaning that
qemu doesn't build on head) because it includes this header which by the nested
include of <net/bpf.h> adds duplicate definitions of 'struct bpf*':

https://lists.freebsd.org/archives/freebsd-pkg-fallout/2021-December/156851.html

(It only started at the start of December, not sure why though, all I know is
I can't build qemu in poudriere on my laptop anymore.)

-- 
John Baldwin



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?9aefa94c-e0e7-c5a5-2aa6-6b427581dae1>