Re: PATCH [0/3]: Simplify the kernel build by removing perl.

On Saturday 03 January 2009 14:10:59 Sam Ravnborg wrote:
I'll fix this and resubmit, it just wasn't ready last night. (If the
merge window is closing soon I could resubmit the other two with Sam's
suggestions and resubmit this one next time 'round, but it was only a
couple days to write in the first place, once I finally figured out what
the perl version was trying to _do_...)

For kbuild only fixes and trivial stuff will be merged until next merge
window. Neither of the three patches fall into that category.

*shrug* I poke my head into kernel development every few months, and have
just enough familiarity with it to remember that "changes go in during the
merge window". Seemed a good time to post 'em.

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?

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)?

I expected headers_install to overwrite destination files and create
directories with -p, so if you interrupt it you can just re-run it again with
the same arguments and it could install everything again cleanly over existing
partial output. I take it this isn't what's happening, or that isn't
sufficient somehow?

I didn't look too closely at what the makefile was doing (it makes my eyes
bleed), I just rewrote the perl script and changed the call. I could try to
upgrade the script to not need the makefile to tell it what files to work on,
and just take the appropriate top level include directory and the output
directory and figure out which files it needs to operate on by itself so it
_does_ work that way. Figuring out where the make file is getting this info
from now shouldn't be too much harder than reading perl scripts and figuring
out what they're doing.

Not during this merge window, though.

The program is a prime candidate for a small C program
and I hope someone can take the challenge to write it.

Good luck with that. Having written most of the busybox sed implementation,
and before that having written my own regex implementation back under OS/2,
I've pretty much gotten my fill of doing regexes in C, at least without a good
reason.

I suppose if I was feeling really bored I could try to implement unifdef as a
sed script, but the only way to get a single invocation of sed to work on a
bunch of individual files coherently is to use -i mode, which A) ain't in
susv3 (although busybox sed supports it), B) would involve a cp of all the
files to the destination first, which is kind of ugly.

Migrating from perl to shell does not help us here
and the shell version was less readable than the perl version.

The shell version is less readable but you never noticed that the perl version
isn't using its $ARCH argument? *shrug* Ok.

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.

Sam

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

Relevant Pages

  • Re: Windows shortcut without dos prompt
    ... Yes I'm using ActiveState Perl 5.6.1. ... Please quote an appropriate amount of context in your reply, ... -- I need to start a perl script from a Windows Start Menu shortcut. ... -- seen a way to create the shortcut so that a DOS window does not open ...
    (comp.lang.perl.misc)
  • Re: Perl CGI.pm POST_MAX problem in FC7 - when will latest version be available in archives?
    ... the CGI script will peg the CPU until the web server ... new upstream perl release will need to include an updated CGI.pm. ... This could get updated via a patch. ... patch CGI.pm to avoid the specific bug you're encountering (CPAN bug ...
    (Fedora)
  • Re: Get Windows Task-Names?!
    ... > Purl Gurl wrote: ... -12189 MSGSRV32.EXE work window name HighCriteria TotalRecorder Mrqx1FQGyj4c1kycrB ... "Power Meter" although this meter is not running. ... which is highly adaptable to Perl. ...
    (comp.lang.perl.misc)
  • Windows: stop automatic closing of the terminal
    ... I had longstanding problems with Activestate Perl under Windows XP ... which closed the terminal window too early. ... the terminal window is closed almost ... I thought I should add something which prompts for a keystroke at the ...
    (perl.beginners)
  • Re: IE and Outlook Express flashing file dialog box
    ... issues with that patch than just this OE flashing behavior. ... OE window and that didn't work for me, ... Sir Timbit ... "Cumulative Security Update for Internet Explorer for Windows XP" ...
    (microsoft.public.windows.inetexplorer.ie6_outlookexpress)