Re: [Git Patch] Makefile: fix wrong dirs when making cscope
On Mon, Nov 05, 2007 at 08:33:17AM +0100, Sam Ravnborg wrote: >Hi Wang. > >Thanks for this fix, but I have a few comments. See below. > >On Mon, Nov 05, 2007 at 03:09:53PM +0800, WANG Cong wrote: >> >> Hi, Sam! >> >> This patch fixed the following errors when doing "make cscope" and >> "make cscope ARCH=um". >> >> FILELST cscope.files >> find: arch/i386: No such file or directory >> MAKEcscope.out >> >> >> FILELST cscope.files >> find: include/asm-i386: No such file or directory >> MAKEcscope.out >> >> >> Signed-off-by: WANG Cong <[EMAIL PROTECTED]> >> Cc: Sam Ravnborg <[EMAIL PROTECTED]> >> >> --- >> Makefile |4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> Index: linux-2.6/Makefile >> === >> --- linux-2.6.orig/Makefile >> +++ linux-2.6/Makefile >> @@ -1322,7 +1322,7 @@ ALLSOURCE_ARCHS := $(ARCH) $(SRCARCH) >> endif >> >> define find-sources >> -( for arch in $(ALLSOURCE_ARCHS) ; do \ >> +( for arch in `echo $(ALLSOURCE_ARCHS)|sed -e "s/i386/x86/"`; do \ >> find $(__srctree)arch/$${arch} $(RCS_FIND_IGNORE) \ >> -name $1 -print; \ >>done ; \ > >Could you change this such that the substitution takes places where we >assign ALLSOURCE_ARCHS so all potential users benefit from this fix. >And on top of this fix it so x86_64 is also replaced by x86 so we fix >both x86 architectures. OK. Thank you. I will try to do that. ;) > > >> @@ -1331,7 +1331,7 @@ define find-sources >>find $(__srctree)include $(RCS_FIND_IGNORE) \ >> \( -name config -o -name 'asm-*' \) -prune \ >> -o -name $1 -print; \ >> - for arch in $(ALLINCLUDE_ARCHS) ; do \ >> + for arch in `echo $(ALLINCLUDE_ARCHS)|sed -e "s/i386/x86/"`; do \ >> find $(__srctree)include/asm-$${arch} $(RCS_FIND_IGNORE) \ >> -name $1 -print; \ >>done ; \ >> >Same comments for ALLINCLUDE_ARCHS > >PS. Yout patch may be obsoleted by ongoing work to eliminate >ARCH=i386 / ARCH=x86_64. But if/when this hits mainline I dunno so >please do the requested changes and send me a new patch. Sam, the root of this problem is the use of `uname -m' command. It always outputs 'i*86' on x86_32 machines. Unless this output changes or we find another way to determine the arch, we will always need to fix this. New patch will come soon. Thanks. WANG Cong - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [Git Patch] Makefile: fix wrong dirs when making cscope
Hi Wang. Thanks for this fix, but I have a few comments. See below. On Mon, Nov 05, 2007 at 03:09:53PM +0800, WANG Cong wrote: > > Hi, Sam! > > This patch fixed the following errors when doing "make cscope" and > "make cscope ARCH=um". > > FILELST cscope.files > find: arch/i386: No such file or directory > MAKEcscope.out > > > FILELST cscope.files > find: include/asm-i386: No such file or directory > MAKEcscope.out > > > Signed-off-by: WANG Cong <[EMAIL PROTECTED]> > Cc: Sam Ravnborg <[EMAIL PROTECTED]> > > --- > Makefile |4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > Index: linux-2.6/Makefile > === > --- linux-2.6.orig/Makefile > +++ linux-2.6/Makefile > @@ -1322,7 +1322,7 @@ ALLSOURCE_ARCHS := $(ARCH) $(SRCARCH) > endif > > define find-sources > -( for arch in $(ALLSOURCE_ARCHS) ; do \ > +( for arch in `echo $(ALLSOURCE_ARCHS)|sed -e "s/i386/x86/"`; do \ > find $(__srctree)arch/$${arch} $(RCS_FIND_IGNORE) \ > -name $1 -print; \ > done ; \ Could you change this such that the substitution takes places where we assign ALLSOURCE_ARCHS so all potential users benefit from this fix. And on top of this fix it so x86_64 is also replaced by x86 so we fix both x86 architectures. > @@ -1331,7 +1331,7 @@ define find-sources > find $(__srctree)include $(RCS_FIND_IGNORE) \ > \( -name config -o -name 'asm-*' \) -prune \ > -o -name $1 -print; \ > - for arch in $(ALLINCLUDE_ARCHS) ; do \ > + for arch in `echo $(ALLINCLUDE_ARCHS)|sed -e "s/i386/x86/"`; do \ > find $(__srctree)include/asm-$${arch} $(RCS_FIND_IGNORE) \ > -name $1 -print; \ > done ; \ > Same comments for ALLINCLUDE_ARCHS PS. Yout patch may be obsoleted by ongoing work to eliminate ARCH=i386 / ARCH=x86_64. But if/when this hits mainline I dunno so please do the requested changes and send me a new patch. Tanks, Sam - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[Git Patch] Makefile: fix wrong dirs when making cscope
Hi, Sam! This patch fixed the following errors when doing "make cscope" and "make cscope ARCH=um". FILELST cscope.files find: arch/i386: No such file or directory MAKEcscope.out FILELST cscope.files find: include/asm-i386: No such file or directory MAKEcscope.out Signed-off-by: WANG Cong <[EMAIL PROTECTED]> Cc: Sam Ravnborg <[EMAIL PROTECTED]> --- Makefile |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) Index: linux-2.6/Makefile === --- linux-2.6.orig/Makefile +++ linux-2.6/Makefile @@ -1322,7 +1322,7 @@ ALLSOURCE_ARCHS := $(ARCH) $(SRCARCH) endif define find-sources -( for arch in $(ALLSOURCE_ARCHS) ; do \ +( for arch in `echo $(ALLSOURCE_ARCHS)|sed -e "s/i386/x86/"`; do \ find $(__srctree)arch/$${arch} $(RCS_FIND_IGNORE) \ -name $1 -print; \ done ; \ @@ -1331,7 +1331,7 @@ define find-sources find $(__srctree)include $(RCS_FIND_IGNORE) \ \( -name config -o -name 'asm-*' \) -prune \ -o -name $1 -print; \ - for arch in $(ALLINCLUDE_ARCHS) ; do \ + for arch in `echo $(ALLINCLUDE_ARCHS)|sed -e "s/i386/x86/"`; do \ find $(__srctree)include/asm-$${arch} $(RCS_FIND_IGNORE) \ -name $1 -print; \ done ; \ - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[Git Patch] Makefile: fix wrong dirs when making cscope
Hi, Sam! This patch fixed the following errors when doing make cscope and make cscope ARCH=um. FILELST cscope.files find: arch/i386: No such file or directory MAKEcscope.out FILELST cscope.files find: include/asm-i386: No such file or directory MAKEcscope.out Signed-off-by: WANG Cong [EMAIL PROTECTED] Cc: Sam Ravnborg [EMAIL PROTECTED] --- Makefile |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) Index: linux-2.6/Makefile === --- linux-2.6.orig/Makefile +++ linux-2.6/Makefile @@ -1322,7 +1322,7 @@ ALLSOURCE_ARCHS := $(ARCH) $(SRCARCH) endif define find-sources -( for arch in $(ALLSOURCE_ARCHS) ; do \ +( for arch in `echo $(ALLSOURCE_ARCHS)|sed -e s/i386/x86/`; do \ find $(__srctree)arch/$${arch} $(RCS_FIND_IGNORE) \ -name $1 -print; \ done ; \ @@ -1331,7 +1331,7 @@ define find-sources find $(__srctree)include $(RCS_FIND_IGNORE) \ \( -name config -o -name 'asm-*' \) -prune \ -o -name $1 -print; \ - for arch in $(ALLINCLUDE_ARCHS) ; do \ + for arch in `echo $(ALLINCLUDE_ARCHS)|sed -e s/i386/x86/`; do \ find $(__srctree)include/asm-$${arch} $(RCS_FIND_IGNORE) \ -name $1 -print; \ done ; \ - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [Git Patch] Makefile: fix wrong dirs when making cscope
Hi Wang. Thanks for this fix, but I have a few comments. See below. On Mon, Nov 05, 2007 at 03:09:53PM +0800, WANG Cong wrote: Hi, Sam! This patch fixed the following errors when doing make cscope and make cscope ARCH=um. FILELST cscope.files find: arch/i386: No such file or directory MAKEcscope.out FILELST cscope.files find: include/asm-i386: No such file or directory MAKEcscope.out Signed-off-by: WANG Cong [EMAIL PROTECTED] Cc: Sam Ravnborg [EMAIL PROTECTED] --- Makefile |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) Index: linux-2.6/Makefile === --- linux-2.6.orig/Makefile +++ linux-2.6/Makefile @@ -1322,7 +1322,7 @@ ALLSOURCE_ARCHS := $(ARCH) $(SRCARCH) endif define find-sources -( for arch in $(ALLSOURCE_ARCHS) ; do \ +( for arch in `echo $(ALLSOURCE_ARCHS)|sed -e s/i386/x86/`; do \ find $(__srctree)arch/$${arch} $(RCS_FIND_IGNORE) \ -name $1 -print; \ done ; \ Could you change this such that the substitution takes places where we assign ALLSOURCE_ARCHS so all potential users benefit from this fix. And on top of this fix it so x86_64 is also replaced by x86 so we fix both x86 architectures. @@ -1331,7 +1331,7 @@ define find-sources find $(__srctree)include $(RCS_FIND_IGNORE) \ \( -name config -o -name 'asm-*' \) -prune \ -o -name $1 -print; \ - for arch in $(ALLINCLUDE_ARCHS) ; do \ + for arch in `echo $(ALLINCLUDE_ARCHS)|sed -e s/i386/x86/`; do \ find $(__srctree)include/asm-$${arch} $(RCS_FIND_IGNORE) \ -name $1 -print; \ done ; \ Same comments for ALLINCLUDE_ARCHS PS. Yout patch may be obsoleted by ongoing work to eliminate ARCH=i386 / ARCH=x86_64. But if/when this hits mainline I dunno so please do the requested changes and send me a new patch. Tanks, Sam - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [Git Patch] Makefile: fix wrong dirs when making cscope
On Mon, Nov 05, 2007 at 08:33:17AM +0100, Sam Ravnborg wrote: Hi Wang. Thanks for this fix, but I have a few comments. See below. On Mon, Nov 05, 2007 at 03:09:53PM +0800, WANG Cong wrote: Hi, Sam! This patch fixed the following errors when doing make cscope and make cscope ARCH=um. FILELST cscope.files find: arch/i386: No such file or directory MAKEcscope.out FILELST cscope.files find: include/asm-i386: No such file or directory MAKEcscope.out Signed-off-by: WANG Cong [EMAIL PROTECTED] Cc: Sam Ravnborg [EMAIL PROTECTED] --- Makefile |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) Index: linux-2.6/Makefile === --- linux-2.6.orig/Makefile +++ linux-2.6/Makefile @@ -1322,7 +1322,7 @@ ALLSOURCE_ARCHS := $(ARCH) $(SRCARCH) endif define find-sources -( for arch in $(ALLSOURCE_ARCHS) ; do \ +( for arch in `echo $(ALLSOURCE_ARCHS)|sed -e s/i386/x86/`; do \ find $(__srctree)arch/$${arch} $(RCS_FIND_IGNORE) \ -name $1 -print; \ done ; \ Could you change this such that the substitution takes places where we assign ALLSOURCE_ARCHS so all potential users benefit from this fix. And on top of this fix it so x86_64 is also replaced by x86 so we fix both x86 architectures. OK. Thank you. I will try to do that. ;) @@ -1331,7 +1331,7 @@ define find-sources find $(__srctree)include $(RCS_FIND_IGNORE) \ \( -name config -o -name 'asm-*' \) -prune \ -o -name $1 -print; \ - for arch in $(ALLINCLUDE_ARCHS) ; do \ + for arch in `echo $(ALLINCLUDE_ARCHS)|sed -e s/i386/x86/`; do \ find $(__srctree)include/asm-$${arch} $(RCS_FIND_IGNORE) \ -name $1 -print; \ done ; \ Same comments for ALLINCLUDE_ARCHS PS. Yout patch may be obsoleted by ongoing work to eliminate ARCH=i386 / ARCH=x86_64. But if/when this hits mainline I dunno so please do the requested changes and send me a new patch. Sam, the root of this problem is the use of `uname -m' command. It always outputs 'i*86' on x86_32 machines. Unless this output changes or we find another way to determine the arch, we will always need to fix this. New patch will come soon. Thanks. WANG Cong - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/