[libcamera-devel,v3,0/6] libcamera: determine IPA_PROXY_PATH at runtime
mbox series

Message ID 20200318115846.7975-1-kgupta@es.iitr.ac.in
Headers show
Series
  • libcamera: determine IPA_PROXY_PATH at runtime
Related show

Message

Kaaira Gupta March 18, 2020, 11:58 a.m. UTC
This is a series of patches to determine IPA_PROXY_PATH at runtime,
removing hard-coded LIBCAMERA_IPA_PROXY_PATH from test.cpp and making
isLibcamerainstalled() and libcameraPath() utils' functions.

Changes since v2:
Patch 1: No changes.
Patch 2: No changes.
Patch 3: Declared functions in utils.h and declared _DYNAMIC extern
outside namespace libcamera
Patch 4: No changes.
Patch 5: No changes.
Patch 6: No changes.

Changes since v1:
Renumbered the patches to account for two additional patches as
described below.

Patch 1: Fixed whitespace problems.
Patch 2: Fixed whitespace problems.
Patch 3: NEW PATCH: Adds isLibcamerainstalled() and libcameraPath()
as global functions in utils module.
Patch 4: Earlier patch 3: fixed styling issues and used helper functions
of utils instead of defining own functions.
Patch 5: NEW PATCH: Replace isLibcamerainstalled() and libcameraPath()
by helper functions defined in utils in ipa_manager
Patch 6: Earlier patch 4: Fixed odd tab in comment description.

Kaaira Gupta (6):
  libcamera: ipa_proxy: use utils::split()
  libcamera: ipa_proxy: rearrange proxies precedence
  libcamera: utils: add Libcamera installed & path
  libcamera: ipa_proxy: search for proxy in build tree
  libcamera: ipa_manager: replace by utils functions
  tests: remove IPA_PROXY_PATH environment variable

 src/libcamera/include/utils.h |  4 +++
 src/libcamera/ipa_manager.cpp | 36 ++------------------------
 src/libcamera/ipa_proxy.cpp   | 37 +++++++++++++++++----------
 src/libcamera/utils.cpp       | 48 +++++++++++++++++++++++++++++++++++
 test/libtest/test.cpp         |  4 ---
 5 files changed, 77 insertions(+), 52 deletions(-)