Documentation: Update iOS build docs to include instructions for gn.

Documentation: Update iOS build docs to include instructions for gn.

Committed: https://crrev.com/88a0fac96baac964ac25fff53a05e7b8a3503c0a
Cr-Commit-Position: refs/heads/master@{#390250}

[Patti Lor, 2016-04-22 01:46:41]

Patchset #2 (id:20001) has been deleted

[Patti Lor, 2016-04-22 01:49:18]

patricialor@chromium.org changed reviewers:
+ rsesek@chromium.org

[Patti Lor, 2016-04-22 01:49:18]

PTAL Robert - updated the docs because it took me a while to find the right GN
flags to set for an iOS build.

[Robert Sesek, 2016-04-22 15:17:39]

rsesek@chromium.org changed reviewers:
+ sdefresne@chromium.org

[Robert Sesek, 2016-04-22 15:17:39]

-> sdefresne

[sdefresne, 2016-04-22 15:22:37]

https://codereview.chromium.org/1904773003/diff/1/docs/ios_build_instructions.md
File docs/ios_build_instructions.md (right):

https://codereview.chromium.org/1904773003/diff/1/docs/ios_build_instructions...
docs/ios_build_instructions.md:54: use_ios_simulator = true
This is not required as the value of use_ios_simulator is automatically
initialized from target_cpu on iOS.
I usually tweak the following settings on iOS:

ios_enable_code_signing = false
is_debug = true
target_os = "ios"
target_cpu = "x64"

Then is_debug control release/debug while target_cpu can take the values "x86",
"x64", "arm", "armv7" or "arm64". The first two implies simulator build, the
last one implies device build. ios_enable_code_signing should be set to false if
you do not have valid code signing key.

[Patti Lor, 2016-04-26 00:03:02]

Thanks Sylvain, PTAL!

https://codereview.chromium.org/1904773003/diff/1/docs/ios_build_instructions.md
File docs/ios_build_instructions.md (right):

https://codereview.chromium.org/1904773003/diff/1/docs/ios_build_instructions...
docs/ios_build_instructions.md:54: use_ios_simulator = true
On 2016/04/22 15:22:37, sdefresne wrote:
> This is not required as the value of use_ios_simulator is automatically
> initialized from target_cpu on iOS.
> I usually tweak the following settings on iOS:
> 
> ios_enable_code_signing = false
> is_debug = true
> target_os = "ios"
> target_cpu = "x64"
> 
> Then is_debug control release/debug while target_cpu can take the values
"x86",
> "x64", "arm", "armv7" or "arm64". The first two implies simulator build, the
> last one implies device build. ios_enable_code_signing should be set to false
if
> you do not have valid code signing key.

Done, thanks for the extra information!

[sdefresne, 2016-04-27 12:44:00]

lgtm

[sdefresne, 2016-04-27 12:44:17]

Thank you for keeping the documentation up-to-date and relevant!

[Patti Lor, 2016-04-27 23:39:47]

The CQ bit was checked by patricialor@chromium.org

[commit-bot: I haz the power, 2016-04-27 23:40:06]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1904773003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1904773003/40001

[commit-bot: I haz the power, 2016-04-28 00:16:54]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-04-30 17:14:53]

Patchset 2 (id:??) landed as
https://crrev.com/88a0fac96baac964ac25fff53a05e7b8a3503c0a
Cr-Commit-Position: refs/heads/master@{#390250}

[commit-bot: I haz the power, 2016-04-30 17:53:14]

Description was changed from

==========
Documentation: Update iOS build docs to include instructions for gn.
==========

to

==========
Documentation: Update iOS build docs to include instructions for gn.

Committed: https://crrev.com/88a0fac96baac964ac25fff53a05e7b8a3503c0a
Cr-Commit-Position: refs/heads/master@{#390250}
==========