Hi Senthil,
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.
--
Senthil Kumaran
http://www.stylesen.org/
http://www.sasenthilkumaran.com/