<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Oct 29, 2024 at 8:35 AM Luca Vizzarro <<a href="mailto:Luca.Vizzarro@arm.com">Luca.Vizzarro@arm.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 29/10/2024 01:23, Patrick Robb wrote:<br>
> On Mon, Oct 21, 2024 at 9:50 AM Luca Vizzarro <<a href="mailto:luca.vizzarro@arm.com" target="_blank">luca.vizzarro@arm.com</a> <br>
> <mailto:<a href="mailto:luca.vizzarro@arm.com" target="_blank">luca.vizzarro@arm.com</a>>> wrote:<br>
> <br>
> <br>
>     +<br>
>     +   DPDK Build Options:<br>
>     +   Arguments in this group (and subgroup) will be applied to a<br>
>     DPDKLocation when the DPDK tree, tarball or revision will be<br>
>     provided, other arguments<br>
>     +   like remote source and build dir are optional. A DPDKLocation<br>
>     from settings are used instead of from config if construct successful.<br>
> <br>
> <br>
> Maybe call this "DPDK Source Options?" To me, "DPDK Build Options" <br>
> invokes meson options supported in DPDK. Up to you though.<br>
> Reviewed-by: Patrick Robb <<a href="mailto:probb@iol.unh.edu" target="_blank">probb@iol.unh.edu</a> <mailto:<a href="mailto:probb@iol.unh.edu" target="_blank">probb@iol.unh.edu</a>>><br>
<br>
Perfectly understandable concern to be honest. But it's also somewhat <br>
complicated, given this refers to the build whether it's uncompiled or <br>
precompiled and the location of the sources. It may require some thought.<br></blockquote><div><br></div><div>"DPDK Build Source" instead of "DPDK Build Options"? </div><div><br></div><div>I don't know why but for some reason "Build Options" strongly translates to meson options in my mind. </div><div><br></div><div>But if it's a headache I think we can drop it - everything is documented in any case so it shouldn't be a big deal.</div></div></div>