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 EF217A04F3 for ; Tue, 7 Jan 2020 19:11:12 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id B4DDF1D551; Tue, 7 Jan 2020 19:11:12 +0100 (CET) Received: from out3-smtp.messagingengine.com (out3-smtp.messagingengine.com [66.111.4.27]) by dpdk.org (Postfix) with ESMTP id 6692D1C195 for ; Tue, 7 Jan 2020 19:11:11 +0100 (CET) Received: from compute1.internal (compute1.nyi.internal [10.202.2.41]) by mailout.nyi.internal (Postfix) with ESMTP id A95BF21F48; Tue, 7 Jan 2020 13:11:10 -0500 (EST) Received: from mailfrontend2 ([10.202.2.163]) by compute1.internal (MEProxy); Tue, 07 Jan 2020 13:11:10 -0500 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=mesmtp; bh=0o8nr5UfWB0IwF7nZFTEViPefy6/cYerQqO8GYrCItM=; b=hno1IsLfhTNX KTjc2B8MR5dmlOwNcHiSvLMbD9F5i0NsVH5wlmbOmPajvWWldx471DVIoI99+64W jKG+OuzvUmx89LwrxGNgL1Y9o5HZN8CTk2eAlB8HRKHQ6yB87TzZdL0BF3Qbo4kW pgopChfgY+gxPb2CexP34rzNj7Cq7kk= 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=fm1; bh=0o8nr5UfWB0IwF7nZFTEViPefy6/cYerQqO8GYrCI tM=; b=mfKGQGR932Hi0mYRYYGSKZEPSy42mSWz623hEuY100mFecBHGB91CNXyt CfmlIC4n4T9lG7j7FvOTf/Zd+UrqXh30eNAmVDJVkRDCtFYvV73WSm5TNs876JxJ x0kfFS11jk3DNAOw2eHnzy9EmQz5hWpmz9pTieKd3GH58nrWL94zee2y91aRw1ip NAuW5+HPKcy+UcpzZtDYvzYJ/O0hI4KAxXlLlWBL01RgjR7y1mbgWEpw2ep3NxUK kook49G8QOjgGKym3IBiJCFghTcqSjbdXeUqdOZwbmDLhg9gkVf7ygY6kJTAauPH MkuBL5nP12aT81VWXyY4QB4sSSMKA== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedufedrvdehhedgtdejucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhephffvufffkfgjfhgggfgtsehtufertddttddvnecuhfhrohhmpefvhhhomhgr shcuofhonhhjrghlohhnuceothhhohhmrghssehmohhnjhgrlhhonhdrnhgvtheqnecuff homhgrihhnpeguphgukhdrohhrghdpohhuthhlohhokhdrtghomhdpgedtmhgvlhhlrghn ohigrdgtohhmnecukfhppeejjedrudefgedrvddtfedrudekgeenucfrrghrrghmpehmrg hilhhfrhhomhepthhhohhmrghssehmohhnjhgrlhhonhdrnhgvthenucevlhhushhtvghr ufhiiigvpedt 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 CB08E30607B0; Tue, 7 Jan 2020 13:11:09 -0500 (EST) From: Thomas Monjalon To: David Marchand , Ali Alnubani Cc: "ci@dpdk.org" Date: Tue, 07 Jan 2020 19:11:07 +0100 Message-ID: <2510667.mvXUDI8C0e@xps> In-Reply-To: References: <20200106111531.26431-1-david.marchand@redhat.com> <34761032.2A8e1yFmEb@xps> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Subject: Re: [dpdk-ci] [PATCH] tools: fix tree detection for "unmaintained" files X-BeenThere: ci@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK CI discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: ci-bounces@dpdk.org Sender: "ci" 07/01/2020 11:08, Ali Alnubani: > Hi David, > Thanks for the fix. > > From: Thomas Monjalon > > 06/01/2020 12:15, David Marchand: > > > Implicitly, if a file has no official maintainer / tree, it is in the > > > end maintained in the master branch. > > > > > > This problem has been seen while looking at test failures reported on > > > the series > > https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatch > > work.dpdk.org%2Fproject%2Fdpdk%2Flist%2F%3Fseries%3D7981&data > > =02%7C01%7Calialnu%40mellanox.com%7C973da6c13b094a514bc008d792f7d > > 986%7Ca652971c7d2e4d9ba6a4d149256f461b%7C0%7C0%7C63713946551617 > > 2792&sdata=US%2FBhk2huEVZDPXCUTqgwKNqqMJFuL7oxd049pXKLe > > M%3D&reserved=0. > > > > > > This series mixes changes in ethdev, net/tap and EAL, and thus should > > > go through the main repository. > > > > > > Signed-off-by: David Marchand > > > --- > > > - if _tree: > > > - tree_list.append(_tree) > > > + # No identified tree for a file means that it should go through > > > + # the main repository. > > > + if not _tree: > > > + _tree = 'dpdk' > > > + tree_list.append(_tree) > > > > It looks good to me. > > This is the expected behaviour but it seems the case was forgotten when > > writing this function. > > > > Waiting for the review/ack from Ali before merging. > > Thanks > > > > Acked-by: Ali Alnubani Applied, thanks