Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 09 May 2014 05:22:53 +0300
From:      "Sulev-Madis Silber (ketas)" <madis555@hot.ee>
To:        Warner Losh <imp@bsdimp.com>
Cc:        freebsd-current@freebsd.org, Guy Yur <guyyur@gmail.com>
Subject:   Re: WITHOUT_NIS after bsd.opts.mk / src.opts.mk split
Message-ID:  <536C3BFD.1010700@hot.ee>
In-Reply-To: <1A74E775-B873-44E1-8323-D3C292A3D02E@bsdimp.com>
References:  <CAC67Hz_TLidKJfg1McqgqGKyOHcpE%2BuhHPHWWiEynpD9YrFV7w@mail.gmail.com> <1A74E775-B873-44E1-8323-D3C292A3D02E@bsdimp.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On 2014-05-09 02:54, Warner Losh wrote:
> 
> On May 8, 2014, at 3:26 PM, Guy Yur <guyyur@gmail.com> wrote:
> 
>> Hi,
>>
>> After the bsd.opts.mk / src.opts.mk split
>> WITHOUT_NIS in src.conf doesn't work.
> 
> It should still work… At least that’s the intention...
> 
>> src.conf is included in src.opts.mk after bsd.own.mk
>> which includes bsd.opts.mk.
> 
> Yea, that’s a problem… It should be included after.
> 
>> Should bsd.opts.mk options overrides now be set in
>> make.conf instead of src.conf?
> 
> That’s a good workaround until I get that fix tested and committed. Or you could include src.conf in make.conf at the end. Either will have the same effect.
> 
> Here’s the fix I’m testing, if you’d like to test that instead...
> 
> diff -r d69444b828c1 share/mk/src.opts.mk
> --- a/share/mk/src.opts.mk
> +++ b/share/mk/src.opts.mk
> @@ -30,17 +30,15 @@
>  .if !target(__<src.opts.mk>__)
>  __<src.opts.mk>__:
>  
> -# Compat -- needed still?
> -.include <bsd.own.mk>
> -
> -# Allow user to configure things, but in the future this will move
> -# elsehwere...
> -
> +# Allow user to configure things that only effect src tree builds.
>  SRCCONF?=	/etc/src.conf
>  .if exists(${SRCCONF}) || ${SRCCONF} != "/etc/src.conf"
>  .include "${SRCCONF}"
>  .endif
>  
> +# Must be included after src.conf
> +.include <bsd.own.mk>
> +
>  #
>  # Define MK_* variables (which are either "yes" or "no") for users
>  # to set via WITH_*/WITHOUT_* in /etc/src.conf and override in the
> 
> 
>> Was on r265455, updated to r265715 and rebuilt with -DNO_CLEAN.
> 
> Yea, sorry about missing this subtle issue in the split. There was another report of something similar that I hadn’t tracked down, but your report pointed me to where I needed to go.
> 
> Warner
> 


Finally! Trying it now...
I was about to take deep look into it because that bothered me so much.



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?536C3BFD.1010700>