Date: Wed, 22 Dec 2021 12:52:15 -0600 From: Warner Losh <imp@bsdimp.com> To: Gary Jennejohn <gljennjohn@gmail.com> Cc: FreeBSD Current <freebsd-current@freebsd.org> Subject: Re: WITHOUT_PF breaks buildworld Message-ID: <CANCZdfrjfcMh_zDnVEqMQHWQ3nL0XkqkqE%2Bf_Nv1jfeCh2XPCA@mail.gmail.com> In-Reply-To: <20211222193725.19f2f03a@ernst.home> References: <20211219114723.338b235e@ernst.home> <61bf1204.1c69fb81.2c8fc.3280SMTPIN_ADDED_BROKEN@mx.google.com> <20211219122443.1c84093f@ernst.home> <CAGgez=%2B_jdDn6dfOeo%2BuBeeFNzBfD57HL4wUrw=X-BmGg7mPyg@mail.gmail.com> <74FC7625-295C-4DEC-BF35-434B5F8D7832@FreeBSD.org> <20211222165154.33ef54fd@ernst.home> <20211222170151.05a4afe3@ernst.home> <CANCZdfqAdO1HMDnWMsEU1LarRj36GzA%2Bog%2BxGkBY6rziA_yLWQ@mail.gmail.com> <20211222193725.19f2f03a@ernst.home>
index | next in thread | previous in thread | raw e-mail
[-- Attachment #1 --]
On Wed, Dec 22, 2021, 12:37 PM Gary Jennejohn <gljennjohn@gmail.com> wrote:
> On Wed, 22 Dec 2021 10:58:50 -0600
> Warner Losh <imp@bsdimp.com> wrote:
>
> > On Wed, Dec 22, 2021, 10:02 AM Gary Jennejohn <gljennjohn@gmail.com>
> wrote:
> >
> > >
> > > This simple patch to /usr/src/include/Makefile fixes it for me:
> > >
> > > --- Makefile.orig 2021-12-22 13:37:02.817745000 +0100
> > > +++ Makefile 2021-12-22 13:37:12.177336000 +0100
> > > @@ -246,9 +246,7 @@
> > > INCSGROUPS+= IPFILTER
> > > .endif
> > >
> > > -.if ${MK_PF} != "no"
> > > INCSGROUPS+= PF
> > > -.endif
> > >
> > > .if ${MK_CDDL} != "no"
> > > INCSGROUPS+= NVPAIR
> > >
> > > Since pf.h is used so widely in the tree it's probably the simplest
> fix.
> > >
> >
> > I like this. I'd unconditionally remove the test in preference to adding
> it
> > conditionally.
> >
>
> If I still had my commit bit I'd do it myself :-)
>
I can do it tonight. I'm on the road today dri ING back from Omaha..
Warner
--
> Gary Jennejohn
>
[-- Attachment #2 --]
<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Dec 22, 2021, 12:37 PM Gary Jennejohn <<a href="mailto:gljennjohn@gmail.com">gljennjohn@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Wed, 22 Dec 2021 10:58:50 -0600<br>
Warner Losh <<a href="mailto:imp@bsdimp.com" target="_blank" rel="noreferrer">imp@bsdimp.com</a>> wrote:<br>
<br>
> On Wed, Dec 22, 2021, 10:02 AM Gary Jennejohn <<a href="mailto:gljennjohn@gmail.com" target="_blank" rel="noreferrer">gljennjohn@gmail.com</a>> wrote:<br>
> <br>
> ><br>
> > This simple patch to /usr/src/include/Makefile fixes it for me:<br>
> ><br>
> > --- Makefile.orig 2021-12-22 13:37:02.817745000 +0100<br>
> > +++ Makefile 2021-12-22 13:37:12.177336000 +0100<br>
> > @@ -246,9 +246,7 @@<br>
> > INCSGROUPS+= IPFILTER<br>
> > .endif<br>
> ><br>
> > -.if ${MK_PF} != "no"<br>
> > INCSGROUPS+= PF<br>
> > -.endif<br>
> ><br>
> > .if ${MK_CDDL} != "no"<br>
> > INCSGROUPS+= NVPAIR<br>
> ><br>
> > Since pf.h is used so widely in the tree it's probably the simplest fix.<br>
> > <br>
> <br>
> I like this. I'd unconditionally remove the test in preference to adding it<br>
> conditionally.<br>
> <br>
<br>
If I still had my commit bit I'd do it myself :-)<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">I can do it tonight. I'm on the road today dri ING back from Omaha..</div><div dir="auto"><br></div><div dir="auto">Warner</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
-- <br>
Gary Jennejohn<br>
</blockquote></div></div></div>
home |
help
Want to link to this message? Use this
URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfrjfcMh_zDnVEqMQHWQ3nL0XkqkqE%2Bf_Nv1jfeCh2XPCA>