On Fri, Nov 17, 2000 at 10:55:32PM +0900, Makoto MATSUSHITA wrote:
> Attached below is a patch which I've made before. If nothing is going
> wrong, please commit this change
I committed something simular. BTW, you made ``make depend'' a
requirement, which we did not have agreement on.
To
On Tue, 14 Nov 2000, Warner Losh wrote:
> make depend is already *REQUIRED* when you are updating a kernel from
> an older version of the kernel. For config -r FOO kernels it isn't.
It is neither necessary nor sufficient for updating. config -r removes
conflicting garbage from previous builds.
David O'Brien wrote:
>
> > > modules-depend:
> > > @mkdir -p ${.OBJDIR}/modules
> > > ! cd $S/modules; env ${MKMODULESENV} ${MAKE} obj
> > > ! env ${MKMODULESENV} ${MAKE} depend
>
> This is broken for non -j case.
Yes, this was known. The right diff was given at the beginn
In message <[EMAIL PROTECTED]> "David O'Brien" writes:
: [stable dropped, this should have only been in a single list to start with!!]
Agreed. My summary:
o We disagree about the support impact
o Peter's stuff may OBE this whole thread
o My change shouldn't be committed u
[stable dropped, this should have only been in a single list to start with!!]
On Tue, Nov 14, 2000 at 01:27:32PM -0700, Warner Losh wrote:
> : I'd rather take a major compile time hit and be deterministic than not.
>
> I'd rather not. We don't do an implicit make obj in the rest of the
> tree.
In message <[EMAIL PROTECTED]> "David O'Brien" writes:
: On Mon, Nov 13, 2000 at 09:17:54PM -0700, Warner Losh wrote:
: > The implications are that make obj isn't done unless you've run make
: > depend first. If a new directory is added and a make depend isn't
: > run, then the modules won't get
On Mon, Nov 13, 2000 at 08:02:47PM -0800, Marcel Moolenaar wrote:
> Any objections?
Yes.
> (patches follow for your convenience)
[its easier to read patches when they aren't quoted in their entirety ;-)]
> > modules-depend:
> > @mkdir -p ${.OBJDIR}/modules
> > ! cd $S/modules;
On Mon, Nov 13, 2000 at 09:17:54PM -0700, Warner Losh wrote:
> The implications are that make obj isn't done unless you've run make
> depend first. If a new directory is added and a make depend isn't
> run, then the modules won't get built into the obj tree, but instead
> will be built into $S/mo
In message <[EMAIL PROTECTED]> Marcel Moolenaar writes:
: BTW: I'm also looking at Warner's patch. Maybe that's the better fix for
: it, but I have to dig into the Makefiles a bit more to get a better
: picture...
The implications are that make obj isn't done unless you've run make
depend first.
Makoto MATSUSHITA wrote:
>
> Ouch... give me one more chance to submit a patch. Here's summary:
I see no reason to not commit Makoto-san's patches with the fix he sent
me for the modules-depend target. The fix is (modulo indentation):
> modules-depend:
> ! cd $S/modules; env ${MKMODULES
[-stable removed from the cc list]
Makoto MATSUSHITA wrote:
>
> marcel> No you can't. $S expands to "../.." which only works for the
> marcel> first cd in the -jX case. The second cd will fail.
>
> Ouch...
Sorry :-)
> give me one more chance to submit a patch. Here's summary:
You're not goin
marcel> No you can't. $S expands to "../.." which only works for the
marcel> first cd in the -jX case. The second cd will fail.
Ouch... give me one more chance to submit a patch. Here's summary:
* src/release/Makefile should use 'module-depend' while
checking dependancy of mod
David O'Brien wrote:
>
> On Mon, Nov 13, 2000 at 01:19:38PM -0500, Marcel Moolenaar wrote:
> > I'll commit a fix with just semi-colons today for all architectures if
> > someone hasn't done that already by that time.
>
> Can you post a patch first. There seems to be some subtleties here that
>
In message <[EMAIL PROTECTED]> "David O'Brien" writes:
: It doesn't warn the user, it errors out (possibly a suttle distinction
: I'm making). Also in the past a `make depend' for the kernel was not
: required. Just highly suggested. Are we really prepared to make it a
: requirement now?
I am,
On Mon, Nov 13, 2000 at 12:23:08PM -0700, Warner Losh wrote:
> : > I think that make has no business doing an implicit make obj for the
> : > all target.
> : Someone has to run `make obj' for the modules tree. How are you doing it
> : locally?
>
> Right now we do it twice. Once in make dpeend a
On Mon, Nov 13, 2000 at 01:19:38PM -0500, Marcel Moolenaar wrote:
> I'll commit a fix with just semi-colons today for all architectures if
> someone hasn't done that already by that time.
Can you post a patch first. There seems to be some subtleties here that
might make a review useful.
--
--
In message <[EMAIL PROTECTED]> "David O'Brien" writes:
: On Sun, Nov 12, 2000 at 11:58:39PM -0700, Warner Losh wrote:
: > In message <[EMAIL PROTECTED]> Makoto MATSUSHITA writes:
: > : It does not fix this problem. However, if we separate the execution of
: > : "make obj" and "make all", we can av
Marcel Moolenaar wrote:
>
> David O'Brien wrote:
> >
> > On Mon, Nov 13, 2000 at 07:51:11PM +0900, Makoto MATSUSHITA wrote:
> > > ! cd $S/modules && env ${MKMODULESENV} ${MAKE} obj all
> > > --- 310,322
> > > ! cd $S/modules && env ${MKMODULESENV} ${MAKE} obj
> > > ! cd $S/module
David O'Brien wrote:
>
> On Mon, Nov 13, 2000 at 07:51:11PM +0900, Makoto MATSUSHITA wrote:
> > ! cd $S/modules && env ${MKMODULESENV} ${MAKE} obj all
> > --- 310,322
> > ! cd $S/modules && env ${MKMODULESENV} ${MAKE} obj
> > ! cd $S/modules && env ${MKMODULESENV} ${MAKE} all
>
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
> >
On Mon, Nov 13, 2000 at 07:51:11PM +0900, Makoto MATSUSHITA wrote:
> ! cd $S/modules && env ${MKMODULESENV} ${MAKE} obj all
> --- 310,322
> ! cd $S/modules && env ${MKMODULESENV} ${MAKE} obj
> ! cd $S/modules && env ${MKMODULESENV} ${MAKE} all
I can certainly commit this type of
On Sun, Nov 12, 2000 at 11:58:39PM -0700, Warner Losh wrote:
> In message <[EMAIL PROTECTED]> Makoto MATSUSHITA writes:
> : It does not fix this problem. However, if we separate the execution of
> : "make obj" and "make all", we can avoid (again, not *fix*) the problem.
> : Maybe this change is re
On Mon, 13 Nov 2000, Makoto MATSUSHITA wrote:
> bde> (In the above example, the targets are built concurrently and race
> bde> each other. This is bad when the `all' target wins the race. The
> bde> `obj' target runs faster, so it usually wins the race except in the
> bde> first directory (3dfx
bde> (In the above example, the targets are built concurrently and race
bde> each other. This is bad when the `all' target wins the race. The
bde> `obj' target runs faster, so it usually wins the race except in the
bde> first directory (3dfx)). More .ORDER statements in *.mk are required.
Tha
On Sun, 12 Nov 2000, Marcel Moolenaar wrote:
> Makoto MATSUSHITA wrote:
> >
> > % make -j 2 modules
> > cd ../../modules && env MAKEOBJDIRPREFIX=/usr/src/sys/compile/GENERIC/modules
>KMODDIR=/boot/kernel make obj all
> > ===> 3dfx
> > ===> 3dfx
> > Warning: Object directory not changed from ori
marcel> The net effect is that these targets are built in parallel,
marcel> which obviously isn't right.
Agreed. If there is no reason to do "make obj all" (specify both 'obj'
and 'all' target at once), it's maybe a better solution.
-- -
Makoto `MAR' MATSUSHITA
To Unsubscribe: send mail to [E
Makoto MATSUSHITA wrote:
>
> % make -j 2 modules
> cd ../../modules && env MAKEOBJDIRPREFIX=/usr/src/sys/compile/GENERIC/modules
>KMODDIR=/boot/kernel make obj all
> ===> 3dfx
> ===> 3dfx
> Warning: Object directory not changed from original /usr/src/sys/modules/3dfx
> (... ok, break it ...)
Th
In message <[EMAIL PROTECTED]> Makoto MATSUSHITA writes:
: It does not fix this problem. However, if we separate the execution of
: "make obj" and "make all", we can avoid (again, not *fix*) the problem.
: Maybe this change is reasonable; there is few meaning doing "make
: kernel-depend" in "doMOD
28 matches
Mail list logo