gavinpugh / vs-android

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

Setting Target Architecture doesn't pass through to GCC #51

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Set the Target Architecture in "General" Properties to armv7-a for a project.
2. Enable showing the gcc command line for that project.
3. Compile a .cpp/.c file in the project.

What is the expected output? What do you see instead?
The command line doesn't contain "-march=armv7-a -mtune=xscale" it doesn't 
specify arm5 or arm7 as the architecture.

What version of the product are you using? On what operating system?
Windows 7, version 0.93.

Please provide any additional information below.
I have made a fix for this issue.  I'm unsure if this is the best way to fix 
the issue, but it works.

I added the following lines to file: Microsoft.Cpp.Android.targets

Line 41, below <MinimalRebuildFromTracking...
   <AndroidArch Condition="'$(AndroidArch)' == 'armv5te'">armv5te</AndroidArch>
   <AndroidArch Condition="'$(AndroidArch)' == 'armv7-a'">armv7-a</AndroidArch>     

I renamed the Attribute contents for Name from "Architecture" to "AndroidArch" 
in file: android_gcc_compile.xml

Here is the new line 106:

  <EnumProperty Name="AndroidArch" Category="General" Visible="false">

Thanks for a great project.

Original issue reported on code.google.com by drew.dun...@smokingguninc.com on 6 Dec 2011 at 6:18

GoogleCodeExporter commented 9 years ago

Original comment by gavin.dj.pugh on 11 Dec 2011 at 3:27

GoogleCodeExporter commented 9 years ago

Original comment by gavin.dj.pugh on 21 Feb 2012 at 11:39

GoogleCodeExporter commented 9 years ago
Thanks, this has been fixed in the v0.94 release.

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

GoogleCodeExporter commented 9 years ago
Issue 85 has been merged into this issue.

Original comment by gavin.dj.pugh on 12 Sep 2012 at 12:26