>> --- Comment #3 from Bernhard <[hidden email]> 2009-03-10 16:14:23 UTC ---
Thanks much for your comments, Bernhard.
I've attached a new extended patch. I made it more self-explanatory
and consistent by adding binutils/gcc/gdb Makefiles' updates related
to our toolchain/patch-kernel.sh changes.
>> First, there are superfluous trailing ';' chars in your patch.
>> Does this:
>> + uncomp="tar cf - --exclude=.svn --no-anchored -C";
>> work with busybox's tar?
Given that busybox tar is 1.18, I say "Yes". It works.
I was not able to build my busybox TAR package because of the build errors.
But, I checked it on my x86-linux box after installing GNU/TAR 1.18.
>> What is "uparm" meant to be? Please use a more descriptive name.
It's been replaced by "uncomp_parm" in the new patch.
>> Last, can you give some examples that show why you need these changes?
Applying Xtensa source overlays could be done without changing patch-kernel.sh,
but they would result in much more extensive changes to other Makefiles.
This change to patch-kernel.sh is the simplest and most elegant way
we found of applying source overlays. It simply expands patch-kernel.sh to
handle tar files, and thus patch not just files, but also source trees.
Isn't it nice ? Can other BUILDROOT users find it useful ?
An example might be Xtensa overlay support for GDB. With this change
to patch-kernel.sh, all we needed to edit in toolchain/gdb/gdb.mk was
to add a conditional include:
ifneq ($(filter xtensa%,$(ARCH)),)
GDB_PATCH_EXTRA:=$(call XTENSA_PATCH,gdb,$(GDB_PATCH_DIR),. ..)
and add $(GDB_PATCH_EXTRA) to the list of what patch-kernel.sh patches: