[Buildroot] svn commit: trunk/buildroot/target/u-boot

[Bernhard Fischer <rep.dot.nop@gmail.com>]

On Wed, Apr 23, 2008 at 07:39:38PM +0200, Ulf Samuelsson wrote:
>
>
>> On Wed, Apr 23, 2008 at 06:22:46PM +0200, Ulf Samuelsson wrote:
>>>
>>>> Hmmm I have never seen patch.$ARCH in buildroot.
>>>> 
>>>> It's usually -$ARCH.patch like toolchain/kernel-headers
>>>> package/qtopia4 and others.
>>>
>>>
>>>If you patch with "*.patch",  then *all* architecture dependent patches will be applied
>>>if you use "-$ARCH.patch"
>>>
>>>You want to apply 
>>>* architecture independent patches
>>>* architecture dependent patches pertaining to your architecture.
>>>
>>>so ".patch.$ARCH" is much cleaner
>> 
>> The concept of per-arch patches is broken per se. You're better off to
>> fix or drop the dysfunctional patches.
>
>
>The per-arch patch is low maintenance, while your idea is high maintenance.

We already had that argument, so let me just repeat that you have to
fixup your port anyway, sooner or later. 'nuff said