xcodes: 1.5.0 -> 1.6.0, move to by name, with lib; cleanup, RFC format

[matteo-pacini]

Fixes #352844 .

Also:

• Moved everything into pkgs/by-name (whilst adopting the new implicit apple-sdk pattern)
• Cleaned up with lib; usage in meta
• Applied RFC format

Things done

• Built on platform(s)
  • x86_64-linux
  • aarch64-linux
  • x86_64-darwin
  • aarch64-darwin
• For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
  • sandbox = relaxed
  • sandbox = true
• Tested, as applicable:
  • NixOS test(s) (look inside nixos/tests)
  • and/or package tests
  • or, for functions and "core" functionality, tests in lib/tests or pkgs/test
  • made sure NixOS tests are linked to the relevant packages
• Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
• Tested basic functionality of all binary files (usually in ./result/bin/)
• 24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)
  • (Package updates) Added a release notes entry if the change is major or breaking
  • (Module updates) Added a release notes entry if the change is significant
  • (Module addition) Added a release notes entry if adding a new NixOS module
• Fits CONTRIBUTING.md.

---

Add a 👍 reaction to pull requests you find important.

[emilazy]

This doesn’t build for me on x86_64-darwin; is that a known issue?

[matteo-pacini]

@emilazy not entirely sure, but I think it flown below the radar because Swift didn't build on Hydra for a long time, and so did the packages that depend on it?

https://hydra.nixos.org/job/nixpkgs/trunk/xcodes.x86_64-darwin#tabs-status

The error is the following:

#import <CoreServices/CoreServices.h>
        ^
/nix/store/wgcbqi40ahn1ay6905v190x5mlv9xdq9-apple-sdk-13.3/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/System/Library/Frameworks/CoreServices.framework/Frameworks/CarbonCore.framework/Headers/MachineExceptions.h:22:10: note: while building module '_Builtin_intrinsics' imported from /nix/store/wgcbqi40ahn1ay6905v190x5mlv9xdq9-apple-sdk-13.3/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/System/Library/Frameworks/CoreServices.framework/Frameworks/CarbonCore.framework/Headers/MachineExceptions.h:22:
#include <emmintrin.h>
         ^
<module-includes>:1:9: note: in file included from <module-includes>:1:
#import "immintrin.h"
        ^
/nix/store/d2jm2gdf5qmyfbqk579f7h65sbi497sr-clang-wrapper-16.0.6/resource-root/include/immintrin.h:31:10: note: in file included from /nix/store/d2jm2gdf5qmyfbqk579f7h65sbi497sr-clang-wrapper-16.0.6/resource-root/include/immintrin.h:31:
#include <emmintrin.h>
         ^
/nix/store/d2jm2gdf5qmyfbqk579f7h65sbi497sr-clang-wrapper-16.0.6/resource-root/include/emmintrin.h:45:9: error: _Float16 is not supported on this target
typedef _Float16 __m128h_u __attribute__((__vector_size__(16), __aligned__(1)));

Not familiar with the internals of Clang/LLVM, but it sounds like it's trying to use an unsupported type (_Float16) on x86_64-darwin.

[emilazy]

Oh, the problem is that you have to use the Swift stdenv. I forget why.

[matteo-pacini]

@emilazy works, thanks!

[emilazy]

I’m not sure if this is any different from swiftPackages.stdenv but I think the latter is canonical.

[matteo-pacini]

Searched on the repo, and you're right - it seems to be more widely adopted.
Just pushed this change, thanks!