* [PATCH] remove .tmp_gas_check @ 2005-02-06 12:50 Olaf Hering 2005-02-06 16:54 ` Tom Rini 0 siblings, 1 reply; 13+ messages in thread From: Olaf Hering @ 2005-02-06 12:50 UTC (permalink / raw) To: Tom Rini, linuxppc-dev make ARCH=ppc tags leaves a .tmp_gas_check file behind. Just remove it after the as test. Signed-off-by: Olaf Hering <olh@suse.de> diff -purNx tags ../linux-2.6.11-rc3.orig/arch/ppc/Makefile ./arch/ppc/Makefile --- ../linux-2.6.11-rc3.orig/arch/ppc/Makefile 2005-02-03 02:57:05.000000000 +0100 +++ ./arch/ppc/Makefile 2005-02-06 10:51:21.000000000 +0100 @@ -112,10 +112,10 @@ include/asm-$(ARCH)/offsets.h: arch/$(AR TOUT := .tmp_gas_check # Ensure this is binutils 2.12.1 (or 2.12.90.0.7) or later for altivec # instructions. -AS_ALTIVEC := $(shell echo dssall | $(AS) -many -o $(TOUT) >/dev/null 2>&1 ; echo $$?) +AS_ALTIVEC := $(shell echo dssall | $(AS) -many -o $(TOUT) >/dev/null 2>&1 ; echo $$? ; rm -f $(TOUT)) # gcc-3.4 and binutils-2.14 are a fatal combination. GCC_VERSION := $(call cc-version) -BAD_GCC_AS := $(shell echo mftb 5 | $(AS) -mppc -many -o $(TOUT) >/dev/null 2>&1 && echo 0 || echo 1) +BAD_GCC_AS := $(shell echo mftb 5 | $(AS) -mppc -many -o $(TOUT) >/dev/null 2>&1 ; echo $$? ; rm -f $(TOUT)) checkbin: ifeq ($(GCC_VERSION)$(BAD_GCC_AS),03041) ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] remove .tmp_gas_check 2005-02-06 12:50 [PATCH] remove .tmp_gas_check Olaf Hering @ 2005-02-06 16:54 ` Tom Rini 2005-02-06 16:56 ` Olaf Hering 0 siblings, 1 reply; 13+ messages in thread From: Tom Rini @ 2005-02-06 16:54 UTC (permalink / raw) To: Olaf Hering; +Cc: linuxppc-dev On Sun, Feb 06, 2005 at 01:50:56PM +0100, Olaf Hering wrote: > make ARCH=ppc tags leaves a .tmp_gas_check file behind. > Just remove it after the as test. > > Signed-off-by: Olaf Hering <olh@suse.de> What is the problem exactly? .tmp_* should be removed in a normal 'clean' target ? -- Tom Rini http://gate.crashing.org/~trini/ ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] remove .tmp_gas_check 2005-02-06 16:54 ` Tom Rini @ 2005-02-06 16:56 ` Olaf Hering 2005-02-06 17:24 ` Tom Rini 0 siblings, 1 reply; 13+ messages in thread From: Olaf Hering @ 2005-02-06 16:56 UTC (permalink / raw) To: Tom Rini; +Cc: linuxppc-dev On Sun, Feb 06, Tom Rini wrote: > On Sun, Feb 06, 2005 at 01:50:56PM +0100, Olaf Hering wrote: > > > make ARCH=ppc tags leaves a .tmp_gas_check file behind. > > Just remove it after the as test. > > > > Signed-off-by: Olaf Hering <olh@suse.de> > > What is the problem exactly? .tmp_* should be removed in a normal > 'clean' target ? make tags should only create the toplevel tags file, nothing else. ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] remove .tmp_gas_check 2005-02-06 16:56 ` Olaf Hering @ 2005-02-06 17:24 ` Tom Rini 2005-02-06 17:25 ` Olaf Hering 0 siblings, 1 reply; 13+ messages in thread From: Tom Rini @ 2005-02-06 17:24 UTC (permalink / raw) To: Olaf Hering; +Cc: linuxppc-dev On Sun, Feb 06, 2005 at 05:56:29PM +0100, Olaf Hering wrote: > On Sun, Feb 06, Tom Rini wrote: > > > On Sun, Feb 06, 2005 at 01:50:56PM +0100, Olaf Hering wrote: > > > > > make ARCH=ppc tags leaves a .tmp_gas_check file behind. > > > Just remove it after the as test. > > > > > > Signed-off-by: Olaf Hering <olh@suse.de> > > > > What is the problem exactly? .tmp_* should be removed in a normal > > 'clean' target ? > > make tags should only create the toplevel tags file, nothing else. My first guess then is there's a dependancy problem, as make tags should just invoke ctags or so, yes? -- Tom Rini http://gate.crashing.org/~trini/ ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] remove .tmp_gas_check 2005-02-06 17:24 ` Tom Rini @ 2005-02-06 17:25 ` Olaf Hering 2005-02-06 20:04 ` Tom Rini 0 siblings, 1 reply; 13+ messages in thread From: Olaf Hering @ 2005-02-06 17:25 UTC (permalink / raw) To: Tom Rini; +Cc: linuxppc-dev On Sun, Feb 06, Tom Rini wrote: > My first guess then is there's a dependancy problem, as make tags should > just invoke ctags or so, yes? The variable is evaluated, even if it is not used. Just run 'make ARCH=ppc tags' and see yourself. ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] remove .tmp_gas_check 2005-02-06 17:25 ` Olaf Hering @ 2005-02-06 20:04 ` Tom Rini 2005-02-06 20:08 ` Olaf Hering 0 siblings, 1 reply; 13+ messages in thread From: Tom Rini @ 2005-02-06 20:04 UTC (permalink / raw) To: Olaf Hering; +Cc: linuxppc-dev On Sun, Feb 06, 2005 at 06:25:52PM +0100, Olaf Hering wrote: > On Sun, Feb 06, Tom Rini wrote: > > > My first guess then is there's a dependancy problem, as make tags should > > just invoke ctags or so, yes? > > The variable is evaluated, even if it is not used. > Just run 'make ARCH=ppc tags' and see yourself. Right. But we only care about that if we're going to compile anything, so perhaps we need to change the test somehow to only get evaluated when needed (ala the -mcpu stuff? Or is that always run as well?). -- Tom Rini http://gate.crashing.org/~trini/ ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] remove .tmp_gas_check 2005-02-06 20:04 ` Tom Rini @ 2005-02-06 20:08 ` Olaf Hering 2005-02-06 20:21 ` Tom Rini 0 siblings, 1 reply; 13+ messages in thread From: Olaf Hering @ 2005-02-06 20:08 UTC (permalink / raw) To: Tom Rini; +Cc: linuxppc-dev On Sun, Feb 06, Tom Rini wrote: > Right. But we only care about that if we're going to compile anything, > so perhaps we need to change the test somehow to only get evaluated when > needed (ala the -mcpu stuff? Or is that always run as well?). I dont want to see that file because its just a tmpfile. mkdir $foo make ARCH=ppc tags & make ARCH=ppc O=$foo -j menuconfig make ARCH=ppc O=$foo -j all Its just in the way when doing diffs. and make clean may remove the tags file. But its just silly to expect 'make tags; make clean' to get rid of it. ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] remove .tmp_gas_check 2005-02-06 20:08 ` Olaf Hering @ 2005-02-06 20:21 ` Tom Rini 2005-02-06 21:15 ` Olaf Hering 0 siblings, 1 reply; 13+ messages in thread From: Tom Rini @ 2005-02-06 20:21 UTC (permalink / raw) To: Olaf Hering; +Cc: linuxppc-dev On Sun, Feb 06, 2005 at 09:08:02PM +0100, Olaf Hering wrote: > On Sun, Feb 06, Tom Rini wrote: > > > Right. But we only care about that if we're going to compile anything, > > so perhaps we need to change the test somehow to only get evaluated when > > needed (ala the -mcpu stuff? Or is that always run as well?). > > I dont want to see that file because its just a tmpfile. > mkdir $foo > make ARCH=ppc tags & > make ARCH=ppc O=$foo -j menuconfig > make ARCH=ppc O=$foo -j all > > Its just in the way when doing diffs. and make clean may remove the tags > file. But its just silly to expect 'make tags; make clean' to get rid of it. I agree you shouldn't have to see it. I'm saying the problem is the variable shouldn't be evaluated. -- Tom Rini http://gate.crashing.org/~trini/ ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] remove .tmp_gas_check 2005-02-06 20:21 ` Tom Rini @ 2005-02-06 21:15 ` Olaf Hering 2005-02-06 21:49 ` Tom Rini 0 siblings, 1 reply; 13+ messages in thread From: Olaf Hering @ 2005-02-06 21:15 UTC (permalink / raw) To: Tom Rini; +Cc: linuxppc-dev On Sun, Feb 06, Tom Rini wrote: > I agree you shouldn't have to see it. I'm saying the problem is the > variable shouldn't be evaluated. This patch seems to work for me. diff -purNx tags ../linux-2.6.11-rc3.orig/arch/ppc/Makefile ./arch/ppc/Makefile --- ../linux-2.6.11-rc3.orig/arch/ppc/Makefile 2005-02-03 02:57:05.000000000 +0100 +++ ./arch/ppc/Makefile 2005-02-06 22:13:01.093237836 +0100 @@ -112,26 +112,27 @@ include/asm-$(ARCH)/offsets.h: arch/$(AR TOUT := .tmp_gas_check # Ensure this is binutils 2.12.1 (or 2.12.90.0.7) or later for altivec # instructions. -AS_ALTIVEC := $(shell echo dssall | $(AS) -many -o $(TOUT) >/dev/null 2>&1 ; echo $$?) # gcc-3.4 and binutils-2.14 are a fatal combination. GCC_VERSION := $(call cc-version) -BAD_GCC_AS := $(shell echo mftb 5 | $(AS) -mppc -many -o $(TOUT) >/dev/null 2>&1 && echo 0 || echo 1) checkbin: -ifeq ($(GCC_VERSION)$(BAD_GCC_AS),03041) - @echo -n '*** ${VERSION}.${PATCHLEVEL} kernels no longer build ' - @echo 'correctly with gcc-3.4 and your version of binutils.' - @echo '*** Please upgrade your binutils or downgrade your gcc' - @false -endif -ifneq ($(AS_ALTIVEC),0) - echo $(AS_ALTIVEC) - @echo -n '*** ${VERSION}.${PATCHLEVEL} kernels no longer build ' - @echo 'correctly with old versions of binutils.' - @echo '*** Please upgrade your binutils to 2.12.1 or newer' - @false -endif - @true + @if test "$(GCC_VERSION)" = "0304" ; then \ + if ! /bin/echo mftb 5 | $(AS) -v -mppc -many -o $(TOUT) >/dev/null 2>&1 ; then \ + echo -n '*** ${VERSION}.${PATCHLEVEL} kernels no longer build '; \ + echo 'correctly with gcc-3.4 and your version of binutils.'; \ + echo '*** Please upgrade your binutils or downgrade your gcc'; \ + rm -f $(TOUT) ;\ + false; \ + fi ; \ + fi + @if ! /bin/echo dssall | $(AS) -many -o $(TOUT) >/dev/null 2>&1 ; then \ + echo -n '*** ${VERSION}.${PATCHLEVEL} kernels no longer build ' ; \ + echo 'correctly with old versions of binutils.' ; \ + echo '*** Please upgrade your binutils to 2.12.1 or newer' ; \ + rm -f $(TOUT) ; \ + false ; \ + fi + @rm -f $(TOUT) CLEAN_FILES += include/asm-$(ARCH)/offsets.h \ arch/$(ARCH)/kernel/asm-offsets.s \ ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] remove .tmp_gas_check 2005-02-06 21:15 ` Olaf Hering @ 2005-02-06 21:49 ` Tom Rini 2005-02-06 21:56 ` Olaf Hering 0 siblings, 1 reply; 13+ messages in thread From: Tom Rini @ 2005-02-06 21:49 UTC (permalink / raw) To: Olaf Hering; +Cc: linuxppc-dev On Sun, Feb 06, 2005 at 10:15:06PM +0100, Olaf Hering wrote: > On Sun, Feb 06, Tom Rini wrote: > > > I agree you shouldn't have to see it. I'm saying the problem is the > > variable shouldn't be evaluated. > > This patch seems to work for me. What I don't like is that checkbin shouldn't be evaluated for 'tags' or other non-compile targets, so you don't need those explicit rm's. Also, you forgot the signed-off-by line :) -- Tom Rini http://gate.crashing.org/~trini/ ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] remove .tmp_gas_check 2005-02-06 21:49 ` Tom Rini @ 2005-02-06 21:56 ` Olaf Hering 2005-02-06 22:43 ` Tom Rini 0 siblings, 1 reply; 13+ messages in thread From: Olaf Hering @ 2005-02-06 21:56 UTC (permalink / raw) To: Tom Rini; +Cc: linuxppc-dev On Sun, Feb 06, Tom Rini wrote: > On Sun, Feb 06, 2005 at 10:15:06PM +0100, Olaf Hering wrote: > > > On Sun, Feb 06, Tom Rini wrote: > > > > > I agree you shouldn't have to see it. I'm saying the problem is the > > > variable shouldn't be evaluated. > > > > This patch seems to work for me. > > What I don't like is that checkbin shouldn't be evaluated for 'tags' or > other non-compile targets, so you don't need those explicit rm's. Also, > you forgot the signed-off-by line :) Why should make tags or oldconfig ever care about the installed toolchain? How should the tmpfile disappear if one calls make checkbin with success? Maybe I misunderstood you. ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] remove .tmp_gas_check 2005-02-06 21:56 ` Olaf Hering @ 2005-02-06 22:43 ` Tom Rini 2005-02-07 8:23 ` Olaf Hering 0 siblings, 1 reply; 13+ messages in thread From: Tom Rini @ 2005-02-06 22:43 UTC (permalink / raw) To: Olaf Hering; +Cc: linuxppc-dev On Sun, Feb 06, 2005 at 10:56:00PM +0100, Olaf Hering wrote: > On Sun, Feb 06, Tom Rini wrote: > > > On Sun, Feb 06, 2005 at 10:15:06PM +0100, Olaf Hering wrote: > > > > > On Sun, Feb 06, Tom Rini wrote: > > > > > > > I agree you shouldn't have to see it. I'm saying the problem is the > > > > variable shouldn't be evaluated. > > > > > > This patch seems to work for me. > > > > What I don't like is that checkbin shouldn't be evaluated for 'tags' or > > other non-compile targets, so you don't need those explicit rm's. Also, > > you forgot the signed-off-by line :) > > Why should make tags or oldconfig ever care about the installed toolchain? It shouldn't, and 'tags' at least shouldn't depend on 'checkbin' (oldconfig I _believe_ actually invokes HOSTCC and creates other .tmp files, but I don't have a tree handy). > How should the tmpfile disappear if one calls make checkbin with success? On make clean, like the other .tmp files that are around at the end of a kernel compile (kallsyms/modules/maybe something else). > Maybe I misunderstood you. I think so. We both agree there should be no ppc-specific .tmp files around after certain make targets. I'm saying that there's always some .tmp files generated on every arch for certain targets, and removed with a clean. PPC should only create .tmp files (And leave them alone) in the cases where every arch generates some .tmp files. -- Tom Rini http://gate.crashing.org/~trini/ ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] remove .tmp_gas_check 2005-02-06 22:43 ` Tom Rini @ 2005-02-07 8:23 ` Olaf Hering 0 siblings, 0 replies; 13+ messages in thread From: Olaf Hering @ 2005-02-07 8:23 UTC (permalink / raw) To: Tom Rini; +Cc: linuxppc-dev On Sun, Feb 06, Tom Rini wrote: > On Sun, Feb 06, 2005 at 10:56:00PM +0100, Olaf Hering wrote: > > On Sun, Feb 06, Tom Rini wrote: > > > > > On Sun, Feb 06, 2005 at 10:15:06PM +0100, Olaf Hering wrote: > > > > > > > On Sun, Feb 06, Tom Rini wrote: > > > > > > > > > I agree you shouldn't have to see it. I'm saying the problem is the > > > > > variable shouldn't be evaluated. > > > > > > > > This patch seems to work for me. > > > > > > What I don't like is that checkbin shouldn't be evaluated for 'tags' or > > > other non-compile targets, so you don't need those explicit rm's. Also, > > > you forgot the signed-off-by line :) > > > > Why should make tags or oldconfig ever care about the installed toolchain? > > It shouldn't, and 'tags' at least shouldn't depend on 'checkbin' tags does not call checkbin, so this part is ok. > > How should the tmpfile disappear if one calls make checkbin with success? > > On make clean, like the other .tmp files that are around at the end of a > kernel compile (kallsyms/modules/maybe something else). Thats ok now. > > Maybe I misunderstood you. > > I think so. We both agree there should be no ppc-specific .tmp files > around after certain make targets. I'm saying that there's always some > .tmp files generated on every arch for certain targets, and removed with > a clean. PPC should only create .tmp files (And leave them alone) in > the cases where every arch generates some .tmp files. Signed-off-by: Olaf Hering <olh@suse.de> diff -purNx tags ../linux-2.6.11-rc3.orig/arch/ppc/Makefile ./arch/ppc/Makefile --- ../linux-2.6.11-rc3.orig/arch/ppc/Makefile 2005-02-03 02:57:05.000000000 +0100 +++ ./arch/ppc/Makefile 2005-02-07 09:21:25.941904406 +0100 @@ -112,26 +112,24 @@ include/asm-$(ARCH)/offsets.h: arch/$(AR TOUT := .tmp_gas_check # Ensure this is binutils 2.12.1 (or 2.12.90.0.7) or later for altivec # instructions. -AS_ALTIVEC := $(shell echo dssall | $(AS) -many -o $(TOUT) >/dev/null 2>&1 ; echo $$?) # gcc-3.4 and binutils-2.14 are a fatal combination. GCC_VERSION := $(call cc-version) -BAD_GCC_AS := $(shell echo mftb 5 | $(AS) -mppc -many -o $(TOUT) >/dev/null 2>&1 && echo 0 || echo 1) checkbin: -ifeq ($(GCC_VERSION)$(BAD_GCC_AS),03041) - @echo -n '*** ${VERSION}.${PATCHLEVEL} kernels no longer build ' - @echo 'correctly with gcc-3.4 and your version of binutils.' - @echo '*** Please upgrade your binutils or downgrade your gcc' - @false -endif -ifneq ($(AS_ALTIVEC),0) - echo $(AS_ALTIVEC) - @echo -n '*** ${VERSION}.${PATCHLEVEL} kernels no longer build ' - @echo 'correctly with old versions of binutils.' - @echo '*** Please upgrade your binutils to 2.12.1 or newer' - @false -endif - @true + @if test "$(GCC_VERSION)" = "0304" ; then \ + if ! /bin/echo mftb 5 | $(AS) -v -mppc -many -o $(TOUT) >/dev/null 2>&1 ; then \ + echo -n '*** ${VERSION}.${PATCHLEVEL} kernels no longer build '; \ + echo 'correctly with gcc-3.4 and your version of binutils.'; \ + echo '*** Please upgrade your binutils or downgrade your gcc'; \ + false; \ + fi ; \ + fi + @if ! /bin/echo dssall | $(AS) -many -o $(TOUT) >/dev/null 2>&1 ; then \ + echo -n '*** ${VERSION}.${PATCHLEVEL} kernels no longer build ' ; \ + echo 'correctly with old versions of binutils.' ; \ + echo '*** Please upgrade your binutils to 2.12.1 or newer' ; \ + false ; \ + fi CLEAN_FILES += include/asm-$(ARCH)/offsets.h \ arch/$(ARCH)/kernel/asm-offsets.s \ ^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2005-02-07 8:24 UTC | newest] Thread overview: 13+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2005-02-06 12:50 [PATCH] remove .tmp_gas_check Olaf Hering 2005-02-06 16:54 ` Tom Rini 2005-02-06 16:56 ` Olaf Hering 2005-02-06 17:24 ` Tom Rini 2005-02-06 17:25 ` Olaf Hering 2005-02-06 20:04 ` Tom Rini 2005-02-06 20:08 ` Olaf Hering 2005-02-06 20:21 ` Tom Rini 2005-02-06 21:15 ` Olaf Hering 2005-02-06 21:49 ` Tom Rini 2005-02-06 21:56 ` Olaf Hering 2005-02-06 22:43 ` Tom Rini 2005-02-07 8:23 ` Olaf Hering
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).