From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by dpdk.org (Postfix) with ESMTP id 5227C5A6B for ; Wed, 4 Nov 2015 05:48:11 +0100 (CET) Received: from orsmga003.jf.intel.com ([10.7.209.27]) by fmsmga101.fm.intel.com with ESMTP; 03 Nov 2015 20:48:10 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.20,241,1444719600"; d="scan'208";a="677925221" Received: from stv-crb-56.sh.intel.com (HELO [10.239.128.116]) ([10.239.128.116]) by orsmga003.jf.intel.com with ESMTP; 03 Nov 2015 20:48:09 -0800 Message-ID: <56398E66.1050407@intel.com> Date: Wed, 04 Nov 2015 12:49:42 +0800 From: "Liu, Yong" User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.5.0 MIME-Version: 1.0 To: Lijuan Tu , dts@dpdk.org References: <1446611202-37081-1-git-send-email-lijuanx.a.tu@intel.com> In-Reply-To: <1446611202-37081-1-git-send-email-lijuanx.a.tu@intel.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [dts] [patch] physmem_dump : remove match "\n" send_expect will remove the end "\r\n", so it match failed if only one string X-BeenThere: dts@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: test suite reviews and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 04 Nov 2015 04:48:11 -0000 Applied. Please separate subject and commit log. Mail subject should be the description of this patch. The detail logs should be in the mail content. On 11/04/2015 12:26 PM, Lijuan Tu wrote: > Signed-off-by: Lijuan Tu > --- > tests/TestSuite_unit_tests_dump.py | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/tests/TestSuite_unit_tests_dump.py b/tests/TestSuite_unit_tests_dump.py > index 2052cb4..8b094b5 100644 > --- a/tests/TestSuite_unit_tests_dump.py > +++ b/tests/TestSuite_unit_tests_dump.py > @@ -129,7 +129,7 @@ class TestUnitTestsDump(TestCase): > match_regex = "Segment (\d)+:" > for element in elements[1:-1]: > match_regex += " %s:(.*?)," % element > - match_regex += " %s:(.*?)\n" % elements[-1] > + match_regex += " %s:(.*?)" % elements[-1] > m = re.compile(r"%s" % match_regex, re.DOTALL) > results = m.findall(out) > phy_info = []