On Thu, 2008-02-28 at 16:34 +0530, Subrata Modak wrote:
> On Thu, 2008-02-28 at 16:21 +0530, Dhaval Giani wrote:
> > On Thu, Feb 28, 2008 at 10:05:21AM +0530, Sudhir Kumar wrote:
> > > Hi
> > > 
> > > This patch creates the framework for controllers testcases in LTP. This 
> > > also
> > > creates the first and second testcase for cpu controller and adds some
> > > documentation.
> > > 
> > 
> > Hi Sudhir,
> > 
> > This is a very big patch, you might want to split it into 3 different
> > patches, one for the framework and one for each testcase.
> 
> Thanks Dhaval for reviewing and providing comments.
> 
> Sudhir,
> 
> We will wait for all review comments from Dhaval and then you can make
> the necessary changes. Let Dhaval finally ack all your changes after
> which we can integrate this to LTP. Let cpu controller test cases go
> during March 2008 release of LTP.
> 
> Also, one comment i have is, you might have missed to include the
> Copyright and License declarations in few of the source code. Just check
> them whenever you are re-sending worked out patches.

Well Sudhir,

I decided to revoke the decision by myself, keeping in mind the fact
delay in release can cause for testing. So, i have decided to include
your PATCHES in this release itself. Since Dhaval did not mention major
technical anomaly(s)for the same, i find it prudent to go ahead in
including the same in this release. All your 17 patches has been duly
merged and you will see them in tomorrowÅ› release.

Please note down all DhavalÅ› review comments and include them in your
next round of cleanups, either from you or from somebody else. Thanks
again for these new test cases. Expecting more from you in future.

Regards--
Subrata

> 
> Regards--
> Subrata
> 
> > 
> > Minor comments follow,
> > 
> > > <cpuctl1.patch>
> > > 
> > > Signed-off-by: Sudhir Kumar <[EMAIL PROTECTED]>
> 
> 
> -------------------------------------------------------------------------
> This SF.net email is sponsored by: Microsoft
> Defy all challenges. Microsoft(R) Visual Studio 2008.
> http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
> _______________________________________________
> Ltp-list mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/ltp-list


-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list

Reply via email to