[libcamera-devel] [PATCH v3 6/6] tests: Remove IPA_MODULE_PATH environment variable
Kieran Bingham
kieran.bingham at ideasonboard.com
Fri Feb 21 13:40:12 CET 2020
On 20/02/2020 21:26, Laurent Pinchart wrote:
> Hi Kieran,
>
> Thank you for the patch.
>
> On Thu, Feb 20, 2020 at 04:57:04PM +0000, Kieran Bingham wrote:
>> 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.
>
> \o/
>
> Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
>
>> Signed-off-by: Kieran Bingham <kieran.bingham at ideasonboard.com>
>> ---
>> test/libtest/test.cpp | 4 ----
>> 1 file changed, 4 deletions(-)
>>
>> 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;
>
> Will we need to do the same for the proxy path ? :-)
Yes, but I haven't touched that yet until we can get the 'method' done
right.
I had considered leaving this as a task for any new comer or workshop
too ...
--
Regards
--
Kieran
More information about the libcamera-devel
mailing list