On Sun, Sep 07, 2014 at 03:34:35PM -0700, yaneurab...@gmail.com wrote:
> On Sep 7, 2014, at 14:08, Glen Barber <g...@freebsd.org> 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

Attachment: pgpKKsOpytnNj.pgp
Description: PGP signature

Reply via email to