[libcamera-devel] [PATCH 1/2] meson: Replace obselete join_paths() with '/' operator
Laurent Pinchart
laurent.pinchart at ideasonboard.com
Sat May 8 02:05:56 CEST 2021
Hi Umang,
Thank you for the patch.
On Fri, May 07, 2021 at 11:28:40AM +0530, Umang Jain wrote:
> Since meson v0.49.0, join_paths() is equivalent to '/' hence,
> drop and replace it with '/' short-hand in meson files.
>
> This commit does not introduce any functional changes.
>
> Signed-off-by: Umang Jain <umang.jain at ideasonboard.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> ---
> Documentation/meson.build | 3 +--
> include/libcamera/ipa/meson.build | 2 +-
> include/libcamera/meson.build | 10 +++++-----
> meson.build | 3 +--
> src/ipa/meson.build | 2 +-
> src/ipa/raspberrypi/data/meson.build | 2 +-
> src/ipa/vimc/data/meson.build | 2 +-
> src/libcamera/meson.build | 2 +-
> src/libcamera/proxy/worker/meson.build | 4 ++--
> 9 files changed, 14 insertions(+), 16 deletions(-)
>
> diff --git a/Documentation/meson.build b/Documentation/meson.build
> index 9950465d..a82ee7c8 100644
> --- a/Documentation/meson.build
> +++ b/Documentation/meson.build
> @@ -1,7 +1,6 @@
> # SPDX-License-Identifier: CC0-1.0
>
> -doc_install_dir = join_paths(get_option('datadir'), 'doc',
> - 'libcamera- at 0@'.format(libcamera_version))
> +doc_install_dir = get_option('datadir') / 'doc' / 'libcamera- at 0@'.format(libcamera_version)
>
> #
> # Doxygen
> diff --git a/include/libcamera/ipa/meson.build b/include/libcamera/ipa/meson.build
> index 2d72c1fc..40c4e737 100644
> --- a/include/libcamera/ipa/meson.build
> +++ b/include/libcamera/ipa/meson.build
> @@ -7,7 +7,7 @@ libcamera_ipa_headers = files([
> ])
>
> install_headers(libcamera_ipa_headers,
> - subdir: join_paths(libcamera_include_dir, 'ipa'))
> + subdir: libcamera_include_dir / 'ipa')
>
> libcamera_generated_ipa_headers = []
>
> diff --git a/include/libcamera/meson.build b/include/libcamera/meson.build
> index c7b8ee8e..4a5b915e 100644
> --- a/include/libcamera/meson.build
> +++ b/include/libcamera/meson.build
> @@ -21,7 +21,7 @@ libcamera_public_headers = files([
> 'transform.h',
> ])
>
> -include_dir = join_paths(libcamera_include_dir, 'libcamera')
> +include_dir = libcamera_include_dir / 'libcamera'
>
> subdir('internal')
> subdir('ipa')
> @@ -48,7 +48,7 @@ foreach header : control_source_files
> output : header + '.h',
> command : [gen_controls, '-o', '@OUTPUT@', '@INPUT@'],
> install : true,
> - install_dir : join_paths('include', include_dir))
> + install_dir : 'include' / include_dir)
> endforeach
>
> libcamera_public_headers += control_headers
> @@ -63,7 +63,7 @@ formats_h = custom_target('formats_h',
> output : 'formats.h',
> command : [gen_formats, '-o', '@OUTPUT@', '@INPUT@'],
> install : true,
> - install_dir : join_paths('include', include_dir))
> + install_dir : 'include' / include_dir)
> libcamera_public_headers += formats_h
>
> # libcamera.h
> @@ -72,7 +72,7 @@ libcamera_h = custom_target('gen-header',
> output : 'libcamera.h',
> command : [gen_header, meson.current_source_dir(), '@OUTPUT@'],
> install : true,
> - install_dir : join_paths('include', include_dir))
> + install_dir : 'include' / include_dir)
>
> libcamera_public_headers += libcamera_h
>
> @@ -86,4 +86,4 @@ libcamera_version_config.set('LIBCAMERA_VERSION_PATCH', version[2])
> configure_file(input : 'version.h.in',
> output : 'version.h',
> configuration : libcamera_version_config,
> - install_dir : join_paths('include', include_dir))
> + install_dir : 'include' / include_dir)
> diff --git a/meson.build b/meson.build
> index 13d88301..46eb1b46 100644
> --- a/meson.build
> +++ b/meson.build
> @@ -148,8 +148,7 @@ endif
> # Create a symlink from the build root to the source root. This is used when
> # running libcamera from the build directory to locate resources in the source
> # directory (such as IPA configuration files).
> -run_command('ln', '-fsT', meson.source_root(),
> - join_paths(meson.build_root(), 'source'))
> +run_command('ln', '-fsT', meson.source_root(), meson.build_root() / 'source')
>
> configure_file(output : 'config.h', configuration : config_h)
>
> diff --git a/src/ipa/meson.build b/src/ipa/meson.build
> index b3f24f42..5b5684a1 100644
> --- a/src/ipa/meson.build
> +++ b/src/ipa/meson.build
> @@ -28,7 +28,7 @@ ipa_names = []
> foreach pipeline : pipelines
> if ipas.contains(pipeline)
> subdir(pipeline)
> - ipa_names += join_paths(ipa_install_dir, ipa_name + '.so')
> + ipa_names += ipa_install_dir / ipa_name + '.so'
> endif
> endforeach
>
> diff --git a/src/ipa/raspberrypi/data/meson.build b/src/ipa/raspberrypi/data/meson.build
> index 253fb9d7..92ad3272 100644
> --- a/src/ipa/raspberrypi/data/meson.build
> +++ b/src/ipa/raspberrypi/data/meson.build
> @@ -10,4 +10,4 @@ conf_files = files([
> ])
>
> install_data(conf_files,
> - install_dir : join_paths(ipa_data_dir, 'raspberrypi'))
> + install_dir : ipa_data_dir / 'raspberrypi')
> diff --git a/src/ipa/vimc/data/meson.build b/src/ipa/vimc/data/meson.build
> index 6532662c..42ec651c 100644
> --- a/src/ipa/vimc/data/meson.build
> +++ b/src/ipa/vimc/data/meson.build
> @@ -5,4 +5,4 @@ conf_files = files([
> ])
>
> install_data(conf_files,
> - install_dir : join_paths(ipa_data_dir, 'vimc'))
> + install_dir : ipa_data_dir / 'vimc')
> diff --git a/src/libcamera/meson.build b/src/libcamera/meson.build
> index e0a48aa2..99b09b3a 100644
> --- a/src/libcamera/meson.build
> +++ b/src/libcamera/meson.build
> @@ -102,7 +102,7 @@ endforeach
>
> libcamera_sources += control_sources
>
> -gen_version = join_paths(meson.source_root(), 'utils', 'gen-version.sh')
> +gen_version = meson.source_root() / 'utils' / 'gen-version.sh'
>
> version_cpp = vcs_tag(command : [gen_version, meson.build_root()],
> input : 'version.cpp.in',
> diff --git a/src/libcamera/proxy/worker/meson.build b/src/libcamera/proxy/worker/meson.build
> index 3796103e..40dada9d 100644
> --- a/src/libcamera/proxy/worker/meson.build
> +++ b/src/libcamera/proxy/worker/meson.build
> @@ -1,6 +1,6 @@
> # SPDX-License-Identifier: CC0-1.0
>
> -proxy_install_dir = join_paths(get_option('libexecdir'), 'libcamera')
> +proxy_install_dir = get_option('libexecdir') / 'libcamera'
>
> # generate {pipeline}_ipa_proxy_worker.cpp
> foreach mojom : ipa_mojoms
> @@ -25,4 +25,4 @@ foreach mojom : ipa_mojoms
> endforeach
>
> config_h.set('IPA_PROXY_DIR',
> - '"' + join_paths(get_option('prefix'), proxy_install_dir) + '"')
> + '"' + get_option('prefix') / proxy_install_dir + '"')
--
Regards,
Laurent Pinchart
More information about the libcamera-devel
mailing list