diff options
author | Sam Ravnborg <sam@ravnborg.org> | 2009-10-11 23:22:58 +0200 |
---|---|---|
committer | Sam Ravnborg <sam@ravnborg.org> | 2009-10-11 23:22:58 +0200 |
commit | 2331d1a6cd3d6e580bc88b9a160066d9e1177fe1 (patch) | |
tree | eba044f60a1fe478e864ead1dc1940a2b0a23b9f /firmware/edgeport/boot2.H16 | |
parent | c01226c3145d173a0d38f9d5b4f229cc23d99ae2 (diff) | |
download | lwn-2331d1a6cd3d6e580bc88b9a160066d9e1177fe1.tar.gz lwn-2331d1a6cd3d6e580bc88b9a160066d9e1177fe1.zip |
kbuild: revert "save ARCH & CROSS_COMPILE ..."
Revert commit 575543347b5baed0ca927cb90ba8807396fe9cc9
It caused following issues:
- On architectures where ARCH= setting is used to select between
32 and 64 bit this was no longer possible without "make mrproper"
- If ARCH was changed then kbuild refused to run "make mrproper"
because ARCH had changed
- When CROSS_COMPILE was changed people were asked to run "make mrproper"
but kbuild refused to run "make mrproper" because CROSS_COMPILE changed.
- Spaces in CROSS_COMPILE was not 'supported'
- If an non-existing ARCH= was used kbuild could get stuck
Lessons learned:
. Despite being simple and straghtforward people uses very different
approaches when building the kernel.
. CROSS_COMPILE is sometimes used for ccache despite cache being
only a CC frontend so one would have expected CC to be
used for this purpose.
. And obviously this was not tested widely enough.
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Cc: Pavel Machek <pavel@ucw.cz>
Cc: Ingo Molnar <mingo@elte.hu>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Andrew Morton <akpm@linux-foundation.org>
Diffstat (limited to 'firmware/edgeport/boot2.H16')
0 files changed, 0 insertions, 0 deletions