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.