Re: WITHOUT_NIS after bsd.opts.mk / src.opts.mk split
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
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
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
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
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
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
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"