Hi,
On Thu, Jan 18, 2018 at 3:22 PM, Emil Velikov <emil.l.veli...@gmail.com> wrote:
> On 18 January 2018 at 11:33, <maxin.j...@gmail.com> wrote:
>> From: "Maxin B. John" <maxin.j...@intel.com>
>>
>> Sort the output to ensure build reproducibility
>>
>> Signed-off-by: Maxin B. John <maxin.j...@intel.com>
>> ---
>> 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 <emil.veli...@collabora.com>
> Fixes: 0ab04ba979b ("anv: Use python to generate ICD json files")
>
> -Emil
Gentle ping !
Regards,
Maxin
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
