<div dir="ltr"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-im" style="color:rgb(80,0,80)">On Wed, Mar 15, 2023 at 4:52 PM Patrick Robb <<a href="mailto:probb@iol.unh.edu" target="_blank">probb@iol.unh.edu</a>> wrote:<br><br></span>Are you sure about the default value?<br><br>Afaics, meson selects by default a "release" buildtype (and I think I<br>always saw this value in the past).<br>I have this with meson 1.0.<br><br>$ meson setup qsdlgfh<br>...<br>$ meson configure qsdlgfh | grep buildtype<br>  buildtype                      release<br>[plain, debug, debugoptimized,   Build type to use<span class="gmail-im" style="color:rgb(80,0,80)"><br><br></span></blockquote><div> </div><div>Thanks, you're right - I missed this in the meson.build file. So the default is release, which is what we run at the lab since we never pass a custom buildtype option in at meson setup.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-im" style="color:rgb(80,0,80)">><br>> The reason I'm writing this email is that I'm wondering whether the buildtype decision made by those who wrote .ci/linux-build.sh for GHA was intentional and important. I know many of the people who have commits on that script follow this mailing list. Obviously if it's in some way more appropriate for CI purposes to run meson setup in this way, I'm happy to make that change at the lab and in the process that would free up bringing Alpine compile testing online. But, if not, then I think it's most appropriate to consider compile on Alpine as broken and avoid bringing coverage for Alpine online until that issue is resolved.<br><br></span>The reason why we went with debugoptimized was primarly for the ABI<br>checks, as by default, the debug symbols were missing (which would<br>match with a "release" default buildtype).<br>See 777014e56d07 ("devtools: add ABI checks").<br></blockquote><div><br></div><div>Gotcha. Thanks that makes sense. Well, I guess what I had (unknowingly) done initially with our Alpine container at the lab was a release build, but I'll do a release build on GHA to sanity check again anyways, which will presumably fail. </div><div><br></div><div>So, I guess the conclusion is if it doesn't build with the default buildtype set by meson.build, it's broken and we will not bring Alpine compile coverage online right now. </div><div><br></div><div>Thanks, let me know if you think I'm missing anything here! -Patrick </div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Mar 15, 2023 at 12:08 PM David Marchand <<a href="mailto:david.marchand@redhat.com">david.marchand@redhat.com</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 Wed, Mar 15, 2023 at 4:52 PM Patrick Robb <<a href="mailto:probb@iol.unh.edu" target="_blank">probb@iol.unh.edu</a>> wrote:<br>
><br>
> Hello all,<br>
><br>
> The lab recently received a request to re-enable Alpine compile jobs, which have been disabled for almost a year. In dry running the compile job, I noticed that it was failing. At the same time, David Marchand did an Alpine compile with Github Actions which was successful. It seems the source of the different behavior is the meson buildtype being used - the build script used by GHA sets meson buildtype to debugoptimized, whereas the script used by the community lab runs with buildtype debug (the meson default). I did my own Github Actions runs (with both buildtype options) to sanity check: <a href="https://github.com/PatrickRobbIOL/dpdk/actions/runs/4427160204/jobs/7764368640" rel="noreferrer" target="_blank">https://github.com/PatrickRobbIOL/dpdk/actions/runs/4427160204/jobs/7764368640</a><br>
<br>
Are you sure about the default value?<br>
<br>
Afaics, meson selects by default a "release" buildtype (and I think I<br>
always saw this value in the past).<br>
I have this with meson 1.0.<br>
<br>
$ meson setup qsdlgfh<br>
...<br>
$ meson configure qsdlgfh | grep buildtype<br>
  buildtype                      release<br>
[plain, debug, debugoptimized,   Build type to use<br>
<br>
><br>
> The reason I'm writing this email is that I'm wondering whether the buildtype decision made by those who wrote .ci/linux-build.sh for GHA was intentional and important. I know many of the people who have commits on that script follow this mailing list. Obviously if it's in some way more appropriate for CI purposes to run meson setup in this way, I'm happy to make that change at the lab and in the process that would free up bringing Alpine compile testing online. But, if not, then I think it's most appropriate to consider compile on Alpine as broken and avoid bringing coverage for Alpine online until that issue is resolved.<br>
<br>
The reason why we went with debugoptimized was primarly for the ABI<br>
checks, as by default, the debug symbols were missing (which would<br>
match with a "release" default buildtype).<br>
See 777014e56d07 ("devtools: add ABI checks").<br>
<br>
<br>
-- <br>
David Marchand<br>
<br>
</blockquote></div><br clear="all"><div><br></div><span class="gmail_signature_prefix">-- </span><br><div dir="ltr" class="gmail_signature"><div dir="ltr"><p dir="ltr" style="line-height:1.2;margin-top:0pt;margin-bottom:0pt"><font color="#000000" face="Arial"><span style="font-size:13.3333px;white-space:pre-wrap">Patrick Robb</span></font></p><p style="color:rgb(34,34,34);line-height:1.2;margin-top:0pt;margin-bottom:0pt"><span style="font-size:10pt;font-family:Arial;color:rgb(0,0,0);background-color:transparent;vertical-align:baseline;white-space:pre-wrap">Technical Service Manager</span></p><p dir="ltr" style="color:rgb(34,34,34);line-height:1.2;margin-top:0pt;margin-bottom:0pt"><span style="font-size:10pt;font-family:Arial;color:rgb(0,0,0);background-color:transparent;vertical-align:baseline;white-space:pre-wrap">UNH InterOperability Laboratory</span></p><p dir="ltr" style="color:rgb(34,34,34);line-height:1.2;margin-top:0pt;margin-bottom:0pt"><span style="font-size:10pt;font-family:Arial;color:rgb(0,0,0);background-color:transparent;vertical-align:baseline;white-space:pre-wrap">21 Madbury Rd, Suite 100, Durham, NH 03824</span></p><p dir="ltr" style="color:rgb(34,34,34);line-height:1.2;margin-top:0pt;margin-bottom:0pt"><span style="font-size:10pt;font-family:Arial;color:rgb(17,85,204);background-color:transparent;vertical-align:baseline;white-space:pre-wrap"><a href="http://www.iol.unh.edu/" style="color:rgb(17,85,204)" target="_blank">www.iol.unh.edu</a></span></p><p dir="ltr" style="color:rgb(34,34,34);line-height:1.2;margin-top:0pt;margin-bottom:0pt"><br></p><p dir="ltr" style="color:rgb(34,34,34);line-height:1.2;margin-top:0pt;margin-bottom:0pt"><span style="font-size:10pt;font-family:Arial;color:rgb(51,51,51);background-color:transparent;vertical-align:baseline;white-space:pre-wrap"><img src="https://lh4.googleusercontent.com/7sTY8VswXadak_YT0J13osh5ockNVRX2BuYaRsKoTTpkpilBokA0WlocYHLB4q7XUgXNHka6-ns47S8R_am0sOt7MYQQ1ILQ3S-P5aezsrjp3-IsJMmMrErHWmTARNgZhpAx06n2" width="150" height="37" style="border: none;"></span></p></div></div>