[Mesa-dev] [PATCH 3/4] anv: Use central api generation scripts.

[Dylan Baker]

Quoting Bas Nieuwenhuizen (2018-08-07 16:14:33)                                                                                                                                                                                                                                       
>                                                                                                                                                                                                                                                                                     
>  anv_extensions_c = custom_target(                                                                                                                                                                                                                                                  
> @@ -36,10 +37,11 @@ anv_extensions_c = custom_target(                                                                                                                                                                                                                               
>    input : ['anv_extensions_gen.py', vk_api_xml],                                                                                                                                                                                                                                   
>    output : 'anv_extensions.c',                                                                                                                                                                                                                                                     
>    command : [                                                                                                                                                                                                                                                                      
> +   'env', 'PYTHONPATH=@0@'.format(join_paths(meson.source_root(), 'src/vulkan/util/')),                                                                                                                                                                                            

This is really gross, you're adding a dependency on a unix console command. I                                                                                                                                                                                                         
know that anv is only built on Unix-like oses, but this will eventually end up                                                                                                                                                                                                        
being used in some code that needs to run on Windows (or mac, does mac have                                                                                                                                                                                                           
env?).                                                                                                                                                                                                                                                                                

I know that some people will object, but IMHO a better solution than mucking                                                                                                                                                                                                          
with the python path (either through sys.path or through PYTHONPATH, is to                                                                                                                                                                                                            
put all of the generators in a src/generators directory and be done with it.                                                                                                                                                                                                          
Sure the intel specific bits (for example) aren't in the src/intel folder, 
that's a small price to avoid having to call env just to run a python script.

Dylan

P.S: Sorry if this goes through twice, I've been having trouble with my mail
setup.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: signature
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20180807/ef440b9b/attachment.sig>