Re: WITHOUT_NIS after bsd.opts.mk / src.opts.mk split

2014-05-09 Thread Guy Yur
Hi,

On Fri, May 9, 2014 at 2:54 AM, Warner Losh  wrote:
>
> On May 8, 2014, at 3:26 PM, Guy Yur  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.

Your fix worked for me.


Another issue that is probably very uncommon so I don't know
if anything should be done about it:
WITHOUT_NIS, WITHOUT_KERBEROS and WITHOUT_OPENSSH also affect
bsd.libnames.mk definition of LIBPAM and MINUSLPAM in case
you build programs with bsd make outside of /usr/src that
use LIBPAM/MINUSLPAM and have NO_SHARED / LDFLAGS+=-static.
The addition of ${LIBYPCLNT} / -lypclnt to LDFLAGS will fail to link if world
was installed without NIS.

A brief look in the ports tree doesn't show any program that does this.

Thanks,
Guy
___
freebsd-current@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

Re: WITHOUT_NIS after bsd.opts.mk / src.opts.mk split

2014-05-08 Thread Sulev-Madis Silber (ketas)
On 2014-05-09 07:32, Warner Losh wrote:
> 
> On May 8, 2014, at 10:12 PM, Sulev-Madis Silber (ketas)  
> wrote:
> 
>> On 2014-05-09 02:54, Warner Losh wrote:
>>>
>>> On May 8, 2014, at 3:26 PM, Guy Yur  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()
>>> :
>>>
>>> -# Compat -- needed still?
>>> -.include 
>>> -
>>> -# 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 
>>> +
>>> #
>>> # 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
>>>
>>
>>
>> Sorry, that didn't exactly help. I don't fully get what went so wrong there?
>>
>> Now I got this during install:
>>
>> ---
>> ===> gnu/lib/libregex/doc (install)
>> install-info: not found
>> *** Error code 127
>> ---
>>
>> It was total WTF error but just in case I tried putting ".include
>> <../src.conf>" back, and it worked!
>> I use __MAKE_CONF, and inside that file I have SRCCONF.
> 
> To be clear, you define SRCCONF in /etc/make.conf (or the file defined by 
> __MAKE_CONF). The file defined by SRCCONF has WITHOUT_NIS=t defined, but 
> that’s not effective, even with my change. However, if you add an include to 
> the file defined by __MAKE_CONF, then it is effective… Is that what you are 
> telling me?
> 
>> 9.2, BTW... unsure if it matters here?
> 
> I’m doing my testing on 10-stable… I’ll have to try on my 9.x system…  But it 
> is a lot slower than my 10.x system...
> 
> Warner
> 


Yes, that's exactly what I mean. It seems to partially work now. I
actually have lot of WITHOUT_*'s. That install error is really weird,
never seen it before. All I know is that before all those changes,
everything worked well. And if I .include file, old behavior (everything
works as expected) is back.
___
freebsd-current@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

Re: WITHOUT_NIS after bsd.opts.mk / src.opts.mk split

2014-05-08 Thread Warner Losh

On May 8, 2014, at 10:12 PM, Sulev-Madis Silber (ketas)  wrote:

> On 2014-05-09 02:54, Warner Losh wrote:
>> 
>> On May 8, 2014, at 3:26 PM, Guy Yur  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()
>> :
>> 
>> -# Compat -- needed still?
>> -.include 
>> -
>> -# 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 
>> +
>> #
>> # 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
>> 
> 
> 
> Sorry, that didn't exactly help. I don't fully get what went so wrong there?
> 
> Now I got this during install:
> 
> ---
> ===> gnu/lib/libregex/doc (install)
> install-info: not found
> *** Error code 127
> ---
> 
> It was total WTF error but just in case I tried putting ".include
> <../src.conf>" back, and it worked!
> I use __MAKE_CONF, and inside that file I have SRCCONF.

To be clear, you define SRCCONF in /etc/make.conf (or the file defined by 
__MAKE_CONF). The file defined by SRCCONF has WITHOUT_NIS=t defined, but that’s 
not effective, even with my change. However, if you add an include to the file 
defined by __MAKE_CONF, then it is effective… Is that what you are telling me?

> 9.2, BTW... unsure if it matters here?

I’m doing my testing on 10-stable… I’ll have to try on my 9.x system…  But it 
is a lot slower than my 10.x system...

Warner


signature.asc
Description: Message signed with OpenPGP using GPGMail


Re: WITHOUT_NIS after bsd.opts.mk / src.opts.mk split

2014-05-08 Thread Sulev-Madis Silber (ketas)
On 2014-05-09 02:54, Warner Losh wrote:
> 
> On May 8, 2014, at 3:26 PM, Guy Yur  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()
>  :
>  
> -# Compat -- needed still?
> -.include 
> -
> -# 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 
> +
>  #
>  # 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
> 


Sorry, that didn't exactly help. I don't fully get what went so wrong there?

Now I got this during install:

---
===> gnu/lib/libregex/doc (install)
install-info: not found
*** Error code 127
---

It was total WTF error but just in case I tried putting ".include
<../src.conf>" back, and it worked!
I use __MAKE_CONF, and inside that file I have SRCCONF.

9.2, BTW... unsure if it matters here?
___
freebsd-current@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

Re: WITHOUT_NIS after bsd.opts.mk / src.opts.mk split

2014-05-08 Thread Sulev-Madis Silber (ketas)
On 2014-05-09 02:54, Warner Losh wrote:
> 
> On May 8, 2014, at 3:26 PM, Guy Yur  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()
>  :
>  
> -# Compat -- needed still?
> -.include 
> -
> -# 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 
> +
>  #
>  # 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.
___
freebsd-current@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

Re: WITHOUT_NIS after bsd.opts.mk / src.opts.mk split

2014-05-08 Thread Warner Losh

On May 8, 2014, at 3:26 PM, Guy Yur  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()
 :
 
-# Compat -- needed still?
-.include 
-
-# 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 
+
 #
 # 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



signature.asc
Description: Message signed with OpenPGP using GPGMail


WITHOUT_NIS after bsd.opts.mk / src.opts.mk split

2014-05-08 Thread Guy Yur
Hi,

After the bsd.opts.mk / src.opts.mk split
WITHOUT_NIS in src.conf doesn't work.

src.conf is included in src.opts.mk after bsd.own.mk
which includes bsd.opts.mk.

Should bsd.opts.mk options overrides now be set in
make.conf instead of src.conf?

Was on r265455, updated to r265715 and rebuilt with -DNO_CLEAN.

Thanks,
Guy
___
freebsd-current@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"