Re: [PATCH] meson: Don't build tests when CLang lacks -fsemantic-interposition

On Tue, Mar 21, 2023 at 05:47:19PM +0100, Michal Privoznik wrote: So at first I wanted to suggest something like: if cc.get_id() == 'clang' and \ not supported_cc_flags.contains('-fsemantic-interposition') \ and get_option('optimization') != '0' # If CLang doesn't support -fsemantic-interposition then our # mocking doesn't work. The best we can do is to not run the # test suite. if get_option('tests').enabled() error('Cannot enable tests because CLang lacks -fsemantic-interposition. Update CLang or disable optimization') else build_tests = false endif else build_tests = not get_option('tests').disabled() endif Which would simply error out if you want the tests to be enabled on such system. However that has the downside that some potential contributor might not notice the tests are being disabled and we would also have to have another place where to track the enablement of tests/optimization, in our CI. It is possible, but after some tries I think your approach is better and the main thing is that we need something to be pushed in order for us not to get into failing CI fatigue which I've seen in other projects where CI failures are just being dismissed after couple of issues. So: Reviewed-by: Martin Kletzander <mkletzan(a)redhat.com&gt;