On 3/8/21 5:46 PM, Stefan Hajnoczi wrote: > On Wed, Feb 24, 2021 at 10:35:15PM +0100, Philippe Mathieu-Daudé wrote: >> We don't need to generate trace files for targets we >> are not building. Restrict the the ones selected. >> >> Signed-off-by: Philippe Mathieu-Daudé <phi...@redhat.com> >> --- >> meson.build | 16 ++++++++-------- >> 1 file changed, 8 insertions(+), 8 deletions(-) >> >> diff --git a/meson.build b/meson.build >> index 05a67c20d93..8503aa1b628 100644 >> --- a/meson.build >> +++ b/meson.build >> @@ -1826,16 +1826,16 @@ >> trace_events_subdirs += [ >> 'accel/tcg', >> 'hw/core', >> - 'target/arm', >> - 'target/hppa', >> - 'target/i386', >> 'target/i386/kvm', > > Please add a comment explaining why an exception is needed here and the > rest of the target directories are handled by the loop below when you > send a non-RFC patch.
Better would be a recursive function that adds the directory if necessary. But I'm happy adding a simple comment =) > > That will help people who add sub-directories in the future. Yes. Anyhow I expect this to clash with Claudio's work in progress (which split target/ and add multiple subdirectories).