{"action":"created","issue":{"url":"https://api.github.com/repos/NixOS/nixpkgs/issues/46704","repository_url":"https://api.github.com/repos/NixOS/nixpkgs","labels_url":"https://api.github.com/repos/NixOS/nixpkgs/issues/46704/labels{/name}","comments_url":"https://api.github.com/repos/NixOS/nixpkgs/issues/46704/comments","events_url":"https://api.github.com/repos/NixOS/nixpkgs/issues/46704/events","html_url":"https://github.com/NixOS/nixpkgs/pull/46704","id":360553621,"node_id":"MDExOlB1bGxSZXF1ZXN0MjE1Nzc2NzU0","number":46704,"title":"stdenv/darwin: integrate a new CoreFoundation","user":{"login":"copumpkin","id":2623,"node_id":"MDQ6VXNlcjI2MjM=","avatar_url":"https://avatars1.githubusercontent.com/u/2623?v=4","gravatar_id":"","url":"https://api.github.com/users/copumpkin","html_url":"https://github.com/copumpkin","followers_url":"https://api.github.com/users/copumpkin/followers","following_url":"https://api.github.com/users/copumpkin/following{/other_user}","gists_url":"https://api.github.com/users/copumpkin/gists{/gist_id}","starred_url":"https://api.github.com/users/copumpkin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/copumpkin/subscriptions","organizations_url":"https://api.github.com/users/copumpkin/orgs","repos_url":"https://api.github.com/users/copumpkin/repos","events_url":"https://api.github.com/users/copumpkin/events{/privacy}","received_events_url":"https://api.github.com/users/copumpkin/received_events","type":"User","site_admin":false},"labels":[{"id":737692408,"node_id":"MDU6TGFiZWw3Mzc2OTI0MDg=","url":"https://api.github.com/repos/NixOS/nixpkgs/labels/10.rebuild-darwin-stdenv","name":"10.rebuild-darwin-stdenv","color":"b60205","default":false},{"id":731735740,"node_id":"MDU6TGFiZWw3MzE3MzU3NDA=","url":"https://api.github.com/repos/NixOS/nixpkgs/labels/10.rebuild-darwin:%20501+","name":"10.rebuild-darwin: 501+","color":"b60205","default":false},{"id":731734523,"node_id":"MDU6TGFiZWw3MzE3MzQ1MjM=","url":"https://api.github.com/repos/NixOS/nixpkgs/labels/10.rebuild-linux:%20501+","name":"10.rebuild-linux: 501+","color":"b60205","default":false},{"id":90308187,"node_id":"MDU6TGFiZWw5MDMwODE4Nw==","url":"https://api.github.com/repos/NixOS/nixpkgs/labels/6.topic:%20darwin","name":"6.topic: darwin","color":"fef2c0","default":false},{"id":9820195,"node_id":"MDU6TGFiZWw5ODIwMTk1","url":"https://api.github.com/repos/NixOS/nixpkgs/labels/6.topic:%20python","name":"6.topic: python","color":"fef2c0","default":false},{"id":897230915,"node_id":"MDU6TGFiZWw4OTcyMzA5MTU=","url":"https://api.github.com/repos/NixOS/nixpkgs/labels/6.topic:%20stdenv","name":"6.topic: stdenv","color":"fef2c0","default":false},{"id":290122734,"node_id":"MDU6TGFiZWwyOTAxMjI3MzQ=","url":"https://api.github.com/repos/NixOS/nixpkgs/labels/8.has:%20package%20(new)","name":"8.has: package (new)","color":"009800","default":false}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2018-09-15T16:16:35Z","updated_at":"2018-10-10T12:50:19Z","closed_at":"2018-09-15T20:42:50Z","author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/NixOS/nixpkgs/pulls/46704","html_url":"https://github.com/NixOS/nixpkgs/pull/46704","diff_url":"https://github.com/NixOS/nixpkgs/pull/46704.diff","patch_url":"https://github.com/NixOS/nixpkgs/pull/46704.patch"},"body":"This also updates the bootstrap tool builder to LLVM 5, but not the ones we actually use for bootstrap. I'll make that change in a subsequent commit so as to provide traceable provenance of the bootstrap tools.\r\n\r\n###### Motivation for this change\r\n\r\nWe have a horribly out of date CoreFoundation in our stdenv and this should modernize it a bit, allowing us to finally advance some other packages and also remove some workarounds.\r\n\r\n###### Things done\r\n\r\n[Here's an evaluation](https://hydra.nixos.org/eval/1478828?compare=1477805#tabs-now-fail) of the last state of this prior to me rebasing onto staging. I didn't feel like running a whole new eval after rebasing. Note that I also added the `CFFileDescriptor.h` stuff to support `watchman` properly since that eval, so if it complains that `watchman` is now failing, it's actually healthy again.\r\n\r\n<!-- Please check what applies. Note that these are not hard requirements but merely serve as information for reviewers. -->\r\n\r\n- [ ] Tested using sandboxing ([nix.useSandbox](http://nixos.org/nixos/manual/options.html#opt-nix.useSandbox) on NixOS, or option `sandbox` in [`nix.conf`](http://nixos.org/nix/manual/#sec-conf-file) on non-NixOS)\r\n- Built on platform(s)\r\n   - [ ] NixOS\r\n   - [x] macOS\r\n   - [ ] other Linux distributions\r\n- [ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside [nixos/tests](https://github.com/NixOS/nixpkgs/blob/master/nixos/tests))\r\n- [ ] Tested compilation of all pkgs that depend on this change using `nix-shell -p nox --run \"nox-review wip\"`\r\n- [ ] Tested execution of all binary files (usually in `./result/bin/`)\r\n- [ ] Determined the impact on package closure size (by running `nix path-info -S` before and after)\r\n- [ ] Fits [CONTRIBUTING.md](https://github.com/NixOS/nixpkgs/blob/master/.github/CONTRIBUTING.md).\r\n\r\n---\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/NixOS/nixpkgs/issues/comments/428558751","html_url":"https://github.com/NixOS/nixpkgs/pull/46704#issuecomment-428558751","issue_url":"https://api.github.com/repos/NixOS/nixpkgs/issues/46704","id":428558751,"node_id":"MDEyOklzc3VlQ29tbWVudDQyODU1ODc1MQ==","user":{"login":"copumpkin","id":2623,"node_id":"MDQ6VXNlcjI2MjM=","avatar_url":"https://avatars1.githubusercontent.com/u/2623?v=4","gravatar_id":"","url":"https://api.github.com/users/copumpkin","html_url":"https://github.com/copumpkin","followers_url":"https://api.github.com/users/copumpkin/followers","following_url":"https://api.github.com/users/copumpkin/following{/other_user}","gists_url":"https://api.github.com/users/copumpkin/gists{/gist_id}","starred_url":"https://api.github.com/users/copumpkin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/copumpkin/subscriptions","organizations_url":"https://api.github.com/users/copumpkin/orgs","repos_url":"https://api.github.com/users/copumpkin/repos","events_url":"https://api.github.com/users/copumpkin/events{/privacy}","received_events_url":"https://api.github.com/users/copumpkin/received_events","type":"User","site_admin":false},"created_at":"2018-10-10T12:50:19Z","updated_at":"2018-10-10T12:50:19Z","author_association":"MEMBER","body":"I've seen it but haven't done anything about it. If absolutely necessary, we could probably manually resurrect the shitty `*FSRef` stuff by manually transplanting and adjusting the code from the old CF releases into the new swift one, but I'd really prefer not to if we can avoid it. Apple doesn't want to support it, and I certainly don't if they don't 😄 "}}
{"actor":{"display_login":"copumpkin"}}