lkml.org 
[lkml]   [2004]   [Jun]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 0/2] kbuild updates
On Mon, Jun 21, 2004 at 12:54:01AM +0200, Martin Schlemmer wrote:
> >
> > #
> > # where's the kernel source?
> > #
> >
> > if [ -d /lib/modules/`uname -r`/source ]; then
> > # 2.6.8 and newer
> > KERNDIR="/lib/modules/`uname -r`/source"
> > else
> > # pre 2.6.8 kernels
> > KERNDIR="/lib/modules/`uname -r`/build"
> > fi
> >
> > Yeah?
>
> Yes, as said before, I can understand the name change. The point is
> more that the 'build' symlink will change in behavior in certain
> circumstances, and because many projects already support 2.6, and
> make use of the 'build' symlink, they will break.

But they were already broken.
And the patch actually helps here.

See the following table:



Current kbuild With patch
simple module, no grepping
kernel source + output mixed OK OK

simple module, no grepping
kernel source separate from output FAIL *1 OK


module grepping in src
kernel source + output mixed OK OK

module grepping in src
kernel source separate from output FAIL *2 FAIL *3




FAIL *1 Module cannot build because it fails to locate the compiled files used for kbuild.
Also the kernel configuration is missing.

FAIL *2 Same as FAIL *1. Module cannot build because kbuild compiled files are missing and
it cannot access kernel configuration.
Also it fails to locate files when grepping.

FAIL *3 Grep will fail because it try to grep in files located under build/



The above table clearly shows that this patch fix building external modules in the
case where no grepping were preformed.
And the error were simplified in the last case.
So improvements all over.

What one should realise is that the patch makes it less painfull when kernels
are build using the separate output directory option.
And when that feature is not used - no change in behaviour occur.

Sam

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 14:04    [from the cache]
©2003-2011 Jasper Spaans