Bruce Evans wrote:
> 
> > Index: Makefile.i386
> > ===================================================================
> > RCS file: /home/ncvs/src/sys/conf/Makefile.i386,v
> > retrieving revision 1.212
> > diff -u -r1.212 Makefile.i386
> > --- Makefile.i386       2000/10/29 09:47:50     1.212
> > +++ Makefile.i386       2000/11/13 07:49:00
> > @@ -271,11 +271,13 @@
> >
> >  modules:
> >         @mkdir -p ${.OBJDIR}/modules
> > -       cd $S/modules && env ${MKMODULESENV} ${MAKE} obj all
> > +       cd $S/modules && env ${MKMODULESENV} ${MAKE} obj && \
> > +               env ${MKMODULESENV} ${MAKE} all
> >
> >  modules-depend:
> >         @mkdir -p ${.OBJDIR}/modules
> > -       cd $S/modules && env ${MKMODULESENV} ${MAKE} obj depend
> > +       cd $S/modules && env ${MKMODULESENV} ${MAKE} obj $$ \
> > +               env ${MKMODULESENV} ${MAKE} depend
> >
> >  modules-clean:
> >         cd $S/modules && env ${MKMODULESENV} ${MAKE} clean
> >
> 
> `&&' should never be used in shell commands in makefiles, although it
> may be only a bad example.

Mostly that, yes. The first try for a "fix" was:

        cd $S/modules && env ${MKMODULESENV} ${MAKE} obj
        cd $S/modules && env ${MKMODULESENV} ${MAKE} all

But since $S expands to "../.." and -j runs a single shell, the first cd
will succeed, but not the second.

I simply copied the "construction" in the second and posted fix.

I'll commit a fix with just semi-colons today for all architectures if
someone hasn't done that already by that time.

-- 
Marcel Moolenaar
  mail: [EMAIL PROTECTED] / [EMAIL PROTECTED]
  tel:  (408) 447-4222


To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message

Reply via email to