On 4/30/25 11:50 AM, Richard Henderson wrote:
On 4/30/25 07:58, Pierrick Bouvier wrote:
+arm_common_system_ss.add(files('cpu.c'), capstone)
I wonder if we should inherit these dependencies from common_ss or system_ss?
I ran into the same issue when working on hw/arm [1], and didn't really
look for a way to do this from main meson.build.
I thought that it's a good thing to document those kind of dependencies
next to the sources who depend on them, so I didn't dig too much.
That said, I'm not opposed to it if you feel your approach is better.
[1] hw/arm/meson.build
-arm_ss.add(files('boot.c'))
+arm_common_ss.add(fdt, files('boot.c'))
-arm_ss.add(when: 'CONFIG_MUSICPAL', if_true: files('musicpal.c'))
+arm_common_ss.add(when: 'CONFIG_MUSICPAL', if_true: [pixman,
files('musicpal.c')])
r~
