mirror of
https://gitlab.freedesktop.org/mesa/mesa.git
synced 2026-01-02 00:50:07 +01:00
meson: fix race condition revealed by using 0.44
Previously there was a special target that blocked for the generation of
anv_entrypoints.h, with meson 0.44 we don't need this, we can use a new
language feature instead. The problem is that previously that blocking
target would hide a race condition for the generation of another header,
anv_extensions.h. Now the build sometimes fails when anv_extensions.h is
not generated in time.
v2: - clarify the race condition in the commit message (Emil)
CC: Mark Janes <mark.a.janes@intel.com>
Fixes: 92550d9b16
("meson: remove workaround for custom target creating .h and .c files")
Signed-off-by: Dylan Baker <dylan.c.baker@intel.com>
Reviewed-by: Scott D Phillips <scott.d.phillips@intel.com>
Reviewed-by: Emil Velikov <emil.velikov@collabora.com>
This commit is contained in:
parent
0c23bd76d1
commit
7772de5283
1 changed files with 3 additions and 3 deletions
|
|
@ -1,4 +1,4 @@
|
|||
# Copyright © 2017 Intel Corporation
|
||||
# Copyright © 2017-2018 Intel Corporation
|
||||
|
||||
# Permission is hereby granted, free of charge, to any person obtaining a copy
|
||||
# of this software and associated documentation files (the "Software"), to deal
|
||||
|
|
@ -96,7 +96,7 @@ foreach g : [['70', ['gen7_cmd_buffer.c']], ['75', ['gen7_cmd_buffer.c']],
|
|||
_gen = g[0]
|
||||
libanv_gen_libs += static_library(
|
||||
'libanv_gen@0@'.format(_gen),
|
||||
[anv_gen_files, g[1], anv_entrypoints[0]],
|
||||
[anv_gen_files, g[1], anv_entrypoints[0], anv_extensions_h],
|
||||
include_directories : [
|
||||
inc_common, inc_compiler, inc_drm_uapi, inc_intel, inc_vulkan_util,
|
||||
inc_vulkan_wsi,
|
||||
|
|
@ -220,7 +220,7 @@ if with_tests
|
|||
'anv_@0@'.format(t),
|
||||
executable(
|
||||
t,
|
||||
['tests/@0@.c'.format(t), anv_entrypoints[0]],
|
||||
['tests/@0@.c'.format(t), anv_entrypoints[0], anv_extensions_h],
|
||||
link_with : libvulkan_intel_test,
|
||||
dependencies : [dep_libdrm, dep_thread, dep_m, dep_valgrind],
|
||||
include_directories : [
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue