Make a distinction between example apps and app/ apps by renaming the
build app function. Moreover, provide a dedicated helper function to
retrieve the path to a DPDK app.
Signed-off-by: Luca Vizzarro <luca.vizzarro@arm.com>
Reviewed-by: Paul Szczepanek <paul.szczepanek@arm.com>
---
dts/framework/remote_session/dpdk.py | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/dts/framework/remote_session/dpdk.py b/dts/framework/remote_session/dpdk.py
index e00c4a9bf9..ed1f808bdd 100644
--- a/dts/framework/remote_session/dpdk.py
+++ b/dts/framework/remote_session/dpdk.py
@@ -267,8 +267,8 @@ def _build_dpdk(self) -> None:
self.remote_dpdk_build_dir,
)
- def build_dpdk_app(self, app_name: str, **meson_dpdk_args: str | bool) -> PurePath:
- """Build one or all DPDK apps.
+ def build_dpdk_example_app(self, app_name: str, **meson_dpdk_args: str | bool) -> PurePath:
+ """Build one or all DPDK example apps.
We have previously been encouraged to avoid using sample apps unless absolutely necessary, so it probably doesn't make sense for us to add this in until we have a specific reason and justification for supporting example apps (and this may never happen).
Requires DPDK to be already built on the SUT node.
@@ -298,6 +298,10 @@ def build_dpdk_app(self, app_name: str, **meson_dpdk_args: str | bool) -> PurePa
self.remote_dpdk_build_dir, "examples", f"dpdk-{app_name}"
)
+ def get_app(self, app_name: str) -> PurePath:
+ """Retrieve path for a DPDK app."""
+ return self._session.join_remote_path(self.remote_dpdk_build_dir, "app", f"dpdk-{app_name}")
+
@cached_property
def remote_dpdk_tree_path(self) -> PurePath:
"""The remote DPDK tree path."""
--
2.43.0
Otherwise this looks fine for dumpcap and the others /app apps - thanks.