darwin: update xcode versions

[spease]

Motivation for this change

All XCode versions listed are quite old

Things done

'hello' should build, but this needs more comprehensive testing imho.

• Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
• Built on platform(s)
  • NixOS
  • macOS
  • other Linux distributions
• Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
• Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
• Tested execution of all binary files (usually in ./result/bin/)
• Determined the impact on package closure size (by running nix path-info -S before and after)
• Ensured that relevant documentation is up to date
• Fits CONTRIBUTING.md.

[nixos-discourse]

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/compiling-for-ios/10776/1

[spease]

Bump. I wonder also if there might be a way to get requireXcode accessible externally, so that overrideAttrs could be used to add new versions too.

[nixos-discourse]

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/how-to-create-custom-cross-compile-overlay/10832/3

[jonringer]

cc @NixOS/darwin-maintainers

[jonringer]

I would change the commit message to be darwin: update xcode versions , as that's more inline with the changes you are proposing

[spease]

I haven't done this on github, would I amend and force-push the local branch, or push another commit, or is the rename to the ticket name enough? There are some newer versions that I can add to this.

[nixos-discourse]

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/trying-to-get-ios-working/10991/1

[spease]

Should be fixed. Let me know if there are any other issues

On Wed, Jan 20, 2021 at 08:40 Matthew Bauer ***@***.***> wrote: ***@***.**** commented on this pull request. ------------------------------ In pkgs/os-specific/darwin/xcode/default.nix <#100687 (comment)>: > let requireXcode = version: sha256: let xip = "Xcode_" + version + ".xip"; # TODO(alexfmpe): Find out how to validate the .xip signature in Linux - unxip = if stdenv.isDarwin + unxip = if targetPlatform.isDarwin this should be stdenv or buildPlatform ~ users will use these instructions on the machine their building things, not what they're targeting — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#100687 (review)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAVRX5DPRZHFGJMZEUFJAC3S24BQ3ANCNFSM4SS5EBBQ> .

-- - Steven