forked from mirrors/linux
		
	kbuild: fix infinite make recursion
Jan Engelhardt <jengelh@computergmbh.de> reported: You can cause a recursion in kbuild/make with the following: make O=$PWD kernel/time.o make mrproper Of course no one would use O=$PWD (that's just the testcase), but this happened too often: /ws/linux/linux-2.6.23$ make O=/ws/linux/linux-2.6.23 kernel/time.o (Oops - should have been O=/ws/linux/obj-2.6.23!) Fixed by an explicit test for this case - we error out if output directory and source directory are the same. Tested-by: Jan Engelhardt <jengelh@computergmbh.de> Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
This commit is contained in:
		
							parent
							
								
									a67cb1319f
								
							
						
					
					
						commit
						4f1127e204
					
				
					 1 changed files with 3 additions and 1 deletions
				
			
		
							
								
								
									
										4
									
								
								Makefile
									
									
									
									
									
								
							
							
						
						
									
										4
									
								
								Makefile
									
									
									
									
									
								
							| 
						 | 
					@ -115,7 +115,9 @@ saved-output := $(KBUILD_OUTPUT)
 | 
				
			||||||
KBUILD_OUTPUT := $(shell cd $(KBUILD_OUTPUT) && /bin/pwd)
 | 
					KBUILD_OUTPUT := $(shell cd $(KBUILD_OUTPUT) && /bin/pwd)
 | 
				
			||||||
$(if $(KBUILD_OUTPUT),, \
 | 
					$(if $(KBUILD_OUTPUT),, \
 | 
				
			||||||
     $(error output directory "$(saved-output)" does not exist))
 | 
					     $(error output directory "$(saved-output)" does not exist))
 | 
				
			||||||
 | 
					# Check that OUTPUT directory is not the same as where we have kernel src
 | 
				
			||||||
 | 
					$(if $(filter-out $(KBUILD_OUTPUT),$(shell /bin/pwd)),, \
 | 
				
			||||||
 | 
					     $(error Output directory (O=...) specifies kernel src dir))
 | 
				
			||||||
PHONY += $(MAKECMDGOALS)
 | 
					PHONY += $(MAKECMDGOALS)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
$(filter-out _all,$(MAKECMDGOALS)) _all:
 | 
					$(filter-out _all,$(MAKECMDGOALS)) _all:
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
		Reference in a new issue