mirror of
				https://github.com/godotengine/godot.git
				synced 2025-10-31 13:41:03 +00:00 
			
		
		
		
	SCons: Add method to detect Emscripten and use it for warnings config
Emscripten is LLVM-based so we want to follow the same logic. But we can't just put it as a match in `methods.using_clang()` as that would mess with the compiler version detection logic used to restrict old GCC and Clang releases. (cherry picked from commit34421683eb) (cherry picked from commite7f7d5f330)
This commit is contained in:
		
							parent
							
								
									e78a8b2424
								
							
						
					
					
						commit
						2ac5a8dd2e
					
				
					 2 changed files with 5 additions and 1 deletions
				
			
		|  | @ -428,7 +428,7 @@ if selected_platform in platform_list: | |||
|             common_warnings += ["-Wno-misleading-indentation"] | ||||
|             if version[0] >= 7: | ||||
|                 common_warnings += ["-Wshadow-local"] | ||||
|         elif methods.using_clang(env): | ||||
|         elif methods.using_clang(env) or methods.using_emcc(env): | ||||
|             # We often implement `operator<` for structs of pointers as a requirement | ||||
|             # for putting them in `Set` or `Map`. We don't mind about unreliable ordering. | ||||
|             common_warnings += ["-Wno-ordered-compare-function-pointers"] | ||||
|  |  | |||
|  | @ -842,6 +842,10 @@ def using_clang(env): | |||
|     return "clang" in os.path.basename(env["CC"]) | ||||
| 
 | ||||
| 
 | ||||
| def using_emcc(env): | ||||
|     return "emcc" in os.path.basename(env["CC"]) | ||||
| 
 | ||||
| 
 | ||||
| def show_progress(env): | ||||
|     import sys | ||||
|     from SCons.Script import Progress, Command, AlwaysBuild | ||||
|  |  | |||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue
	
	 Rémi Verschelde
						Rémi Verschelde