Re: [RFC] [PATCH] linux-2.5.49_subarch-cleanup_A2

john stultz (johnstul@us.ibm.com)
26 Nov 2002 12:16:09 -0800


On Tue, 2002-11-26 at 11:54, Sam Ravnborg wrote:
> One detail..
> Move the "mcore-y := mach-default" assignment above the
> block that deal with subarch. Then you do not need the "ifndef ..."

Ah, good suggestion! Thanks! I'll apply and resubmit later today.

> I recall that Linus previously have asked for shell commands
> when moving files - so consider including only the patch below and
> then a number of "mv arch/i386/X include/....".
> This makes it much more visible what you actually change.
>
> Another alternative it to do it in bitkeeper, then it is visible
> from the cset that you move files (use bk mv).

Hmm. I was under the impression that shell command and sed script
patches were only for those w/ enough kernel penguin points to blow.
Although, please someone correct me if I'm wrong. Regardless BK should
notice the renames in the diff and import them as such, so it shouldn't
make much difference to Linus.

Thanks again for the feedback!

-john

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