mac: Update build instructions to include details about 10.10 SDK.

mac: Update build instructions to include details about 10.10 SDK.

The previous build instructions didn't talk about SDKs at all. The previous
build instructions were also a little bit out of date.

BUG=NONE

Committed: https://crrev.com/64a06c22f867f628031b40eed61568d7ce3d2693
Cr-Commit-Position: refs/heads/master@{#350872}

[erikchen, 2015-09-25 18:14:22]

erikchen@chromium.org changed reviewers:
+ andybons@chromium.org

[erikchen, 2015-09-25 18:14:23]

andybons: Please review.

[Bons, 2015-09-25 18:18:38]

lgtm

https://codereview.chromium.org/1369003002/diff/1/docs/mac_build_instructions.md
File docs/mac_build_instructions.md (right):

https://codereview.chromium.org/1369003002/diff/1/docs/mac_build_instructions...
docs/mac_build_instructions.md:19: whether you have it. If you don't have it,
you need to get one from an
I would write this as:

*   The OSX 10.10 SDK. Run
    ```sh
    ls `xcode-select -p` /Platforms/MacOSX.platform/Developer/SDKs
    ```
    to check whether you have it. If you don't have it, you need to get one from
an
    install of Xcode 6, and place it in the above directory.

[erikchen, 2015-09-25 18:23:53]

https://codereview.chromium.org/1369003002/diff/1/docs/mac_build_instructions.md
File docs/mac_build_instructions.md (right):

https://codereview.chromium.org/1369003002/diff/1/docs/mac_build_instructions...
docs/mac_build_instructions.md:19: whether you have it. If you don't have it,
you need to get one from an
On 2015/09/25 18:18:38, Bons wrote:
> I would write this as:
> 
> *   The OSX 10.10 SDK. Run
>     ```sh
>     ls `xcode-select -p` /Platforms/MacOSX.platform/Developer/SDKs
>     ```
>     to check whether you have it. If you don't have it, you need to get one
from
> an
>     install of Xcode 6, and place it in the above directory.

Done.

[erikchen, 2015-09-25 18:23:57]

The CQ bit was checked by erikchen@chromium.org

[erikchen, 2015-09-25 18:23:57]

The patchset sent to the CQ was uploaded after l-g-t-m from
andybons@chromium.org
Link to the patchset: https://codereview.chromium.org/1369003002/#ps20001
(title: "Comments from andybons.")

[commit-bot: I haz the power, 2015-09-25 18:24:26]

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

[commit-bot: I haz the power, 2015-09-25 18:35:47]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-09-25 18:37:02]

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