Hi Andy,

On Monday 12 November 2012 11:41 PM, Andy Doan wrote:
I just spent a little time looking at the branch and have a few comments:

I just made sure all your comments are addressed in revision 448. Please have a look at it. The new job file which I used to test this is attached.

<snip>
  The following changes are incorporated as per Andy's review comments:

  1) Remove testdefX
  2) Remove specifying 'files'
  3) Do not guess git/bzr repos
4) Get rid of .get() syntax for accessing dictionary keys whereever possible
  5) Split code in _configure_target
6) Copy all the files required for test to target directory (mwhudson) - no symlinks
</snip>

Thank You.
--
Senthil Kumaran S
http://www.stylesen.org/
http://www.sasenthilkumaran.com/

Attachment: qemu-new.json
Description: application/json

_______________________________________________
linaro-validation mailing list
[email protected]
http://lists.linaro.org/mailman/listinfo/linaro-validation

Reply via email to