Re: NO INET6 warning

2014-09-07 Thread Adrian Chadd
No, because I may end up building a kernel that has no INET6 for a cut
down platform that has no src.conf option set.


-a


On 7 September 2014 19:34, Glen Barber  wrote:
> On Sun, Sep 07, 2014 at 03:34:35PM -0700, yaneurab...@gmail.com wrote:
>> On Sep 7, 2014, at 14:08, Glen Barber  wrote:
>>
>> > On Sun, Sep 07, 2014 at 02:03:35PM -0700, Sean Bruno wrote:
>> >> make[4]: "/home/sbruno/bsd/fbsd_head/sys/modules/if_gif/Makefile" line
>> >> 12: warning: Couldn't read shell's output for
>> >> "cat 
>> >> /home/sbruno/bsd/obj/mips/mips.mips/home/sbruno/bsd/fbsd_head/sys/WZR-300HP/opt_inet6.h"
>> >>
>> >>
>> >> Shouldn't this cat be done in a saner way?
>> >>
>> >
>> > This is done quite often throughout the tree, and bmake(1) does not like
>> > it when there is no value assigned from the '!=' expansion.
>> >
>> > The fix I've seen most commonly done is to echo a newline after the
>> > assignment, such as:
>> >
>> > Index: sys/modules/if_gif/Makefile
>> > ===
>> > --- sys/modules/if_gif/Makefile (revision 271215)
>> > +++ sys/modules/if_gif/Makefile (working copy)
>> > @@ -9,7 +9,7 @@ KMOD=   if_gif
>> > SRCS=   if_gif.c in_gif.c opt_inet.h opt_inet6.h opt_mrouting.h
>> >
>> > .if defined(KERNBUILDDIR)
>> > -OPT_INET6!= cat ${KERNBUILDDIR}/opt_inet6.h
>> > +OPT_INET6!= cat ${KERNBUILDDIR}/opt_inet6.h; echo
>> > .if empty(OPT_INET6)
>> > MK_INET6_SUPPORT=no
>> > .endif
>>
>> Shouldn’t this all be removed and replaced with equivalent logic provided by 
>> kern.opts.mk?
>
> Probably.
>
> Glen
>
___
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: NO INET6 warning

2014-09-07 Thread Glen Barber
On Sun, Sep 07, 2014 at 03:34:35PM -0700, yaneurab...@gmail.com wrote:
> On Sep 7, 2014, at 14:08, Glen Barber  wrote:
> 
> > On Sun, Sep 07, 2014 at 02:03:35PM -0700, Sean Bruno wrote:
> >> make[4]: "/home/sbruno/bsd/fbsd_head/sys/modules/if_gif/Makefile" line
> >> 12: warning: Couldn't read shell's output for
> >> "cat 
> >> /home/sbruno/bsd/obj/mips/mips.mips/home/sbruno/bsd/fbsd_head/sys/WZR-300HP/opt_inet6.h"
> >> 
> >> 
> >> Shouldn't this cat be done in a saner way?
> >> 
> > 
> > This is done quite often throughout the tree, and bmake(1) does not like
> > it when there is no value assigned from the '!=' expansion.
> > 
> > The fix I've seen most commonly done is to echo a newline after the
> > assignment, such as:
> > 
> > Index: sys/modules/if_gif/Makefile
> > ===
> > --- sys/modules/if_gif/Makefile (revision 271215)
> > +++ sys/modules/if_gif/Makefile (working copy)
> > @@ -9,7 +9,7 @@ KMOD=   if_gif
> > SRCS=   if_gif.c in_gif.c opt_inet.h opt_inet6.h opt_mrouting.h
> > 
> > .if defined(KERNBUILDDIR)
> > -OPT_INET6!= cat ${KERNBUILDDIR}/opt_inet6.h
> > +OPT_INET6!= cat ${KERNBUILDDIR}/opt_inet6.h; echo
> > .if empty(OPT_INET6)
> > MK_INET6_SUPPORT=no
> > .endif
> 
> Shouldn’t this all be removed and replaced with equivalent logic provided by 
> kern.opts.mk?

Probably.

Glen



pgpKKsOpytnNj.pgp
Description: PGP signature


Re: NO INET6 warning

2014-09-07 Thread yaneurabeya
On Sep 7, 2014, at 14:08, Glen Barber  wrote:

> On Sun, Sep 07, 2014 at 02:03:35PM -0700, Sean Bruno wrote:
>> make[4]: "/home/sbruno/bsd/fbsd_head/sys/modules/if_gif/Makefile" line
>> 12: warning: Couldn't read shell's output for
>> "cat 
>> /home/sbruno/bsd/obj/mips/mips.mips/home/sbruno/bsd/fbsd_head/sys/WZR-300HP/opt_inet6.h"
>> 
>> 
>> Shouldn't this cat be done in a saner way?
>> 
> 
> This is done quite often throughout the tree, and bmake(1) does not like
> it when there is no value assigned from the '!=' expansion.
> 
> The fix I've seen most commonly done is to echo a newline after the
> assignment, such as:
> 
> Index: sys/modules/if_gif/Makefile
> ===
> --- sys/modules/if_gif/Makefile   (revision 271215)
> +++ sys/modules/if_gif/Makefile   (working copy)
> @@ -9,7 +9,7 @@ KMOD= if_gif
> SRCS= if_gif.c in_gif.c opt_inet.h opt_inet6.h opt_mrouting.h
> 
> .if defined(KERNBUILDDIR)
> -OPT_INET6!= cat ${KERNBUILDDIR}/opt_inet6.h
> +OPT_INET6!= cat ${KERNBUILDDIR}/opt_inet6.h; echo
> .if empty(OPT_INET6)
> MK_INET6_SUPPORT=no
> .endif

Shouldn’t this all be removed and replaced with equivalent logic 
provided by kern.opts.mk?
Thanks!
-Garrett


signature.asc
Description: Message signed with OpenPGP using GPGMail


Re: NO INET6 warning

2014-09-07 Thread Sean Bruno
On Sun, 2014-09-07 at 17:08 -0400, Glen Barber wrote:
> On Sun, Sep 07, 2014 at 02:03:35PM -0700, Sean Bruno wrote:
> > make[4]: "/home/sbruno/bsd/fbsd_head/sys/modules/if_gif/Makefile" line
> > 12: warning: Couldn't read shell's output for
> > "cat 
> > /home/sbruno/bsd/obj/mips/mips.mips/home/sbruno/bsd/fbsd_head/sys/WZR-300HP/opt_inet6.h"
> > 
> > 
> > Shouldn't this cat be done in a saner way?
> > 
> 
> This is done quite often throughout the tree, and bmake(1) does not like
> it when there is no value assigned from the '!=' expansion.
> 
> The fix I've seen most commonly done is to echo a newline after the
> assignment, such as:
> 
> Index: sys/modules/if_gif/Makefile
> ===
> --- sys/modules/if_gif/Makefile   (revision 271215)
> +++ sys/modules/if_gif/Makefile   (working copy)
> @@ -9,7 +9,7 @@ KMOD= if_gif
>  SRCS=if_gif.c in_gif.c opt_inet.h opt_inet6.h opt_mrouting.h
>  
>  .if defined(KERNBUILDDIR)
> -OPT_INET6!= cat ${KERNBUILDDIR}/opt_inet6.h
> +OPT_INET6!= cat ${KERNBUILDDIR}/opt_inet6.h; echo
>  .if empty(OPT_INET6)
>  MK_INET6_SUPPORT=no
>  .endif
> 
> Glen
> 


LGTM

sean

___
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: NO INET6 warning

2014-09-07 Thread Glen Barber
On Sun, Sep 07, 2014 at 02:03:35PM -0700, Sean Bruno wrote:
> make[4]: "/home/sbruno/bsd/fbsd_head/sys/modules/if_gif/Makefile" line
> 12: warning: Couldn't read shell's output for
> "cat 
> /home/sbruno/bsd/obj/mips/mips.mips/home/sbruno/bsd/fbsd_head/sys/WZR-300HP/opt_inet6.h"
> 
> 
> Shouldn't this cat be done in a saner way?
> 

This is done quite often throughout the tree, and bmake(1) does not like
it when there is no value assigned from the '!=' expansion.

The fix I've seen most commonly done is to echo a newline after the
assignment, such as:

Index: sys/modules/if_gif/Makefile
===
--- sys/modules/if_gif/Makefile (revision 271215)
+++ sys/modules/if_gif/Makefile (working copy)
@@ -9,7 +9,7 @@ KMOD=   if_gif
 SRCS=  if_gif.c in_gif.c opt_inet.h opt_inet6.h opt_mrouting.h
 
 .if defined(KERNBUILDDIR)
-OPT_INET6!= cat ${KERNBUILDDIR}/opt_inet6.h
+OPT_INET6!= cat ${KERNBUILDDIR}/opt_inet6.h; echo
 .if empty(OPT_INET6)
 MK_INET6_SUPPORT=no
 .endif

Glen



pgpG3iSutZF5b.pgp
Description: PGP signature


NO INET6 warning

2014-09-07 Thread Sean Bruno
make[4]: "/home/sbruno/bsd/fbsd_head/sys/modules/if_gif/Makefile" line
12: warning: Couldn't read shell's output for
"cat 
/home/sbruno/bsd/obj/mips/mips.mips/home/sbruno/bsd/fbsd_head/sys/WZR-300HP/opt_inet6.h"


Shouldn't this cat be done in a saner way?



sean

___
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"