From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jerin Jacob Subject: Re: [PATCH 1/2] test: store only the test case name Date: Mon, 20 Mar 2017 15:34:30 +0530 Message-ID: <20170320100429.jf2lpdpybb6tt4yl@localhost.localdomain> References: <1489930654-16827-1-git-send-email-jerin.jacob@caviumnetworks.com> <1489930654-16827-2-git-send-email-jerin.jacob@caviumnetworks.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: "dev@dpdk.org" , "thomas.monjalon@6wind.com" To: "Van Haaren, Harry" Return-path: Received: from NAM01-BN3-obe.outbound.protection.outlook.com (mail-bn3nam01on0073.outbound.protection.outlook.com [104.47.33.73]) by dpdk.org (Postfix) with ESMTP id E58791075 for ; Mon, 20 Mar 2017 11:04:47 +0100 (CET) Content-Disposition: inline In-Reply-To: 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 Mon, Mar 20, 2017 at 09:58:04AM +0000, Van Haaren, Harry wrote: > > From: Jerin Jacob [mailto:jerin.jacob@caviumnetworks.com] > > Sent: Sunday, March 19, 2017 1:38 PM > > To: dev@dpdk.org > > Cc: thomas.monjalon@6wind.com; Van Haaren, Harry ; Jerin > > Jacob > > Subject: [dpdk-dev] [PATCH 1/2] test: store only the test case name > > > > Store only the test case name in unit test case structure.The actor who > > renders the test status can add appropriate test status. This enables > > adding the new test case status without storing the additional > > information in the unit test case structure. > > > > Signed-off-by: Jerin Jacob > > --- > > test/test/test.c | 14 ++++++-------- > > test/test/test.h | 23 +++++++++-------------- > > 2 files changed, 15 insertions(+), 22 deletions(-) > > > > diff --git a/test/test/test.c b/test/test/test.c > > > It looks like the patch got corrupted somehow - the above line should be app/test/test.c, not test/test/test.c > Applying fails here, but with a s/test/app in the right places, the patches (this and next) both apply cleanly, and work well. > This handles the event/sw timeout_ticks() unsupported function as should so, > > Tested-by: Harry van Haaren > > > With the above apply issue resolved, also Recently there was a rework in test directory. Now test/test points to old app/test in dpdk-master.So no issues. http://dpdk.org/browse/dpdk/tree/test/test > > Acked-by: Harry van Haaren