Fix naming of macOS SDK and clarify version #18589

1. 
   achow101 commented at 0:16 am on April 11, 2020: member

   Fixes the MacOSX10.14.sdk.tar.gz creation command to have MacOSX.sdk be correctly named as MacOSX10.14.sdk and for the resulting file to be placed in the current directory. Gitian requires that tar.gz contains a folder named MacOSX10.14.sdk and the command did not do this originally. Having the file be placed in the current directory is a convenience so builders don’t have to go find it.

   Also clarifies which version of Xcode to download and where it can be downloaded.

2. fanquake added the label Docs on Apr 11, 2020
3. 
   DrahtBot commented at 4:03 am on April 11, 2020: member

   The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

   Conflicts

   Reviewers, this pull request conflicts with the following ones:

   • #18072 (Use libc++ headers from macOS SDK instead of from clang by dongcarl)
   • #17919 (depends: Allow building with system clang by dongcarl)

   If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

4. 
   Emzy commented at 12:41 pm on April 11, 2020: contributor

   Approach ACK It fixed my gitian build setup.
5. fanquake requested review from dongcarl on Apr 12, 2020
6. 
   Sjors commented at 2:18 pm on April 19, 2020: member

   See also #18674
7. 
   Sjors commented at 6:26 pm on April 19, 2020: member

   On macOS the equivalent is tar -s "/MacOSX.sdk/MacOSX10.14.sdk/"
8. 
   achow101 commented at 10:03 pm on April 21, 2020: member

   @Sjors Was that the exact command you used? I can update the doc with what actually worked.
9. 
   Sjors commented at 7:46 am on April 23, 2020: member

   0tar -s "/MacOSX.sdk/MacOSX10.14.sdk/" -C Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/ -czf MacOSX10.14.sdk.tar.gz MacOSX.sdk
   

10. fanquake added the label Needs backport (0.20) on Apr 23, 2020
11. Fix naming of macOS SDK and clarify version eb37275a6f
12. achow101 force-pushed on Apr 23, 2020
13. 
    Sjors commented at 6:51 pm on April 23, 2020: member

    ACK eb37275 for the macOS instruction
14. fanquake approved
15. 
    fanquake commented at 9:22 am on April 24, 2020: member

    ACK eb37275a6f972c81caef010b4ee9c5dc88edc759 - tested the macOS and Linux SDK extraction. Also noticed something seemingly broken with Apple tar, but will open an issue to follow up.
16. fanquake merged this on Apr 24, 2020
17. fanquake closed this on Apr 24, 2020

    ---

18. fanquake referenced this in commit a9ca65bd29 on Apr 24, 2020
19. fanquake removed the label Needs backport (0.20) on Apr 24, 2020
20. laanwj referenced this in commit fb5b098598 on May 11, 2020
21. zkbot referenced this in commit 2a39656e6d on Jul 30, 2020
22. zkbot referenced this in commit e0692ed4df on Aug 7, 2020
23. backpacker69 referenced this in commit f539dadfe3 on Mar 28, 2021
24. furszy referenced this in commit 816f42d7ac on May 25, 2021
25. DrahtBot locked this on Feb 15, 2022

Contributors
achow101 DrahtBot Emzy Sjors fanquake

Review Requested
dongcarl

Labels
Docs