[Ctk-developers] QtTesting in master

[Sascha Zelzer]

Hi,

+1 one from me (but see my comments below)


Thanks for putting so much effort into this, it looks really good! (I 
just skimmed through the code changes...)

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:

https://github.com/commontk/CTK/compare/master...qttesting#L11L724

and also remove that file:

https://github.com/commontk/CTK/compare/master...qttesting#diff-10


Thanks for your work,

Sascha


On 05/10/2012 11:23 PM, Julien Finet wrote:
> Hi CTKers,
>
> We have been working toward integrating the QtTesting framework [1] 
> into CTK. Kudos to Benjamin for his hard work !
>
> The Qt testing framework is an effort from the ParaView team which now 
> resides in its own repository [2a][2b].
> 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)
>
> It is still work in progress and more work will happen in the coming 
> weeks [4]
> However we would like to merge the branch into CTK master in order to 
> keep working on other CTK issues for Slicer.
> Indeed Slicer currently points to the QtTesting head and we'd like to 
> go back to pointing on master.
>
> Does anyone think we shouldn't merge or has any comment?
>
> Thanks,
> Julien.
>
> [1] http://paraview.org/Wiki/Testing_design
> [2a]http://paraview.org/gitweb?p=QtTesting.git
> [2b] https://github.com/commontk/QtTesting
> [3] https://github.com/commontk/CTK/tree/qttesting
> [4] 
> 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> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/ctk-developers/attachments/20120512/3f9dcd83/attachment.html>