Hi Felix, On Thu, 2015-07-30 at 14:12 +0300, Alexey Brodkin wrote: > If "findstring" is used without leading and trailing spaces unexpected matches > may happen. For example consider ARC=arc then "findstring $(ARCH)" will > report a false match with "aarch64". > > But "findstring $ARCH " (note trailing space) will correctly skip > matches for both "aarch64" and "aarch64_be". > > This patch is built-tested against NetGear WNDR3800. > > Signed-off-by: Alexey Brodkin <[email protected]> > Cc: Felix Fietkau <[email protected]> > Cc: Jo-Philipp Wich <[email protected]> > --- > > In v2 removed unncecessary space before findstring. > > include/kernel.mk | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/include/kernel.mk b/include/kernel.mk > index 7a0a170..6a613fe 100644 > --- a/include/kernel.mk > +++ b/include/kernel.mk > @@ -62,15 +62,15 @@ endif > > ifneq (,$(findstring uml,$(BOARD))) > LINUX_KARCH=um > -else ifneq (,$(findstring $(ARCH), aarch64 aarch64_be)) > +else ifneq (,$(findstring $(ARCH) , aarch64 aarch64_be )) > LINUX_KARCH := arm64 > -else ifneq (,$(findstring $(ARCH), armeb)) > +else ifneq (,$(findstring $(ARCH) , armeb )) > LINUX_KARCH := arm > -else ifneq (,$(findstring $(ARCH), mipsel mips64 mips64el)) > +else ifneq (,$(findstring $(ARCH) , mipsel mips64 mips64el )) > LINUX_KARCH := mips > -else ifneq (,$(findstring $(ARCH), sh2 sh3 sh4)) > +else ifneq (,$(findstring $(ARCH) , sh2 sh3 sh4 )) > LINUX_KARCH := sh > -else ifneq (,$(findstring $(ARCH), i386 x86_64)) > +else ifneq (,$(findstring $(ARCH) , i386 x86_64 )) > LINUX_KARCH := x86 > else > LINUX_KARCH := $(ARCH)
I'm wondering if there're any comments on this one. Otherwise please consider applying. This patch is a prerequisite for ARC port submission I'm going to send out shortly. -Alexey _______________________________________________ openwrt-devel mailing list [email protected] https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel
