From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id E3D9FA0597; Sat, 11 Apr 2020 00:23:01 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 0EC9B1D159; Sat, 11 Apr 2020 00:23:01 +0200 (CEST) Received: from mail-pf1-f196.google.com (mail-pf1-f196.google.com [209.85.210.196]) by dpdk.org (Postfix) with ESMTP id B95541C2F1 for ; Sat, 11 Apr 2020 00:22:59 +0200 (CEST) Received: by mail-pf1-f196.google.com with SMTP id b8so1592345pfp.8 for ; Fri, 10 Apr 2020 15:22:59 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20150623.gappssmtp.com; s=20150623; h=date:from:to:cc:subject:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=SS35S0vmJe+X5fOXA/rqMtxqxt6pegIJiUEiRQ8Q4KA=; b=CocAzVuW2QKsfRkIdC3UI1Cs6LecvClq/RSxMvXhT8kod0kkEFhjGBGoKoRW9u9e3S SD9RWuItxPP8Ioc516D1VT8HUEKgI6Mb6ZHF+473BQjEhvD0DHHBgUx9dP2uEz5iiVJj nHuyTi8ZMd3A9LVf9X4yvJbltbW2sBpH9K0t8nrsH5kehIs4TQehH64OlGhUSt3WOT0Z GyGievDfUxU/jtz2RUIU97cqb1vAiCMU1wTBHUMry+8+WI4HLBohIvPkG2dKBJNANxeA KwaH1w1Z+jTmEZusrNVIji+FVyfEC/KvAXKefXeOTSqlu1vmTqXE77z09/TjNKmSivyD JQNQ== 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=SS35S0vmJe+X5fOXA/rqMtxqxt6pegIJiUEiRQ8Q4KA=; b=AwwAteEYqM9/Vf4kUPmOwKLK087c1v7TWMzzgEqLnbkF6fw+3ihHeCDdSEjPsv5A+n RRPWOKkQ5sqQ3guDG/DPBoDByOdBXJmsQrKEh+CeEqGbV7/mdEMzD6KOY9yypFsjvDZm 5xF5gzJ2j5P9j8wR9rXJZ0DvbNkRGkt9gy0iioP6HXiHNcLKmArrkrQC3mKL/w/vIflR IiEwQ174X8BpPEXzu7Gp+jjpgwWYOKid0qBiLwcewbwmOJdMWvLtVTJlcobNqH4qGA12 Y7PYBXvYqJshWgACJBzpgo9nhDJbA7nAHEznZ6bgR93Lf0gbr1GQCe3fTtpfz1645EGd 3EsQ== X-Gm-Message-State: AGi0PubBtTGr4D6bMtLxdkJuGQG5eIkwVp9Ka9A9tx2AwplLLXKAXisp ds4ssDmsW2ZeaLXgeNFm031BdQ== X-Google-Smtp-Source: APiQypJs99QFSObdphzSLTPxYPhTTq9Tm5fWxoRws7nLGyB8tv4fXUpAVKLBMvEPKRYrsNfj2DPgvw== X-Received: by 2002:aa7:9a89:: with SMTP id w9mr6937479pfi.149.1586557378831; Fri, 10 Apr 2020 15:22:58 -0700 (PDT) Received: from hermes.lan (204-195-22-127.wavecable.com. [204.195.22.127]) by smtp.gmail.com with ESMTPSA id w142sm2662287pff.111.2020.04.10.15.22.58 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 10 Apr 2020 15:22:58 -0700 (PDT) Date: Fri, 10 Apr 2020 15:22:55 -0700 From: Stephen Hemminger To: Lukasz Wojciechowski Cc: Thomas Monjalon , Bruce Richardson , dev@dpdk.org, Aaron Conole , Ruifeng Wang , David Marchand , Gavin Hu Message-ID: <20200410152255.4565de32@hermes.lan> In-Reply-To: References: <20200409180332.592074-1-thomas@monjalon.net> <1813726.1xdlsreqCQ@thomas> <20200410132547.GB42@bricha3-MOBL.ger.corp.intel.com> <1933780.kXSN5OTJKJ@thomas> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [dpdk-dev] [PATCH v3] test: remove meson dependency on /proc file X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 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" On Fri, 10 Apr 2020 22:47:44 +0200 Lukasz Wojciechowski wrote: > W dniu 10.04.2020 o=C2=A016:41, Thomas Monjalon pisze: > > 10/04/2020 15:25, Bruce Richardson: =20 > >> On Fri, Apr 10, 2020 at 02:27:30PM +0200, Thomas Monjalon wrote: =20 > >>> 10/04/2020 12:42, Bruce Richardson: =20 > >>>> On Fri, Apr 10, 2020 at 12:29:50PM +0200, Thomas Monjalon wrote: =20 > >>>>> --- a/app/test/meson.build > >>>>> +++ b/app/test/meson.build > >>>>> has_hugepage =3D true > >>>>> if is_linux =20 > >>>> Since you check for linux in the script, you can drop these two line= s, =20 > >>> The issue is for Windows. > >>> I am not sure how we will skip shell scripts > >>> when adding Windows support for this application. > >>> So there are two options now: > >>> a) remove Linux check before calling the script and ignore Windows s= upport for now > >>> b) keep Linux check without knowing whether it will be useful for Wi= ndows support > >>> > >>> We vote a? > >>> =20 > >> c) Write all such scripts in python, to allow them to run everywhere. = :-) > >> > >> Given that windows is the problem, having the is_linux check in the > >> meson.build file makes most sense - in which case I don't think we nee= d the > >> check for linux in the script. =20 > > Yes, let's make it simple for now. > > =20 >=20 > a) is ok as the current meson.build won't work in Windows environment at= =20 > all. > c) would be best, but I guess it's a little bit to much for this patch=20 > and the whole build system could be adjusted to windows in separate threa= d. >=20 > And I agree with Bruce, that the check for Linux in script is not needed. >=20 > > =20 Building in a container may not give access to /proc.