search

gavinpugh / vs-android

Integrated development of Android NDK C/C++ software with Microsoft Visual Studio.
Other
119 stars 34 forks source link

Ant deploy should be stored in user configuration, not the project file #72

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
While the ant library name and ant build type are project settings, whether to 
deploy, deploy and run, or do nothing on building is something that different 
users will want to set differently. This is a particular issue for a build 
machine which typically won't deploy to a device.

Therefore it would make sense to store the ant deploy setting in the 
vcxproj.user file.

Original issue reported on code.google.com by m...@rebellion.co.uk on 13 Mar 2012 at 12:39

GoogleCodeExporter commented 9 years ago 
I hacked this in for myself by adding the following after <EnumProperty 
Name="AntBuildType" ... /> in android_ant_build.xml
  <EnumProperty Name="AntDeploy" DisplayName="Deploy To Device" Category="General">
    <EnumValue Name="None" DisplayName="No Deploy" />
    <EnumValue Name="Deploy" DisplayName="Deploy Only" />
    <EnumValue Name="DeployAndRun" DisplayName="Deploy and Run" />
  </EnumProperty>

Adding the following after <PropertyPageSchema ... android_ant_build.xml ... /> 
in vs-android.CppBuild.targets
    <PropertyPageSchema Include="$(VCTargetsPath)\Platforms\Android\Props\android_ant_build_peruser.xml" Condition="'$(ConfigurationType)' == 'Application'" />

and adding the attached file android_ant_build_peruser.xml.

Ideally I'd want the defaults to have no deployment so that build servers just 
work and users can choose for themselves to change the deployment settings.

Also I note that this stuff doesn't really belong in "Ant build" since ant 
isn't used - it's all done with adb.

Original comment by m...@rebellion.co.uk on 13 Mar 2012 at 3:55

Attachments:

GoogleCodeExporter commented 9 years ago 
Oops, I meant that the AntDeploy EnumProperty was removed from 
android_ant_build.xml.

Original comment by m...@rebellion.co.uk on 13 Mar 2012 at 3:56

GoogleCodeExporter commented 9 years ago

Original comment by gavin.dj.pugh on 20 Mar 2012 at 8:32

GoogleCodeExporter commented 9 years ago 
Fixed in v0.94. I split out the deploy stuff completely from Ant.

Original comment by gavin.dj.pugh on 25 Jul 2012 at 7:19