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 E6AEFA04C9; Mon, 14 Sep 2020 10:08:43 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 4E42F2BAB; Mon, 14 Sep 2020 10:08:43 +0200 (CEST) Received: from out4-smtp.messagingengine.com (out4-smtp.messagingengine.com [66.111.4.28]) by dpdk.org (Postfix) with ESMTP id 1F349FFA for ; Mon, 14 Sep 2020 10:08:42 +0200 (CEST) Received: from compute7.internal (compute7.nyi.internal [10.202.2.47]) by mailout.nyi.internal (Postfix) with ESMTP id ACC045C00E2; Mon, 14 Sep 2020 04:08:40 -0400 (EDT) Received: from mailfrontend2 ([10.202.2.163]) by compute7.internal (MEProxy); Mon, 14 Sep 2020 04:08:40 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h= from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding:content-type; s=fm2; bh= 4LJffvQlJ/cXTmQBwVWp7uYDuYVNABRe3fuYmViyJAA=; b=GFZPt6sJ5vePdd3N 1suhTM5pi0gw9Xq0fv51xUe23OcaLTyR5YPK8HiyW4rgV1WLJcR2rQWCP25Kvvw1 JnoTLQxKkcoopPgb2c/kv230S3Z9j1wYvK078EEtqR9g3E4O6p5fc68ChnuzR1Ch 9CHFpj79qx9qvwcpRAS5ZEP22k+dZk4fnLAtYQw3SKrb68T9LZSQ57wi7lTFakTF mpc6VgqobhE0P9Bbfo3bd+KUJIfTmapPImvWkRlyWG7sbsD2DyKt+e0wRkqhNsIv +3WNyyoZIJ2L7r7N3p1UofxhqgzjNH5DmB0x6FovaSGlNfv+Irit3spdmwz7fQj7 vOpzcw== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm3; bh=4LJffvQlJ/cXTmQBwVWp7uYDuYVNABRe3fuYmViyJ AA=; b=OIxlfOfKGnSf0V9hBp0fALxOZuuucVE/LsP5hBehg4KWTVXPshndGYmj/ KZFaUZPyH2Uyx4QezUcEigYs7rZ+i8G/oLhDpKHh7txc2JXKEjp63KVi3AUX/i74 egWKrdzUCF0BVrgM5oGLeNl5WRmEv+KkuNzgCBL5gNneLpFXqT6sU2KR+btvwMfg LsVU/z+9VxUg40slGzTa206MA3gq1hPxXBRmFBopoGexvqPnuCZayS999DdvIBFW X4q7RGqecu4PdHJtk6VYdaVHPIL7vD3d0ZN1xZW5itqTD5jYtvB378o4TAmdzcb6 OFY87tCVRFMTb1YFxw8HlpAfCxoTQ== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeduiedrudeiiecutefuodetggdotefrodftvfcurf hrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfghnecuuegr ihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenucfjug hrpefhvffufffkjghfggfgtgesthfuredttddtvdenucfhrhhomhepvfhhohhmrghsucfo ohhnjhgrlhhonhcuoehthhhomhgrshesmhhonhhjrghlohhnrdhnvghtqeenucggtffrrg htthgvrhhnpedugefgvdefudfftdefgeelgffhueekgfffhfeujedtteeutdejueeiiedv ffegheenucfkphepjeejrddufeegrddvtdefrddukeegnecuvehluhhsthgvrhfuihiivg eptdenucfrrghrrghmpehmrghilhhfrhhomhepthhhohhmrghssehmohhnjhgrlhhonhdr nhgvth X-ME-Proxy: Received: from xps.localnet (184.203.134.77.rev.sfr.net [77.134.203.184]) by mail.messagingengine.com (Postfix) with ESMTPA id 146433064674; Mon, 14 Sep 2020 04:08:38 -0400 (EDT) From: Thomas Monjalon To: Conor Walsh Cc: dev@dpdk.org, david.marchand@redhat.com, ray.kinsella@intel.com, nhorman@tuxdriver.com, aconole@redhat.com, maicolgabriel@hotmail.com, bruce.richardson@intel.com Date: Mon, 14 Sep 2020 10:08:37 +0200 Message-ID: <6873366.jM3skLiTAc@thomas> In-Reply-To: <20200911160332.256343-1-conor.walsh@intel.com> References: <20200910142121.3995680-1-conor.walsh@intel.com> <20200911160332.256343-1-conor.walsh@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Subject: Re: [dpdk-dev] [PATCH v3 0/4] abi breakage checks for meson 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" Hi, > This patchset allows developers to check ABI breakages during build > time. > Currently checking that the DPDK ABI has not changed before up-streaming > code is not intuitive. The current method, requires the contributor to > use either the test-build.sh and test-meson-build.sh tools, along side The contributor *MUST* test compilation with test-meson-build.sh in any case. > some environmental variables to test their changes. Contributors in many It is just one variable to add to a file: export DPDK_ABI_REF_VERSION=v20.11 in ~/.config/dpdk/devel.config or dpdk/.develconfig > cases are either unaware or unable to do this themselves, leading to a > potentially serious situation where they are unknowingly up-streaming > code that breaks the ABI. These breakages are then caught by Travis, but > it is more efficient if this is caught locally before up-streaming. I think you are proposing a complex solution to a non-issue. And I don't understand how your method is more straight-forward for the user, given this statement in your last patch: " This patch adds the ability to run ABI breakage checks to meson. To do this the developer needs to set the meson build type to debug and set the version of DPDK that they want to check the ABI against. "