Bug 3391 - Add support for specifying an external kernel tree
Summary: Add support for specifying an external kernel tree
Status: RESOLVED FIXED
Alias: None
Product: buildroot
Classification: Unclassified
Component: Other (show other bugs)
Version: unspecified
Hardware: All All
: P5 enhancement
Target Milestone: ---
Assignee: Thomas Petazzoni
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-03-07 16:04 UTC by Martin Hicks
Modified: 2011-10-09 20:45 UTC (History)
1 user (show)

See Also:
Host:
Target:
Build:


Attachments
external kernel tree patch (3.33 KB, application/octet-stream)
2011-03-07 16:04 UTC, Martin Hicks
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Hicks 2011-03-07 16:04:52 UTC
Created attachment 3079 [details]
external kernel tree patch

I didn't get any comments from the list, so I'm posting this here so I doesn't get lost.  One thing that I changed from my original post is to conditionally create
the .stamp_{downloaded,extracted,patched} files.  Doing do unconditionally causes unnecessary invocation of make in the kernel directory during each rebuild.

From my RFC e-mail to the buildroot list:

I wanted to compile a kernel from an external git tree, so adding
support for using this tree as the kernel source for buildroot
seemed logical.

The thing I'm seeking comments on is my short-circuiting of
the .stamp_{downloaded,extraced,patched} bits.  Can someone
suggest a more elegant way of doing this?

..I removed patching support, since you should probably be using
some other tool to manage patches in the external tree anyways.

Comments appreciated
Comment 1 Will Wagner 2011-03-07 18:55:05 UTC
Martin,

I have used this a bit and it seems fine. One addition I have hacked in is an option to allow the output files to reside in the buildroot output folder using the O= option to kernel make.

This allows me to have a directory containing the kernel source which does not get modified by a build and then the build files go into $(O)/build/linux-custom. If you are interested I could probably clean up my patch and submit.
Comment 2 Martin Hicks 2011-03-07 19:18:27 UTC
> 
> I have used this a bit and it seems fine. One addition I have hacked in is an
> option to allow the output files to reside in the buildroot output folder using
> the O= option to kernel make.

Yeah, good idea.  If you send me a patch I'll wrap it up as one patch for Peter and repost to bugzilla.

Thanks,
mh
Comment 3 Thomas Petazzoni 2011-09-18 09:49:16 UTC
This bug will be fixed by the for-2011.11/pkg-infra branch which I have posted to the list.
Comment 4 Peter Korsgaard 2011-10-09 20:45:11 UTC
Fixed in git by Thomas/David (2690e76aa2)