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 614A1A0508; Thu, 7 Apr 2022 07:05:16 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 0C11F40689; Thu, 7 Apr 2022 07:05:16 +0200 (CEST) Received: from mail-io1-f42.google.com (mail-io1-f42.google.com [209.85.166.42]) by mails.dpdk.org (Postfix) with ESMTP id 3D1CF4014F for ; Thu, 7 Apr 2022 07:05:14 +0200 (CEST) Received: by mail-io1-f42.google.com with SMTP id g21so5486468iom.13 for ; Wed, 06 Apr 2022 22:05:14 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc:content-transfer-encoding; bh=kC3v7mOnFPNIQ1WUvdBlNKbUaKhSYQOMDsyDtWBDbfw=; b=QvZxlZovoIWXAB6MuehIXb9RsDnDIonDXplhXm3VyiCIUNpVdTY9dhnIFgpBpBCtMP OXNGKwNS8CpesBa5fWr0MfvU/fJstYsgUz4crvcrd/TlK6WUGT+UNBD519FW9CNRr7z6 f90I3ewiwEWIUvD7dP3MB8ZomzuFtd+A4NS1cXoy8cRxFu5qKPayaxTS9pEypUNd9xek i3s/YsltTNxlgR/neumtyQJf2VcFS6pNCvG4e0avlGKY9wu9/2m1BYp/jFX8Uo8pPUjU 5Xfo8ckSOpEZ3EilM7MCrgVUMt9DP3eZXXIT7ZD1G14qc9RiqcXrlGMhK1KQr2bVzG4k jVOA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc:content-transfer-encoding; bh=kC3v7mOnFPNIQ1WUvdBlNKbUaKhSYQOMDsyDtWBDbfw=; b=iJxXXZnPrPhpCchVo5L5yuIbcdDn/891njBx4RMwBaKMbV8dEmihUAM0qzPbWhndyW +ziAiUqgw1LD08glUemhqeAnbMuIa6IUfPHZFiXcEyycFNBz4HwpY0sdVOxqvK/YUx2g FEhLfkAKzsQi/ZKg6/2uDVsKpScbJkDoqjNyPSxV4OiHZF7rnN9SVrrpJ8iZVF7kvtSl pfBWfWGzpxv64ZzCGH5W4ita2EQr7nO37nCoh8zF/YuVTm5MnzMdbybFWky+aGGpH0HX qS+3tRpnL5YlNkxXW+Ue8jXpQXmxfe7HH3fQS1a56KqlBSRvjZ7jLrSo8rdEosnPZNgv VCtA== X-Gm-Message-State: AOAM5329W+Ny7zOqt5GPLqteX9W/KPWodREanYKYvJpK5w5FfQmyQ+Sq UBsUeV9VhZa9YnC4rjnNbVeNfvVYTtNxnvfZbII= X-Google-Smtp-Source: ABdhPJycuOhWtUn37qUAkBlkCDl+SfnRhRhquG+WZwOh4Ji26NmVdgdaxDpMcucMdLK1oBssufnMXGKfUJLuS+kuFKo= X-Received: by 2002:a02:604f:0:b0:30f:e6f1:3883 with SMTP id d15-20020a02604f000000b0030fe6f13883mr6661911jaf.266.1649307913555; Wed, 06 Apr 2022 22:05:13 -0700 (PDT) MIME-Version: 1.0 References: <20220406145606.2913834-1-juraj.linkes@pantheon.tech> In-Reply-To: <20220406145606.2913834-1-juraj.linkes@pantheon.tech> From: Jerin Jacob Date: Thu, 7 Apr 2022 10:34:47 +0530 Message-ID: Subject: Re: [RFC PATCH v1 00/15] merge DTS core files to DPDK To: =?UTF-8?Q?Juraj_Linke=C5=A1?= Cc: Thomas Monjalon , David Marchand , Honnappa Nagarahalli , ohilyard@iol.unh.edu, "Tu, Lijuan" , dpdk-dev Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable 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 On Wed, Apr 6, 2022 at 8:26 PM Juraj Linke=C5=A1 wrote: > > These are the basic libraries that other libraries depend on. There's > also the basic framework functionality related to test execution. > > Juraj Linke=C5=A1 (15): > dts: merge DTS dep/tclclient.tgz to DPDK > dts: merge DTS dep/tgen.tgz to DPDK > dts: merge DTS dts to DPDK > dts: merge DTS framework/__init__.py to DPDK > dts: merge DTS framework/asan_test.py to DPDK > dts: merge DTS framework/checkCase.py to DPDK > dts: merge DTS framework/dts.py to DPDK > dts: merge DTS framework/exception.py to DPDK > dts: merge DTS framework/logger.py to DPDK > dts: merge DTS framework/packet.py to DPDK > dts: merge DTS framework/project_dpdk.py to DPDK > dts: merge DTS framework/serializer.py to DPDK > dts: merge DTS framework/utils.py to DPDK > dts: merge DTS main.py to DPDK > dts: merge DTS version.py to DPDK merge->import > > dts/dep/tclclient.tgz | Bin 0 -> 199327 bytes > dts/dep/tgen.tgz | Bin 0 -> 134392 bytes Some top level comments: - I think, we should not check in binary files. - git commit comment should much more than "dts: merge DTS xxxx to DPDK" wh= ere the commit log should have details on check in. -Add the documentation from the first patch and update the documentation per patch based on the content.