Re: git: fda7daf06301 - main - pfctl: Stop sharing pf_ruleset.c with the kernel

2021-01-06 Thread Florian Smeets via dev-commits-src-all
On 06.01.21 18:38, Kristof Provost wrote: > I think clean_dep is intended for kernel use. This seems to work for me: > > diff --git a/tools/build/depend-cleanup.sh > b/tools/build/depend-cleanup.sh > index 8b249ccf55fa..0b3ff9bcc635 100755 > --- a/tools/build/depend-cleanup.sh > ++

Re: git: fda7daf06301 - main - pfctl: Stop sharing pf_ruleset.c with the kernel

2021-01-06 Thread Kristof Provost
On 6 Jan 2021, at 17:58, Kristof Provost wrote: On 6 Jan 2021, at 16:45, Kyle Evans wrote: On Wed, Jan 6, 2021 at 8:30 AM Cy Schubert wrote: In message om> , Kyle Evans writes: On Wed, Jan 6, 2021, 03:22 Kristof Provost wrote: On 6 Jan 2021, at 10:20, Hartmann, O. wrote: amd64. kernel c

Re: git: fda7daf06301 - main - pfctl: Stop sharing pf_ruleset.c with the kernel

2021-01-06 Thread Kristof Provost
On 6 Jan 2021, at 16:45, Kyle Evans wrote: On Wed, Jan 6, 2021 at 8:30 AM Cy Schubert wrote: In message om> , Kyle Evans writes: On Wed, Jan 6, 2021, 03:22 Kristof Provost wrote: On 6 Jan 2021, at 10:20, Hartmann, O. wrote: amd64. kernel config is custom, no debug, IPFW, MAC. pf is not u

Re: git: fda7daf06301 - main - pfctl: Stop sharing pf_ruleset.c with the kernel

2021-01-06 Thread Hartmann, O.
On Wed, 6 Jan 2021 06:58:40 -0600 Kyle Evans wrote: > I note that the original report says buildkernel, but the log exercpt is > from buildworld. Looks like pf_ruleset.c kept the name, so this needs a > dependency cleanup rule in tools/build to build the right one on > WITHOUT_CLEAN builds. > >

Re: git: fda7daf06301 - main - pfctl: Stop sharing pf_ruleset.c with the kernel

2021-01-06 Thread Cy Schubert
In message , Kyle Evans writes: > On Wed, Jan 6, 2021 at 8:30 AM Cy Schubert wrote: > > In message c > > om> > > , Kyle Evans writes: > > > On Wed, Jan 6, 2021, 03:22 Kristof Provost wrote: > > > > > > > On 6 Jan 2021, at 10:20, Hartmann, O. wrote: > > > > > > > > > > amd64. > > > > > kernel co

Re: git: fda7daf06301 - main - pfctl: Stop sharing pf_ruleset.c with the kernel

2021-01-06 Thread Kyle Evans
On Wed, Jan 6, 2021 at 8:30 AM Cy Schubert wrote: > In message om> > , Kyle Evans writes: > > On Wed, Jan 6, 2021, 03:22 Kristof Provost wrote: > > > > > On 6 Jan 2021, at 10:20, Hartmann, O. wrote: > > > > > > > > amd64. > > > > kernel config is custom, no debug, IPFW, MAC. pf is not used, we u

Re: git: fda7daf06301 - main - pfctl: Stop sharing pf_ruleset.c with the kernel

2021-01-06 Thread Cy Schubert
Yes, this is the problem. -- Cheers, Cy Schubert FreeBSD UNIX: Web: https://FreeBSD.org NTP: Web: https://nwtime.org The need of the many outweighs the greed of the few. In message , Kyle Evans writes: > I note that the original report says buildkernel, but the l

Re: git: fda7daf06301 - main - pfctl: Stop sharing pf_ruleset.c with the kernel

2021-01-06 Thread Kyle Evans
I note that the original report says buildkernel, but the log exercpt is from buildworld. Looks like pf_ruleset.c kept the name, so this needs a dependency cleanup rule in tools/build to build the right one on WITHOUT_CLEAN builds. On Wed, Jan 6, 2021, 03:22 Kristof Provost wrote: > On 6 Jan 202

Re: git: fda7daf06301 - main - pfctl: Stop sharing pf_ruleset.c with the kernel

2021-01-06 Thread Kristof Provost
On 6 Jan 2021, at 10:20, Hartmann, O. wrote: > > amd64. > kernel config is custom, no debug, IPFW, MAC. pf is not used, we use IPFW. Please share your kernel config as well. Best regards, Kristof ___ dev-commits-src-all@freebsd.org mailing list https://l

Re: git: fda7daf06301 - main - pfctl: Stop sharing pf_ruleset.c with the kernel

2021-01-06 Thread Hartmann, O.
On Wed, 06 Jan 2021 07:28:27 +0100 "Kristof Provost" wrote: > On 6 Jan 2021, at 5:50, Hartmann, O. wrote: > > On Tue, 5 Jan 2021 22:37:54 GMT > > Kristof Provost wrote: > > > >> The branch main has been updated by kp: > >> > >> URL: > >> https://cgit.FreeBSD.org/src/commit/?id=fda7daf06301bee

Re: git: fda7daf06301 - main - pfctl: Stop sharing pf_ruleset.c with the kernel

2021-01-05 Thread Kristof Provost
On 6 Jan 2021, at 5:50, Hartmann, O. wrote: On Tue, 5 Jan 2021 22:37:54 GMT Kristof Provost wrote: The branch main has been updated by kp: URL: https://cgit.FreeBSD.org/src/commit/?id=fda7daf06301beef1bdad39891232a12c6925b22 commit fda7daf06301beef1bdad39891232a12c6925b22 Author: Krist

Re: git: fda7daf06301 - main - pfctl: Stop sharing pf_ruleset.c with the kernel

2021-01-05 Thread Hartmann, O.
On Tue, 5 Jan 2021 22:37:54 GMT Kristof Provost wrote: > The branch main has been updated by kp: > > URL: > https://cgit.FreeBSD.org/src/commit/?id=fda7daf06301beef1bdad39891232a12c6925b22 > > commit fda7daf06301beef1bdad39891232a12c6925b22 > Author: Kristof Provost > AuthorDate: 2020-12-