On 18 January 2018 at 11:33,  <[email protected]> wrote:
> From: "Maxin B. John" <[email protected]>
>
> Sort the output to ensure build reproducibility
>
> Signed-off-by: Maxin B. John <[email protected]>
> ---
>  src/intel/vulkan/anv_icd.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/intel/vulkan/anv_icd.py b/src/intel/vulkan/anv_icd.py
> index 4ed01fa..31bb068 100644
> --- a/src/intel/vulkan/anv_icd.py
> +++ b/src/intel/vulkan/anv_icd.py
> @@ -44,4 +44,4 @@ if __name__ == '__main__':
>      }
>
>      with open(args.out, 'w') as f:
> -        json.dump(json_data, f, indent = 4)
> +        json.dump(json_data, f, indent = 4, sort_keys=True)

I'm slightly confused what exactly causes the issue.
Please shed some light so we can add it to the commit message.

Regardless, the patch seems correct and is

Reviewed-by: Emil Velikov <[email protected]>
Fixes: 0ab04ba979b ("anv: Use python to generate ICD json files")

-Emil
_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to