freebsd / atf

Libraries to write tests in C, C++ and shell
Other
127 stars 44 forks source link

Missing required programs on some of ATFs own tests #31

Closed brd closed 6 years ago

brd commented 7 years ago

In atf/libatf-c++/Kyuafile, the following tests should have:

atf_test_program{name="atf_c++_test", required_programs="c++"} atf_test_program{name="check_test", required_programs="cc"} atf_test_program{name="macros_test", required_programs="c++"}

Same in atf/libatf-c/Kyuafile:

atf_test_program{name="atf_c_test", required_programs="cc"} atf_test_program{name="check_test", required_programs="cc"} atf_test_program{name="macros_test", required_programs="cc"}

Thanks

brd commented 7 years ago

Ping?

jmmv commented 6 years ago

It's not that simple unfortunately, because the compiler can be parameterized both at build time and at run time. I've just pushed a possible fix to the issue-31 branch and if it comes back as clean, I'll merge it.