More patches

BogDan bog_dan_ro at yahoo.com
Wed Sep 21 18:45:00 UTC 2011


Hi Daniel,

I check your patches almost all of them are ok, the last one I think is a misunderstanding, AndroidDeployConfiguration doesn't have any configuration which needs to be saved, I'll change it. 
I also tried to merge master branch but I had some problems, it seems you've made some changes to debugging plugin, so, I'll try tomorrow (night) to fix them and to test QtCreator then I'll push the changes.

Thank you !

Cheers,
BogDan.




>________________________________
>From: Daniel Teske <daniel.teske at nokia.com>
>To: necessitas-devel at kde.org
>Sent: Thursday, September 15, 2011 4:52 PM
>Subject: Re: More patches
>
>Hi,
>
>some more patches for your review.
>
>0001:
>Returning mobile from the targetFeatures essentially just switches for which 
>new project templates the target is enabled by default. The Qt Quick 
>Application Wizard preselects all mobile targets, whereas the qt gui wizard 
>enables desktop targets. As such returning mobile is probably a better choice.
>
>0002:
>There have been a few chanegs to the TargetFactory since the 
>AndroidTargetFactory was written, I have removed some no longer needed 
>virtuals and added a missed one.
>
>0003:
>As part of the RemoteLinux refactoring we moved the DeployConfigurationFactory 
>from the being attached to the target to the object pool. This patch makes the 
>same change to the Android DeployConfigurationFactory.
>
>0004:
>Just a unused member
>
>0005:
>This adds a AndroidDeployConfiguration class and readds save and restore code 
>to the AndroidDeployStep. Thus saving and restoring the settings for that step 
>now works for me.
>
>daniel
>
>_______________________________________________
>Necessitas-devel mailing list
>Necessitas-devel at kde.org
>https://mail.kde.org/mailman/listinfo/necessitas-devel
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/necessitas-devel/attachments/20110921/488eef13/attachment.html>


More information about the Necessitas-devel mailing list