[{"id":25318,"web_url":"https://patchwork.libcamera.org/comment/25318/","msgid":"<20221006072408.wvbmonbqn73f4cqx@uno.localdomain>","date":"2022-10-06T07:24:08","subject":"Re: [libcamera-devel] [PATCH 1/2] test: Rename 't' to 'test' in\n\tmeson.build","submitter":{"id":3,"url":"https://patchwork.libcamera.org/api/people/3/","name":"Jacopo Mondi","email":"jacopo@jmondi.org"},"content":"Hi Laurent\n\nOn Wed, Oct 05, 2022 at 10:39:43PM +0300, Laurent Pinchart via libcamera-devel wrote:\n> The 't' name is very short and not very explicit. Rename it to 'test'\n> instead.\n>\n> Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>\n\nReviewed-by: Jacopo Mondi <jacopo@jmondi.org>\n\nThanks\n  j\n\n> ---\n>  test/camera/meson.build           |  6 +++---\n>  test/controls/meson.build         |  6 +++---\n>  test/gstreamer/meson.build        |  6 +++---\n>  test/ipa/meson.build              |  6 +++---\n>  test/ipc/meson.build              |  6 +++---\n>  test/log/meson.build              |  6 +++---\n>  test/media_device/meson.build     |  6 +++---\n>  test/meson.build                  | 30 +++++++++++++++---------------\n>  test/pipeline/ipu3/meson.build    |  6 +++---\n>  test/pipeline/rkisp1/meson.build  |  6 +++---\n>  test/process/meson.build          |  6 +++---\n>  test/serialization/meson.build    |  6 +++---\n>  test/stream/meson.build           |  6 +++---\n>  test/v4l2_subdevice/meson.build   |  6 +++---\n>  test/v4l2_videodevice/meson.build |  6 +++---\n>  15 files changed, 57 insertions(+), 57 deletions(-)\n>\n> diff --git a/test/camera/meson.build b/test/camera/meson.build\n> index 668d5c03fb6e..df0520293bdb 100644\n> --- a/test/camera/meson.build\n> +++ b/test/camera/meson.build\n> @@ -11,10 +11,10 @@ camera_tests = [\n>      ['camera_reconfigure',      'camera_reconfigure.cpp'],\n>  ]\n>\n> -foreach t : camera_tests\n> -    exe = executable(t[0], t[1],\n> +foreach test : camera_tests\n> +    exe = executable(test[0], test[1],\n>                       dependencies : libcamera_private,\n>                       link_with : test_libraries,\n>                       include_directories : test_includes_internal)\n> -    test(t[0], exe, suite : 'camera', is_parallel : false)\n> +    test(test[0], exe, suite : 'camera', is_parallel : false)\n>  endforeach\n> diff --git a/test/controls/meson.build b/test/controls/meson.build\n> index 0103543e1fc2..8cf23be6487c 100644\n> --- a/test/controls/meson.build\n> +++ b/test/controls/meson.build\n> @@ -7,10 +7,10 @@ control_tests = [\n>      ['control_value',               'control_value.cpp'],\n>  ]\n>\n> -foreach t : control_tests\n> -    exe = executable(t[0], t[1],\n> +foreach test : control_tests\n> +    exe = executable(test[0], test[1],\n>                       dependencies : libcamera_public,\n>                       link_with : test_libraries,\n>                       include_directories : test_includes_internal)\n> -    test(t[0], exe, suite : 'controls', is_parallel : false)\n> +    test(test[0], exe, suite : 'controls', is_parallel : false)\n>  endforeach\n> diff --git a/test/gstreamer/meson.build b/test/gstreamer/meson.build\n> index 10058fc5206f..46f6a77a771e 100644\n> --- a/test/gstreamer/meson.build\n> +++ b/test/gstreamer/meson.build\n> @@ -10,11 +10,11 @@ gstreamer_tests = [\n>  ]\n>  gstreamer_dep = dependency('gstreamer-1.0', required: true)\n>\n> -foreach t : gstreamer_tests\n> -    exe = executable(t[0], t[1], 'gstreamer_test.cpp',\n> +foreach test : gstreamer_tests\n> +    exe = executable(test[0], test[1], 'gstreamer_test.cpp',\n>                       dependencies : [libcamera_private, gstreamer_dep],\n>                       link_with : test_libraries,\n>                       include_directories : test_includes_internal)\n>\n> -    test(t[0], exe, suite : 'gstreamer', is_parallel : false)\n> +    test(test[0], exe, suite : 'gstreamer', is_parallel : false)\n>  endforeach\n> diff --git a/test/ipa/meson.build b/test/ipa/meson.build\n> index 7938633e54ab..295807fd8a45 100644\n> --- a/test/ipa/meson.build\n> +++ b/test/ipa/meson.build\n> @@ -5,11 +5,11 @@ ipa_test = [\n>      ['ipa_interface_test',  'ipa_interface_test.cpp'],\n>  ]\n>\n> -foreach t : ipa_test\n> -    exe = executable(t[0], [t[1], libcamera_generated_ipa_headers],\n> +foreach test : ipa_test\n> +    exe = executable(test[0], [test[1], libcamera_generated_ipa_headers],\n>                       dependencies : libcamera_private,\n>                       link_with : [libipa, test_libraries],\n>                       include_directories : [libipa_includes, test_includes_internal])\n>\n> -    test(t[0], exe, suite : 'ipa')\n> +    test(test[0], exe, suite : 'ipa')\n>  endforeach\n> diff --git a/test/ipc/meson.build b/test/ipc/meson.build\n> index 2a6cd7fb8433..ce21135b701d 100644\n> --- a/test/ipc/meson.build\n> +++ b/test/ipc/meson.build\n> @@ -5,11 +5,11 @@ ipc_tests = [\n>      ['unixsocket',     'unixsocket.cpp'],\n>  ]\n>\n> -foreach t : ipc_tests\n> -    exe = executable(t[0], t[1],\n> +foreach test : ipc_tests\n> +    exe = executable(test[0], test[1],\n>                       dependencies : libcamera_private,\n>                       link_with : test_libraries,\n>                       include_directories : test_includes_internal)\n>\n> -    test(t[0], exe, suite : 'ipc')\n> +    test(test[0], exe, suite : 'ipc')\n>  endforeach\n> diff --git a/test/log/meson.build b/test/log/meson.build\n> index ac87841a24e1..d2d51e471d52 100644\n> --- a/test/log/meson.build\n> +++ b/test/log/meson.build\n> @@ -5,11 +5,11 @@ log_test = [\n>      ['log_process', 'log_process.cpp'],\n>  ]\n>\n> -foreach t : log_test\n> -    exe = executable(t[0], t[1],\n> +foreach test : log_test\n> +    exe = executable(test[0], test[1],\n>                       dependencies : libcamera_private,\n>                       link_with : test_libraries,\n>                       include_directories : test_includes_internal)\n>\n> -    test(t[0], exe, suite : 'log')\n> +    test(test[0], exe, suite : 'log')\n>  endforeach\n> diff --git a/test/media_device/meson.build b/test/media_device/meson.build\n> index 83dfe8f11d2f..7d34509d6797 100644\n> --- a/test/media_device/meson.build\n> +++ b/test/media_device/meson.build\n> @@ -14,11 +14,11 @@ lib_mdev_test = static_library('lib_mdev_test', lib_mdev_test_sources,\n>                                 dependencies : libcamera_private,\n>                                 include_directories : test_includes_internal)\n>\n> -foreach t : media_device_tests\n> -    exe = executable(t[0], t[1],\n> +foreach test : media_device_tests\n> +    exe = executable(test[0], test[1],\n>                       dependencies : libcamera_private,\n>                       link_with : [test_libraries, lib_mdev_test],\n>                       include_directories : test_includes_internal)\n>\n> -    test(t[0], exe, suite : 'media_device', is_parallel : false)\n> +    test(test[0], exe, suite : 'media_device', is_parallel : false)\n>  endforeach\n> diff --git a/test/meson.build b/test/meson.build\n> index 9bf7bf34e796..4bc01d78f4e6 100644\n> --- a/test/meson.build\n> +++ b/test/meson.build\n> @@ -64,44 +64,44 @@ internal_non_parallel_tests = [\n>      ['mapped-buffer',                   'mapped-buffer.cpp'],\n>  ]\n>\n> -foreach t : public_tests\n> +foreach test : public_tests\n>      deps = [libcamera_public]\n> -    if t.length() > 2\n> -        deps += t[2]\n> +    if test.length() > 2\n> +        deps += test[2]\n>      endif\n>\n> -    exe = executable(t[0], t[1],\n> +    exe = executable(test[0], test[1],\n>                       dependencies : deps,\n>                       link_with : test_libraries,\n>                       include_directories : test_includes_public)\n>\n> -    test(t[0], exe)\n> +    test(test[0], exe)\n>  endforeach\n>\n> -foreach t : internal_tests\n> +foreach test : internal_tests\n>      deps = [libcamera_private]\n> -    if t.length() > 2\n> -        deps += t[2]\n> +    if test.length() > 2\n> +        deps += test[2]\n>      endif\n>\n> -    exe = executable(t[0], t[1],\n> +    exe = executable(test[0], test[1],\n>                       dependencies : deps,\n>                       link_with : test_libraries,\n>                       include_directories : test_includes_internal)\n>\n> -    test(t[0], exe)\n> +    test(test[0], exe)\n>  endforeach\n>\n> -foreach t : internal_non_parallel_tests\n> +foreach test : internal_non_parallel_tests\n>      deps = [libcamera_private]\n> -    if t.length() > 2\n> -        deps += t[2]\n> +    if test.length() > 2\n> +        deps += test[2]\n>      endif\n>\n> -    exe = executable(t[0], t[1],\n> +    exe = executable(test[0], test[1],\n>                       dependencies : deps,\n>                       link_with : test_libraries,\n>                       include_directories : test_includes_internal)\n>\n> -    test(t[0], exe, is_parallel : false)\n> +    test(test[0], exe, is_parallel : false)\n>  endforeach\n> diff --git a/test/pipeline/ipu3/meson.build b/test/pipeline/ipu3/meson.build\n> index 16701080ca2b..5fcf5730b0b0 100644\n> --- a/test/pipeline/ipu3/meson.build\n> +++ b/test/pipeline/ipu3/meson.build\n> @@ -4,11 +4,11 @@ ipu3_test = [\n>      ['ipu3_pipeline_test',            'ipu3_pipeline_test.cpp'],\n>  ]\n>\n> -foreach t : ipu3_test\n> -    exe = executable(t[0], t[1],\n> +foreach test : ipu3_test\n> +    exe = executable(test[0], test[1],\n>                       dependencies : libcamera_private,\n>                       link_with : test_libraries,\n>                       include_directories : test_includes_internal)\n>\n> -    test(t[0], exe, suite : 'ipu3', is_parallel : false)\n> +    test(test[0], exe, suite : 'ipu3', is_parallel : false)\n>  endforeach\n> diff --git a/test/pipeline/rkisp1/meson.build b/test/pipeline/rkisp1/meson.build\n> index 364b5711a0f9..c82cc78972f6 100644\n> --- a/test/pipeline/rkisp1/meson.build\n> +++ b/test/pipeline/rkisp1/meson.build\n> @@ -4,11 +4,11 @@ rkisp1_test = [\n>      ['rkisp1_pipeline_test',            'rkisp1_pipeline_test.cpp'],\n>  ]\n>\n> -foreach t : rkisp1_test\n> -    exe = executable(t[0], t[1],\n> +foreach test : rkisp1_test\n> +    exe = executable(test[0], test[1],\n>                       dependencies : libcamera_private,\n>                       link_with : test_libraries,\n>                       include_directories : test_includes_internal)\n>\n> -    test(t[0], exe, suite : 'rkisp1', is_parallel : false)\n> +    test(test[0], exe, suite : 'rkisp1', is_parallel : false)\n>  endforeach\n> diff --git a/test/process/meson.build b/test/process/meson.build\n> index af86b277db63..c656d7343e54 100644\n> --- a/test/process/meson.build\n> +++ b/test/process/meson.build\n> @@ -4,11 +4,11 @@ process_tests = [\n>      ['process_test',   'process_test.cpp'],\n>  ]\n>\n> -foreach t : process_tests\n> -    exe = executable(t[0], t[1],\n> +foreach test : process_tests\n> +    exe = executable(test[0], test[1],\n>                       dependencies : libcamera_private,\n>                       link_with : test_libraries,\n>                       include_directories : test_includes_internal)\n>\n> -    test(t[0], exe, suite : 'process', is_parallel : false)\n> +    test(test[0], exe, suite : 'process', is_parallel : false)\n>  endforeach\n> diff --git a/test/serialization/meson.build b/test/serialization/meson.build\n> index 26e42b150b40..c4206cb3632b 100644\n> --- a/test/serialization/meson.build\n> +++ b/test/serialization/meson.build\n> @@ -7,10 +7,10 @@ serialization_tests = [\n>      ['ipa_data_serializer_test',  'ipa_data_serializer_test.cpp'],\n>  ]\n>\n> -foreach t : serialization_tests\n> -    exe = executable(t[0], [t[1], 'serialization_test.cpp'],\n> +foreach test : serialization_tests\n> +    exe = executable(test[0], [test[1], 'serialization_test.cpp'],\n>                       dependencies : libcamera_private,\n>                       link_with : test_libraries,\n>                       include_directories : test_includes_internal)\n> -    test(t[0], exe, suite : 'serialization', is_parallel : false)\n> +    test(test[0], exe, suite : 'serialization', is_parallel : false)\n>  endforeach\n> diff --git a/test/stream/meson.build b/test/stream/meson.build\n> index 89f51c18a63e..3a97868bce1c 100644\n> --- a/test/stream/meson.build\n> +++ b/test/stream/meson.build\n> @@ -5,10 +5,10 @@ stream_tests = [\n>      ['stream_formats',  'stream_formats.cpp'],\n>  ]\n>\n> -foreach t : stream_tests\n> -    exe = executable(t[0], t[1],\n> +foreach test : stream_tests\n> +    exe = executable(test[0], test[1],\n>                       dependencies : libcamera_public,\n>                       link_with : test_libraries,\n>                       include_directories : test_includes_internal)\n> -    test(t[0], exe, suite: 'stream')\n> +    test(test[0], exe, suite: 'stream')\n>  endforeach\n> diff --git a/test/v4l2_subdevice/meson.build b/test/v4l2_subdevice/meson.build\n> index d82be3c60b75..4e607d41fc54 100644\n> --- a/test/v4l2_subdevice/meson.build\n> +++ b/test/v4l2_subdevice/meson.build\n> @@ -5,10 +5,10 @@ v4l2_subdevice_tests = [\n>    ['test_formats',              'test_formats.cpp'],\n>  ]\n>\n> -foreach t : v4l2_subdevice_tests\n> -    exe = executable(t[0], [t[1], 'v4l2_subdevice_test.cpp'],\n> +foreach test : v4l2_subdevice_tests\n> +    exe = executable(test[0], [test[1], 'v4l2_subdevice_test.cpp'],\n>          dependencies : libcamera_private,\n>          link_with : test_libraries,\n>          include_directories : test_includes_internal)\n> -    test(t[0], exe, suite : 'v4l2_subdevice', is_parallel : false)\n> +    test(test[0], exe, suite : 'v4l2_subdevice', is_parallel : false)\n>  endforeach\n> diff --git a/test/v4l2_videodevice/meson.build b/test/v4l2_videodevice/meson.build\n> index 7a26f53d106a..f85374854c48 100644\n> --- a/test/v4l2_videodevice/meson.build\n> +++ b/test/v4l2_videodevice/meson.build\n> @@ -15,10 +15,10 @@ v4l2_videodevice_tests = [\n>      ['v4l2_m2mdevice',      'v4l2_m2mdevice.cpp'],\n>  ]\n>\n> -foreach t : v4l2_videodevice_tests\n> -    exe = executable(t[0], [t[1], 'v4l2_videodevice_test.cpp'],\n> +foreach test : v4l2_videodevice_tests\n> +    exe = executable(test[0], [test[1], 'v4l2_videodevice_test.cpp'],\n>                       dependencies : libcamera_private,\n>                       link_with : test_libraries,\n>                       include_directories : test_includes_internal)\n> -    test(t[0], exe, suite : 'v4l2_videodevice', is_parallel : false)\n> +    test(test[0], exe, suite : 'v4l2_videodevice', is_parallel : false)\n>  endforeach\n> --\n> Regards,\n>\n> Laurent Pinchart\n>","headers":{"Return-Path":"<libcamera-devel-bounces@lists.libcamera.org>","X-Original-To":"parsemail@patchwork.libcamera.org","Delivered-To":"parsemail@patchwork.libcamera.org","Received":["from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id B2F98C0DA4\n\tfor <parsemail@patchwork.libcamera.org>;\n\tThu,  6 Oct 2022 07:24:13 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 0D23862CEA;\n\tThu,  6 Oct 2022 09:24:13 +0200 (CEST)","from relay5-d.mail.gandi.net (relay5-d.mail.gandi.net\n\t[217.70.183.197])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id E619E60A83\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tThu,  6 Oct 2022 09:24:10 +0200 (CEST)","(Authenticated sender: jacopo@jmondi.org)\n\tby mail.gandi.net (Postfix) with ESMTPSA id 4B7131C0009;\n\tThu,  6 Oct 2022 07:24:10 +0000 (UTC)"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=libcamera.org;\n\ts=mail; t=1665041053;\n\tbh=pg+yFbnqSgu0QX0LcLrmxPybjTHukE1V38iCAAayWHU=;\n\th=Date:To:References:In-Reply-To:Subject:List-Id:List-Unsubscribe:\n\tList-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To:Cc:\n\tFrom;\n\tb=DjYn21rxS/iEA40k4js4WxNLhr3fmK5EFnt2+OAiKfr8XUa4KFiZnimV/3qQdey+y\n\tnHvA4jhqVlLANHyKgc2Pvp4jOx1Oi2t47NI+CeTgLMEcLJMh8kwBKKeMXdTkDOieAr\n\tTBwjhHPlJmSfnO4ppfxmj+UX38jvrD+nm2VCHwYhonEQoEGrwQL6ufTscCyso1/QQP\n\tpJqBstq8O8rai0AHwUuT/hGwFnPiRzk4vRe/kGttgZ9OpAzPrsNyenq+6FHxvOw86i\n\tgZ0jPOliAfS5dZTOOLjDpuXfs1CvedNcVa9mMuFo8Gx2oitJ1w9TOYmn/WAw8mjdEc\n\trYKeiM5n6yDFA==","Date":"Thu, 6 Oct 2022 09:24:08 +0200","To":"Laurent Pinchart <laurent.pinchart@ideasonboard.com>","Message-ID":"<20221006072408.wvbmonbqn73f4cqx@uno.localdomain>","References":"<20221005193944.16383-1-laurent.pinchart@ideasonboard.com>\n\t<20221005193944.16383-2-laurent.pinchart@ideasonboard.com>","MIME-Version":"1.0","Content-Type":"text/plain; charset=utf-8","Content-Disposition":"inline","In-Reply-To":"<20221005193944.16383-2-laurent.pinchart@ideasonboard.com>","Subject":"Re: [libcamera-devel] [PATCH 1/2] test: Rename 't' to 'test' in\n\tmeson.build","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"<libcamera-devel.lists.libcamera.org>","List-Unsubscribe":"<https://lists.libcamera.org/options/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=unsubscribe>","List-Archive":"<https://lists.libcamera.org/pipermail/libcamera-devel/>","List-Post":"<mailto:libcamera-devel@lists.libcamera.org>","List-Help":"<mailto:libcamera-devel-request@lists.libcamera.org?subject=help>","List-Subscribe":"<https://lists.libcamera.org/listinfo/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=subscribe>","From":"Jacopo Mondi via libcamera-devel <libcamera-devel@lists.libcamera.org>","Reply-To":"Jacopo Mondi <jacopo@jmondi.org>","Cc":"libcamera-devel@lists.libcamera.org","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>"}},{"id":25329,"web_url":"https://patchwork.libcamera.org/comment/25329/","msgid":"<3e7045b5-9b3f-4a13-68f3-686e7cb9114e@ideasonboard.com>","date":"2022-10-07T13:29:25","subject":"Re: [libcamera-devel] [PATCH 1/2] test: Rename 't' to 'test' in\n\tmeson.build","submitter":{"id":86,"url":"https://patchwork.libcamera.org/api/people/86/","name":"Umang Jain","email":"umang.jain@ideasonboard.com"},"content":"Hi Laurent,\n\nThanks for the patch.\n\nOn 10/6/22 1:09 AM, Laurent Pinchart via libcamera-devel wrote:\n> The 't' name is very short and not very explicit. Rename it to 'test'\n> instead.\n>\n> Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>\n\nReviewed-by: Umang Jain <umang.jain@ideasonboard.com>\n\n> ---\n>   test/camera/meson.build           |  6 +++---\n>   test/controls/meson.build         |  6 +++---\n>   test/gstreamer/meson.build        |  6 +++---\n>   test/ipa/meson.build              |  6 +++---\n>   test/ipc/meson.build              |  6 +++---\n>   test/log/meson.build              |  6 +++---\n>   test/media_device/meson.build     |  6 +++---\n>   test/meson.build                  | 30 +++++++++++++++---------------\n>   test/pipeline/ipu3/meson.build    |  6 +++---\n>   test/pipeline/rkisp1/meson.build  |  6 +++---\n>   test/process/meson.build          |  6 +++---\n>   test/serialization/meson.build    |  6 +++---\n>   test/stream/meson.build           |  6 +++---\n>   test/v4l2_subdevice/meson.build   |  6 +++---\n>   test/v4l2_videodevice/meson.build |  6 +++---\n>   15 files changed, 57 insertions(+), 57 deletions(-)\n>\n> diff --git a/test/camera/meson.build b/test/camera/meson.build\n> index 668d5c03fb6e..df0520293bdb 100644\n> --- a/test/camera/meson.build\n> +++ b/test/camera/meson.build\n> @@ -11,10 +11,10 @@ camera_tests = [\n>       ['camera_reconfigure',      'camera_reconfigure.cpp'],\n>   ]\n>   \n> -foreach t : camera_tests\n> -    exe = executable(t[0], t[1],\n> +foreach test : camera_tests\n> +    exe = executable(test[0], test[1],\n>                        dependencies : libcamera_private,\n>                        link_with : test_libraries,\n>                        include_directories : test_includes_internal)\n> -    test(t[0], exe, suite : 'camera', is_parallel : false)\n> +    test(test[0], exe, suite : 'camera', is_parallel : false)\n>   endforeach\n> diff --git a/test/controls/meson.build b/test/controls/meson.build\n> index 0103543e1fc2..8cf23be6487c 100644\n> --- a/test/controls/meson.build\n> +++ b/test/controls/meson.build\n> @@ -7,10 +7,10 @@ control_tests = [\n>       ['control_value',               'control_value.cpp'],\n>   ]\n>   \n> -foreach t : control_tests\n> -    exe = executable(t[0], t[1],\n> +foreach test : control_tests\n> +    exe = executable(test[0], test[1],\n>                        dependencies : libcamera_public,\n>                        link_with : test_libraries,\n>                        include_directories : test_includes_internal)\n> -    test(t[0], exe, suite : 'controls', is_parallel : false)\n> +    test(test[0], exe, suite : 'controls', is_parallel : false)\n>   endforeach\n> diff --git a/test/gstreamer/meson.build b/test/gstreamer/meson.build\n> index 10058fc5206f..46f6a77a771e 100644\n> --- a/test/gstreamer/meson.build\n> +++ b/test/gstreamer/meson.build\n> @@ -10,11 +10,11 @@ gstreamer_tests = [\n>   ]\n>   gstreamer_dep = dependency('gstreamer-1.0', required: true)\n>   \n> -foreach t : gstreamer_tests\n> -    exe = executable(t[0], t[1], 'gstreamer_test.cpp',\n> +foreach test : gstreamer_tests\n> +    exe = executable(test[0], test[1], 'gstreamer_test.cpp',\n>                        dependencies : [libcamera_private, gstreamer_dep],\n>                        link_with : test_libraries,\n>                        include_directories : test_includes_internal)\n>   \n> -    test(t[0], exe, suite : 'gstreamer', is_parallel : false)\n> +    test(test[0], exe, suite : 'gstreamer', is_parallel : false)\n>   endforeach\n> diff --git a/test/ipa/meson.build b/test/ipa/meson.build\n> index 7938633e54ab..295807fd8a45 100644\n> --- a/test/ipa/meson.build\n> +++ b/test/ipa/meson.build\n> @@ -5,11 +5,11 @@ ipa_test = [\n>       ['ipa_interface_test',  'ipa_interface_test.cpp'],\n>   ]\n>   \n> -foreach t : ipa_test\n> -    exe = executable(t[0], [t[1], libcamera_generated_ipa_headers],\n> +foreach test : ipa_test\n> +    exe = executable(test[0], [test[1], libcamera_generated_ipa_headers],\n>                        dependencies : libcamera_private,\n>                        link_with : [libipa, test_libraries],\n>                        include_directories : [libipa_includes, test_includes_internal])\n>   \n> -    test(t[0], exe, suite : 'ipa')\n> +    test(test[0], exe, suite : 'ipa')\n>   endforeach\n> diff --git a/test/ipc/meson.build b/test/ipc/meson.build\n> index 2a6cd7fb8433..ce21135b701d 100644\n> --- a/test/ipc/meson.build\n> +++ b/test/ipc/meson.build\n> @@ -5,11 +5,11 @@ ipc_tests = [\n>       ['unixsocket',     'unixsocket.cpp'],\n>   ]\n>   \n> -foreach t : ipc_tests\n> -    exe = executable(t[0], t[1],\n> +foreach test : ipc_tests\n> +    exe = executable(test[0], test[1],\n>                        dependencies : libcamera_private,\n>                        link_with : test_libraries,\n>                        include_directories : test_includes_internal)\n>   \n> -    test(t[0], exe, suite : 'ipc')\n> +    test(test[0], exe, suite : 'ipc')\n>   endforeach\n> diff --git a/test/log/meson.build b/test/log/meson.build\n> index ac87841a24e1..d2d51e471d52 100644\n> --- a/test/log/meson.build\n> +++ b/test/log/meson.build\n> @@ -5,11 +5,11 @@ log_test = [\n>       ['log_process', 'log_process.cpp'],\n>   ]\n>   \n> -foreach t : log_test\n> -    exe = executable(t[0], t[1],\n> +foreach test : log_test\n> +    exe = executable(test[0], test[1],\n>                        dependencies : libcamera_private,\n>                        link_with : test_libraries,\n>                        include_directories : test_includes_internal)\n>   \n> -    test(t[0], exe, suite : 'log')\n> +    test(test[0], exe, suite : 'log')\n>   endforeach\n> diff --git a/test/media_device/meson.build b/test/media_device/meson.build\n> index 83dfe8f11d2f..7d34509d6797 100644\n> --- a/test/media_device/meson.build\n> +++ b/test/media_device/meson.build\n> @@ -14,11 +14,11 @@ lib_mdev_test = static_library('lib_mdev_test', lib_mdev_test_sources,\n>                                  dependencies : libcamera_private,\n>                                  include_directories : test_includes_internal)\n>   \n> -foreach t : media_device_tests\n> -    exe = executable(t[0], t[1],\n> +foreach test : media_device_tests\n> +    exe = executable(test[0], test[1],\n>                        dependencies : libcamera_private,\n>                        link_with : [test_libraries, lib_mdev_test],\n>                        include_directories : test_includes_internal)\n>   \n> -    test(t[0], exe, suite : 'media_device', is_parallel : false)\n> +    test(test[0], exe, suite : 'media_device', is_parallel : false)\n>   endforeach\n> diff --git a/test/meson.build b/test/meson.build\n> index 9bf7bf34e796..4bc01d78f4e6 100644\n> --- a/test/meson.build\n> +++ b/test/meson.build\n> @@ -64,44 +64,44 @@ internal_non_parallel_tests = [\n>       ['mapped-buffer',                   'mapped-buffer.cpp'],\n>   ]\n>   \n> -foreach t : public_tests\n> +foreach test : public_tests\n>       deps = [libcamera_public]\n> -    if t.length() > 2\n> -        deps += t[2]\n> +    if test.length() > 2\n> +        deps += test[2]\n>       endif\n>   \n> -    exe = executable(t[0], t[1],\n> +    exe = executable(test[0], test[1],\n>                        dependencies : deps,\n>                        link_with : test_libraries,\n>                        include_directories : test_includes_public)\n>   \n> -    test(t[0], exe)\n> +    test(test[0], exe)\n>   endforeach\n>   \n> -foreach t : internal_tests\n> +foreach test : internal_tests\n>       deps = [libcamera_private]\n> -    if t.length() > 2\n> -        deps += t[2]\n> +    if test.length() > 2\n> +        deps += test[2]\n>       endif\n>   \n> -    exe = executable(t[0], t[1],\n> +    exe = executable(test[0], test[1],\n>                        dependencies : deps,\n>                        link_with : test_libraries,\n>                        include_directories : test_includes_internal)\n>   \n> -    test(t[0], exe)\n> +    test(test[0], exe)\n>   endforeach\n>   \n> -foreach t : internal_non_parallel_tests\n> +foreach test : internal_non_parallel_tests\n>       deps = [libcamera_private]\n> -    if t.length() > 2\n> -        deps += t[2]\n> +    if test.length() > 2\n> +        deps += test[2]\n>       endif\n>   \n> -    exe = executable(t[0], t[1],\n> +    exe = executable(test[0], test[1],\n>                        dependencies : deps,\n>                        link_with : test_libraries,\n>                        include_directories : test_includes_internal)\n>   \n> -    test(t[0], exe, is_parallel : false)\n> +    test(test[0], exe, is_parallel : false)\n>   endforeach\n> diff --git a/test/pipeline/ipu3/meson.build b/test/pipeline/ipu3/meson.build\n> index 16701080ca2b..5fcf5730b0b0 100644\n> --- a/test/pipeline/ipu3/meson.build\n> +++ b/test/pipeline/ipu3/meson.build\n> @@ -4,11 +4,11 @@ ipu3_test = [\n>       ['ipu3_pipeline_test',            'ipu3_pipeline_test.cpp'],\n>   ]\n>   \n> -foreach t : ipu3_test\n> -    exe = executable(t[0], t[1],\n> +foreach test : ipu3_test\n> +    exe = executable(test[0], test[1],\n>                        dependencies : libcamera_private,\n>                        link_with : test_libraries,\n>                        include_directories : test_includes_internal)\n>   \n> -    test(t[0], exe, suite : 'ipu3', is_parallel : false)\n> +    test(test[0], exe, suite : 'ipu3', is_parallel : false)\n>   endforeach\n> diff --git a/test/pipeline/rkisp1/meson.build b/test/pipeline/rkisp1/meson.build\n> index 364b5711a0f9..c82cc78972f6 100644\n> --- a/test/pipeline/rkisp1/meson.build\n> +++ b/test/pipeline/rkisp1/meson.build\n> @@ -4,11 +4,11 @@ rkisp1_test = [\n>       ['rkisp1_pipeline_test',            'rkisp1_pipeline_test.cpp'],\n>   ]\n>   \n> -foreach t : rkisp1_test\n> -    exe = executable(t[0], t[1],\n> +foreach test : rkisp1_test\n> +    exe = executable(test[0], test[1],\n>                        dependencies : libcamera_private,\n>                        link_with : test_libraries,\n>                        include_directories : test_includes_internal)\n>   \n> -    test(t[0], exe, suite : 'rkisp1', is_parallel : false)\n> +    test(test[0], exe, suite : 'rkisp1', is_parallel : false)\n>   endforeach\n> diff --git a/test/process/meson.build b/test/process/meson.build\n> index af86b277db63..c656d7343e54 100644\n> --- a/test/process/meson.build\n> +++ b/test/process/meson.build\n> @@ -4,11 +4,11 @@ process_tests = [\n>       ['process_test',   'process_test.cpp'],\n>   ]\n>   \n> -foreach t : process_tests\n> -    exe = executable(t[0], t[1],\n> +foreach test : process_tests\n> +    exe = executable(test[0], test[1],\n>                        dependencies : libcamera_private,\n>                        link_with : test_libraries,\n>                        include_directories : test_includes_internal)\n>   \n> -    test(t[0], exe, suite : 'process', is_parallel : false)\n> +    test(test[0], exe, suite : 'process', is_parallel : false)\n>   endforeach\n> diff --git a/test/serialization/meson.build b/test/serialization/meson.build\n> index 26e42b150b40..c4206cb3632b 100644\n> --- a/test/serialization/meson.build\n> +++ b/test/serialization/meson.build\n> @@ -7,10 +7,10 @@ serialization_tests = [\n>       ['ipa_data_serializer_test',  'ipa_data_serializer_test.cpp'],\n>   ]\n>   \n> -foreach t : serialization_tests\n> -    exe = executable(t[0], [t[1], 'serialization_test.cpp'],\n> +foreach test : serialization_tests\n> +    exe = executable(test[0], [test[1], 'serialization_test.cpp'],\n>                        dependencies : libcamera_private,\n>                        link_with : test_libraries,\n>                        include_directories : test_includes_internal)\n> -    test(t[0], exe, suite : 'serialization', is_parallel : false)\n> +    test(test[0], exe, suite : 'serialization', is_parallel : false)\n>   endforeach\n> diff --git a/test/stream/meson.build b/test/stream/meson.build\n> index 89f51c18a63e..3a97868bce1c 100644\n> --- a/test/stream/meson.build\n> +++ b/test/stream/meson.build\n> @@ -5,10 +5,10 @@ stream_tests = [\n>       ['stream_formats',  'stream_formats.cpp'],\n>   ]\n>   \n> -foreach t : stream_tests\n> -    exe = executable(t[0], t[1],\n> +foreach test : stream_tests\n> +    exe = executable(test[0], test[1],\n>                        dependencies : libcamera_public,\n>                        link_with : test_libraries,\n>                        include_directories : test_includes_internal)\n> -    test(t[0], exe, suite: 'stream')\n> +    test(test[0], exe, suite: 'stream')\n>   endforeach\n> diff --git a/test/v4l2_subdevice/meson.build b/test/v4l2_subdevice/meson.build\n> index d82be3c60b75..4e607d41fc54 100644\n> --- a/test/v4l2_subdevice/meson.build\n> +++ b/test/v4l2_subdevice/meson.build\n> @@ -5,10 +5,10 @@ v4l2_subdevice_tests = [\n>     ['test_formats',              'test_formats.cpp'],\n>   ]\n>   \n> -foreach t : v4l2_subdevice_tests\n> -    exe = executable(t[0], [t[1], 'v4l2_subdevice_test.cpp'],\n> +foreach test : v4l2_subdevice_tests\n> +    exe = executable(test[0], [test[1], 'v4l2_subdevice_test.cpp'],\n>           dependencies : libcamera_private,\n>           link_with : test_libraries,\n>           include_directories : test_includes_internal)\n> -    test(t[0], exe, suite : 'v4l2_subdevice', is_parallel : false)\n> +    test(test[0], exe, suite : 'v4l2_subdevice', is_parallel : false)\n>   endforeach\n> diff --git a/test/v4l2_videodevice/meson.build b/test/v4l2_videodevice/meson.build\n> index 7a26f53d106a..f85374854c48 100644\n> --- a/test/v4l2_videodevice/meson.build\n> +++ b/test/v4l2_videodevice/meson.build\n> @@ -15,10 +15,10 @@ v4l2_videodevice_tests = [\n>       ['v4l2_m2mdevice',      'v4l2_m2mdevice.cpp'],\n>   ]\n>   \n> -foreach t : v4l2_videodevice_tests\n> -    exe = executable(t[0], [t[1], 'v4l2_videodevice_test.cpp'],\n> +foreach test : v4l2_videodevice_tests\n> +    exe = executable(test[0], [test[1], 'v4l2_videodevice_test.cpp'],\n>                        dependencies : libcamera_private,\n>                        link_with : test_libraries,\n>                        include_directories : test_includes_internal)\n> -    test(t[0], exe, suite : 'v4l2_videodevice', is_parallel : false)\n> +    test(test[0], exe, suite : 'v4l2_videodevice', is_parallel : false)\n>   endforeach","headers":{"Return-Path":"<libcamera-devel-bounces@lists.libcamera.org>","X-Original-To":"parsemail@patchwork.libcamera.org","Delivered-To":"parsemail@patchwork.libcamera.org","Received":["from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id 1315BBD16B\n\tfor <parsemail@patchwork.libcamera.org>;\n\tFri,  7 Oct 2022 13:29:34 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 4DC7662D13;\n\tFri,  7 Oct 2022 15:29:33 +0200 (CEST)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 4060760A88\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tFri,  7 Oct 2022 15:29:32 +0200 (CEST)","from [192.168.1.103] (unknown [103.251.226.3])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 9030A13FA;\n\tFri,  7 Oct 2022 15:29:30 +0200 (CEST)"],"DKIM-Signature":["v=1; a=rsa-sha256; c=relaxed/simple; d=libcamera.org;\n\ts=mail; t=1665149373;\n\tbh=Ecm7KZdPYTpIpZoZ+YxKJ8Y2tKRG7HXIl+eJC4QmDJA=;\n\th=Date:To:References:In-Reply-To:Subject:List-Id:List-Unsubscribe:\n\tList-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To:\n\tFrom;\n\tb=UC2tXSQIoW6tTOePHytDxYeaVUwZmeNowjZp/Gs/YSASqnGOKvjJXmY5OHS29Am0u\n\tc1LsfO1W+45T50FB395zKRbV7FrCI7DSEmkuSCtnDfsySe2APy6Er0uVvRbw3saz8M\n\t73aZEYD9h2xGcSRSv2bVl2v2MrvlDxCegtnHMXYX/Y0heOHJeg+DlK0OmZ/Pmgphtm\n\tpeKg4IV41muh1wvQc84ud0ERyM93PLfOit7Dn8sur2FhOhS896ENlhgMLzphApqvg5\n\tPgI99ZACzBVALp/Iu8vEZtF00nzDoyJOvIe1jlxsyCoj29ku3+S0sP3Y0A0Eo1pZG1\n\tTaAMkXr2wE3DQ==","v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1665149371;\n\tbh=Ecm7KZdPYTpIpZoZ+YxKJ8Y2tKRG7HXIl+eJC4QmDJA=;\n\th=Date:Subject:To:References:From:In-Reply-To:From;\n\tb=hHc+iiujTtEcf7EAYsH+Qu75yPWIQwtqUiRbPWo6Cuuu7fBqKaY0Mm5lcDgTRbQsc\n\t6GPJCvpPjbSQDrVJolLzW/OcuvlmtuBr+xhXuzk38Uv7lugErG/847qWMEgyQEBawQ\n\tmTR2J95+2p/eVtC7wsfRce7SP53KbethBap3exx0="],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key; \n\tunprotected) header.d=ideasonboard.com\n\theader.i=@ideasonboard.com\n\theader.b=\"hHc+iiuj\"; dkim-atps=neutral","Message-ID":"<3e7045b5-9b3f-4a13-68f3-686e7cb9114e@ideasonboard.com>","Date":"Fri, 7 Oct 2022 18:59:25 +0530","MIME-Version":"1.0","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101\n\tThunderbird/102.2.1","Content-Language":"en-US","To":"Laurent Pinchart <laurent.pinchart@ideasonboard.com>,\n\tlibcamera-devel@lists.libcamera.org","References":"<20221005193944.16383-1-laurent.pinchart@ideasonboard.com>\n\t<20221005193944.16383-2-laurent.pinchart@ideasonboard.com>","In-Reply-To":"<20221005193944.16383-2-laurent.pinchart@ideasonboard.com>","Content-Type":"text/plain; charset=UTF-8; format=flowed","Content-Transfer-Encoding":"7bit","Subject":"Re: [libcamera-devel] [PATCH 1/2] test: Rename 't' to 'test' in\n\tmeson.build","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"<libcamera-devel.lists.libcamera.org>","List-Unsubscribe":"<https://lists.libcamera.org/options/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=unsubscribe>","List-Archive":"<https://lists.libcamera.org/pipermail/libcamera-devel/>","List-Post":"<mailto:libcamera-devel@lists.libcamera.org>","List-Help":"<mailto:libcamera-devel-request@lists.libcamera.org?subject=help>","List-Subscribe":"<https://lists.libcamera.org/listinfo/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=subscribe>","From":"Umang Jain via libcamera-devel <libcamera-devel@lists.libcamera.org>","Reply-To":"Umang Jain <umang.jain@ideasonboard.com>","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>"}}]