<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Apr 30, 2024 at 4:13 PM Mattias Rönnblom <<a href="mailto:hofors@lysator.liu.se">hofors@lysator.liu.se</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 2024-04-30 15:52, Patrick Robb wrote:<br>
> <br>
> <br>
> On Sun, Apr 28, 2024 at 3:46 AM Mattias Rönnblom <<a href="mailto:hofors@lysator.liu.se" target="_blank">hofors@lysator.liu.se</a> <br>
> <mailto:<a href="mailto:hofors@lysator.liu.se" target="_blank">hofors@lysator.liu.se</a>>> wrote:<br>
> <br>
>     It would be great if the unit test suite (app/test/*) was compiled (and<br>
>     run) using a C++ (C++11) compiler as well. At least, if such is<br>
>     available.<br>
> <br>
> <br>
> Sure, the UNH Lab can try this.<br>
> <br>
> <br>
>     With the current state of affairs, header file macros or functions are<br>
>     not verified to be functional (or even valid) C++.<br>
> <br>
>     "C is a subset of C++", which was never true, is becoming less and<br>
>     less so.<br>
> <br>
>     If all unit tests aren't valid C++, maybe one could start with an "opt<br>
>     in" model.<br>
> <br>
> <br>
> Okay, so basically run the fast-test suite, record all that don't pass, <br>
> submit a bugzilla ticket stating which unit tests are not valid on a <br>
> certain c++ compiler, then bring CI Testing online using the valid <br>
> subset of fast-tests. This should work.<br>
> <br>
<br>
Sounds good.<br>
<br>
Just to be clear: the above includes extending the DPDK build system to <br>
build the app/test/dpdk-test binary in two versions: one C and one C++, <br>
so that anyone can run the C++ tests locally as well. Correct?<br></blockquote><div><br></div><div>Okay, so now I am understanding this is not yet available. When I responded this morning I was figuring that c++ compiler support was available and I simply wasn't aware, and that we could quite easily set cc={some c++ compiler}, meson would pick it up, and we would be able to build DPDK and then run unit tests in this manner in CI testing. </div><div><br></div><div>I didn't mean to suggest we would submit patches extending the build system to this end. That's probably a little out of scope for what we try to accomplish at the Community Lab. </div><div><br></div><div>But if the aforementioned build system support is added, of course we are willing to add that as a build environment for unit tests and report those respective results.</div><div> </div></div></div>