From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id B6C92A0C57; Sun, 29 Aug 2021 00:18:15 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id A48D04068B; Sun, 29 Aug 2021 00:18:15 +0200 (CEST) Received: from mail-lf1-f42.google.com (mail-lf1-f42.google.com [209.85.167.42]) by mails.dpdk.org (Postfix) with ESMTP id 9E2AF4068A for ; Sun, 29 Aug 2021 00:18:14 +0200 (CEST) Received: by mail-lf1-f42.google.com with SMTP id j4so22496707lfg.9 for ; Sat, 28 Aug 2021 15:18:14 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=date:from:to:cc:subject:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=t9wzE3t5Ur9MpP+JrFO4JZ1vWb8WdfaaQ2KaInarue8=; b=RznwI7RRbS2jUwITXRSYQaKWIg2rlP+CIAleEWQbOTxzK1pF+wRiJxt2Zbi9jeXFTo yhUDSrBVvcocQzgS+Kwv4TmHXPwp5bCf2xYf3HUNMEoc5AvD2kb9lQ2/5QCuXxsoVUlW M3ox2OiHZC5cCqkTXsHbLZIAUL8c2INvmLzQ4BYnxN3mf19Be9h+Vo7aCNdwcQJjmEhm kgT7U/UlJRINIp627+gP5odasc+fUJJt0kycv15Y/v7qLfPHYhvMREKZWRQc3nZoquPt HFE5RZn8cyCI0pB+QaF4rhOJgnP2idsf0KnrAsNhCim4B4TX1VqwGxxiOMynfE6N18ig hEGQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=t9wzE3t5Ur9MpP+JrFO4JZ1vWb8WdfaaQ2KaInarue8=; b=Cdc2NOaAgTECBfEoYofISxft6Jt9zCAzH8zZd3qOGS98p400XgVKqFy7zjYc9+/ixT BaJwCbjRFQI96Pq46/SpeWsKWi9SpdHXfOEMMwjMvkMaO3ppdtvrgX3Pu/gYedevN4sk dJphcb1I77q7JKFLLpqDegCGiCaJwwJQpCp93vaFVXrFs9CkRPVqBydYqRNU8+Iuggjj C3ygB07eO/VHc7ApL8rZRR0gJ4hmFe3m/peahxSInS6et380CgXvqo0Wbji0tZ6C+RCx uf6G+mgXwuLfwR/4f6pannWcnRR/u8NZBpeunhchlrf2trInS7EYbTGOBbXkV7gT5jZQ ST8Q== X-Gm-Message-State: AOAM532h3XFP0exyyvqUDPDtrMF0lWhj81CCFlgmNlidDdQpnvbNjnHf ocqWLohY08lF5iIvFqcdS98= X-Google-Smtp-Source: ABdhPJyWHny03gJkoaQgiXUeFlXr4oib/juAqS+/ZFEFefXkRUAqenG+/3uvZBw90AICed5uWkp/rA== X-Received: by 2002:ac2:434f:: with SMTP id o15mr11680244lfl.124.1630189094245; Sat, 28 Aug 2021 15:18:14 -0700 (PDT) Received: from sovereign (broadband-37-110-65-23.ip.moscow.rt.ru. [37.110.65.23]) by smtp.gmail.com with ESMTPSA id c10sm974572lfm.271.2021.08.28.15.18.13 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 28 Aug 2021 15:18:13 -0700 (PDT) Date: Sun, 29 Aug 2021 01:18:12 +0300 From: Dmitry Kozlyuk To: Jie Zhou Cc: dev@dpdk.org, roretzla@microsoft.com, navasile@linux.microsoft.com, dmitrym@microsoft.com, pallavi.kadam@intel.com, talshn@nvidia.com, thomas@monjalon.net, aconole@redhat.com Message-ID: <20210829011812.37662dbb@sovereign> In-Reply-To: <1629306834-6277-14-git-send-email-jizh@linux.microsoft.com> References: <1629267476-901-1-git-send-email-jizh@linux.microsoft.com> <1629306834-6277-1-git-send-email-jizh@linux.microsoft.com> <1629306834-6277-14-git-send-email-jizh@linux.microsoft.com> X-Mailer: Claws Mail 3.17.8 (GTK+ 2.24.32; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] [PATCH v2 13/13] app/test: enable subset of unit tests on Windows X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 2021-08-18 10:13 (UTC-0700), Jie Zhou: > In app/test/meson.build > - Specify the subset of source files to compile on Windows > - Specify the subset of tests to be enabled on Windows > Signed-off-by: Jie Zhou > --- > app/test/meson.build | 859 ++++++++++++++++++++++++++----------------- > 1 file changed, 515 insertions(+), 344 deletions(-) Commit messages should explain the reason for the change, which itself is precisely described by the diff below. For example, why these specific tests are selected? Please don't create two lists, it will be hard to maintain. > diff --git a/app/test/meson.build b/app/test/meson.build > index 7240304916..1e77657510 100644 > --- a/app/test/meson.build > +++ b/app/test/meson.build [...] > @@ -362,7 +533,7 @@ endif > if dpdk_conf.has('RTE_EVENT_SKELETON') > test_deps += 'event_skeleton' > endif > -if dpdk_conf.has('RTE_LIB_TELEMETRY') > +if not(is_windows) and dpdk_conf.has('RTE_LIB_TELEMETRY') > test_sources += ['test_telemetry_json.c', 'test_telemetry_data.c'] > fast_tests += [['telemetry_json_autotest', true], ['telemetry_data_autotest', true]] > endif "not" is not a function, "if not is_windows and ..." is the preferred style.