Hi Hari,
On Sat, 2013-03-23 at 00:52 +0530, Harigopal Gollamudi wrote:
please find the attached patch. Kindly review this and update me
I went through your new patch. What I found was, the new pattern you have put in does not relate to the old parse pattern!
- pattern: "^(?P<test_case_id>*?)\s+(?P<measurement>[\d.]+)\s +(?P<units>\S+)\s+(?P<result>(pass|fail))\s*$" + pattern: "^(?P<test_case_id>\S*\s*\S*)\s+(?P<measurement>\d +.\d+)\s+(?P<units>\S+)\s+(?P<result>\S+)$"
The new change is not the equivalent of the old and it is not correct (I am concerned about the regex here). I request you to fix the above and run a sample job in validation.linaro.org with the 0xbench yaml file. Please include link to the sample run in your next email.
Thank You.