mirror of
				https://github.com/torvalds/linux.git
				synced 2025-11-04 02:30:34 +02:00 
			
		
		
		
	kbuild: remove redundant $(wildcard ...) for cmd_files calculation
I do not see any reason why $(wildcard ...) needs to be called twice for computing cmd_files. Remove the first one. Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
This commit is contained in:
		
							parent
							
								
									9a234a2e38
								
							
						
					
					
						commit
						2982c95357
					
				
					 4 changed files with 4 additions and 8 deletions
				
			
		
							
								
								
									
										3
									
								
								Makefile
									
									
									
									
									
								
							
							
						
						
									
										3
									
								
								Makefile
									
									
									
									
									
								
							| 
						 | 
				
			
			@ -1693,8 +1693,7 @@ cmd_crmodverdir = $(Q)mkdir -p $(MODVERDIR) \
 | 
			
		|||
 | 
			
		||||
# read all saved command lines
 | 
			
		||||
 | 
			
		||||
targets := $(wildcard $(sort $(targets)))
 | 
			
		||||
cmd_files := $(wildcard .*.cmd $(foreach f,$(targets),$(dir $(f)).$(notdir $(f)).cmd))
 | 
			
		||||
cmd_files := $(wildcard .*.cmd $(foreach f,$(sort $(targets)),$(dir $(f)).$(notdir $(f)).cmd))
 | 
			
		||||
 | 
			
		||||
ifneq ($(cmd_files),)
 | 
			
		||||
  $(cmd_files): ;	# Do not try to update included dependency files
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -583,8 +583,7 @@ FORCE:
 | 
			
		|||
# optimization, we don't need to read them if the target does not
 | 
			
		||||
# exist, we will rebuild anyway in that case.
 | 
			
		||||
 | 
			
		||||
targets := $(wildcard $(sort $(targets)))
 | 
			
		||||
cmd_files := $(wildcard $(foreach f,$(targets),$(dir $(f)).$(notdir $(f)).cmd))
 | 
			
		||||
cmd_files := $(wildcard $(foreach f,$(sort $(targets)),$(dir $(f)).$(notdir $(f)).cmd))
 | 
			
		||||
 | 
			
		||||
ifneq ($(cmd_files),)
 | 
			
		||||
  include $(cmd_files)
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -114,9 +114,8 @@ $(check-file): scripts/headers_check.pl $(output-files) FORCE
 | 
			
		|||
 | 
			
		||||
endif
 | 
			
		||||
 | 
			
		||||
targets := $(wildcard $(sort $(targets)))
 | 
			
		||||
cmd_files := $(wildcard \
 | 
			
		||||
             $(foreach f,$(targets),$(dir $(f)).$(notdir $(f)).cmd))
 | 
			
		||||
             $(foreach f,$(sort $(targets)),$(dir $(f)).$(notdir $(f)).cmd))
 | 
			
		||||
 | 
			
		||||
ifneq ($(cmd_files),)
 | 
			
		||||
	include $(cmd_files)
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -143,8 +143,7 @@ FORCE:
 | 
			
		|||
# optimization, we don't need to read them if the target does not
 | 
			
		||||
# exist, we will rebuild anyway in that case.
 | 
			
		||||
 | 
			
		||||
targets := $(wildcard $(sort $(targets)))
 | 
			
		||||
cmd_files := $(wildcard $(foreach f,$(targets),$(dir $(f)).$(notdir $(f)).cmd))
 | 
			
		||||
cmd_files := $(wildcard $(foreach f,$(sort $(targets)),$(dir $(f)).$(notdir $(f)).cmd))
 | 
			
		||||
 | 
			
		||||
ifneq ($(cmd_files),)
 | 
			
		||||
  include $(cmd_files)
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in a new issue