lkml.org 
[lkml]   [2009]   [Jan]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: PATCH [0/3]: Simplify the kernel build by removing perl.
On Sat, Jan 03, 2009 at 07:45:34PM -0600, Rob Landley wrote:
> > With respect to your three patches the plan is to:
> > - add the updated timeconst patch to kbuild-next
> > - add the updated cpu-feature patch to kbuild-next
> >
> > - the patch touching headers_install will not be merged.
> > The way forward for headers_install is to combine the
> > unifdef feature and the header modifications.
>
> Since you're turning down an existing patch in favor of a theoretical patch, I
> assume you have plans to do this yourself?

If noone else beats me I will do so - yes.
>
> > And this must be in a single program that can process
> > all headers in one go so the install process becomes so fast
> > that we do not worry about if it was done before or not.
> > Then we can avoid all the .* files in the directory
> > where we isntall the headers.
>
> What if they run out of disk space halfway through writing a file and thus it
> creates a short file (or a 0 length file where the dentry was created but no
> blocks could be allocated for the write)?

Then they fail and make will know. Then may leave a file or 100
but it still failed. At next run everything will be done right
assuming the culprint has been fixed.

> I can try to make the shell version more readable, and more powerful. It's
> already noticeably faster than the perl version. I have no objections to
> making unifdef do all of this, I just haven't got any interest either.

I have no interest in merging a shell version.
I clearly expressed above that we need a _single_ program doing
all of the preparations and we do not need a reimplmentatio of the
current headers_install.
I also explained why.

Sam


\
 
 \ /
  Last update: 2009-01-04 09:11    [W:0.130 / U:0.224 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site