[libcamera-devel] [PATCH LIBCAMERA v3 2/6] libcamera: ipa_proxy: rearrange proxies precedence
Kieran Bingham
kieran.bingham at ideasonboard.com
Wed Mar 18 13:16:44 CET 2020
Hi Kaaira,
On 18/03/2020 11:58, Kaaira Gupta wrote:
> User environment path in LIBCAMERA_IPA_PROXY_PATH should take precedence
> over system loading locations. Change precedence accordingly
>
Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>
> Signed-off-by: Kaaira Gupta <kgupta at es.iitr.ac.in>
> ---
> src/libcamera/ipa_proxy.cpp | 11 ++++++-----
> 1 file changed, 6 insertions(+), 5 deletions(-)
>
> diff --git a/src/libcamera/ipa_proxy.cpp b/src/libcamera/ipa_proxy.cpp
> index c7347c8..b409e1d 100644
> --- a/src/libcamera/ipa_proxy.cpp
> +++ b/src/libcamera/ipa_proxy.cpp
> @@ -75,13 +75,9 @@ IPAProxy::~IPAProxy()
> */
> std::string IPAProxy::resolvePath(const std::string &file) const
> {
> - /* Try finding the exec target from the install directory first */
> std::string proxyFile = "/" + file;
> - std::string proxyPath = std::string(IPA_PROXY_DIR) + proxyFile;
> - if (!access(proxyPath.c_str(), X_OK))
> - return proxyPath;
>
> - /* No exec target in install directory; check env variable. */
> + /* Check env variable first. */
> const char *execPaths = utils::secure_getenv("LIBCAMERA_IPA_PROXY_PATH");
> if (execPaths) {
> for (const auto &dir : utils::split(execPaths, ":")) {
> @@ -95,6 +91,11 @@ std::string IPAProxy::resolvePath(const std::string &file) const
> }
> }
>
> + /* Try finding the exec target from the install directory. */
> + std::string proxyPath = std::string(IPA_PROXY_DIR) + proxyFile;
> + if (!access(proxyPath.c_str(), X_OK))
> + return proxyPath;
> +
> return std::string();
> }
>
>
--
Regards
--
Kieran
More information about the libcamera-devel
mailing list