<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
Hi,<br>
<br>
+1 one from me (but see my comments below)<br>
<br>
<br>
Thanks for putting so much effort into this, it looks really good!
(I just skimmed through the code changes...)<br>
<br>
There is one small issue though: In the current CTK branch [3] you
seem to have some left-overs from the past were you referenced
QtTesting as an external project. If I understood the changes
correctly, you should probably remove this line:<br>
<br>
<a class="moz-txt-link-freetext" href="https://github.com/commontk/CTK/compare/master...qttesting#L11L724">https://github.com/commontk/CTK/compare/master...qttesting#L11L724</a><br>
<br>
and also remove that file:<br>
<br>
<a class="moz-txt-link-freetext" href="https://github.com/commontk/CTK/compare/master...qttesting#diff-10">https://github.com/commontk/CTK/compare/master...qttesting#diff-10</a><br>
<br>
<br>
Thanks for your work,<br>
<br>
Sascha<br>
<br>
<br>
On 05/10/2012 11:23 PM, Julien Finet wrote:
<blockquote
cite="mid:CAKoWPPGAoZbopMbY741Fw2m75De=ta0qA2xxHgPweQNSkgaPXg@mail.gmail.com"
type="cite">Hi CTKers,
<div><br>
</div>
<div>We have been working toward integrating the QtTesting
framework [1] into CTK. Kudos to Benjamin for his hard work !</div>
<br class="Apple-interchange-newline">
<div>The Qt testing framework is an effort from the ParaView team
which now resides in its own repository [2a][2b]. </div>
<div>Our integration work is currently in a QtTesting branch [3]
in CTK and is currently in use by Slicer (builds and package
fine on all our dashboard machines)</div>
<div><br>
</div>
<div>It is still work in progress and more work will happen in the
coming weeks [4] </div>
<div>However we would like to merge the branch into CTK master in
order to keep working on other CTK issues for Slicer. </div>
<div>Indeed Slicer currently points to the QtTesting head and we'd
like to go back to pointing on master.</div>
<div><br>
</div>
<div>Does anyone think we shouldn't merge or has any comment?</div>
<div><br>
</div>
<div>Thanks,</div>
<div>Julien.</div>
<div><br>
</div>
<div>
<div>[1] <a moz-do-not-send="true"
href="http://paraview.org/Wiki/Testing_design">http://paraview.org/Wiki/Testing_design</a></div>
<div>[2a]<span
style="color:rgb(102,102,102);font-family:Helvetica,arial,freesans,clean,sans-serif;font-size:13px;line-height:20px;background-color:rgb(250,250,250)"> <a
moz-do-not-send="true"
href="http://paraview.org/gitweb?p=QtTesting.git">http://paraview.org/gitweb?p=QtTesting.git</a> </span></div>
<div>[2b] <a moz-do-not-send="true"
href="https://github.com/commontk/QtTesting">https://github.com/commontk/QtTesting</a></div>
<div>[3] <a moz-do-not-send="true"
href="https://github.com/commontk/CTK/tree/qttesting">https://github.com/commontk/CTK/tree/qttesting</a></div>
<div>[4] <a moz-do-not-send="true"
href="https://github.com/commontk/CTK/issues?direction=desc&labels=Testing&page=1&sort=created&state=open">https://github.com/commontk/CTK/issues?direction=desc&labels=Testing&page=1&sort=created&state=open</a>
</div>
</div>
</blockquote>
<br>
</body>
</html>