[libcamera-devel,v4,6/6] tests: Remove IPA_MODULE_PATH environment variable
diff mbox series

Message ID 20200221163130.4791-7-kieran.bingham@ideasonboard.com
State Accepted
Headers show
Series
  • Support loading IPAs from the build tree
Related show

Commit Message

Kieran Bingham Feb. 21, 2020, 4:31 p.m. UTC 
The tests declare a hard-coded LIBCAMERA_IPA_MODULE_PATH to allow tests
to run from the test-suite.

This requires tests to be run only from the root of the build directory,
otherwise (for example, by running in their local directory) they will
not be able to correctly locate the IPA modules.

Now that the build path for the IPA manager is determined at runtime we
can remove the redundant setting of the LIBCAMERA_IPA_MODULE_PATH for
tests.

Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
---
 test/libtest/test.cpp | 4 ----
 1 file changed, 4 deletions(-)

Patch
diff mbox series 

diff --git a/test/libtest/test.cpp b/test/libtest/test.cpp
index 333d2160e276..6cd3fbe6df06 100644
--- a/test/libtest/test.cpp
+++ b/test/libtest/test.cpp
@@ -21,10 +21,6 @@  int Test::execute()
 {
 	int ret;
 
-	ret = setenv("LIBCAMERA_IPA_MODULE_PATH", "src/ipa", 1);
-	if (ret)
-		return errno;
-
 	ret = setenv("LIBCAMERA_IPA_PROXY_PATH", "src/libcamera/proxy/worker", 1);
 	if (ret)
 		return errno;