Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

xcbuild: 0ab861ab -> 0.1.1 #21895

Merged
merged 2 commits into from Jan 19, 2017
Merged

xcbuild: 0ab861ab -> 0.1.1 #21895

merged 2 commits into from Jan 19, 2017

Conversation

matthewbauer
Copy link
Member

Motivation for this change

These changes fix an issue with building under i686-linux (http://hydra.nixos.org/job/nixpkgs/trunk/xcbuild.i686-linux). I've also added a wrapper for xcode-select so that it detects the pure SDK instead of looking in /Applications/Xcode.app.

Things done
  • Tested using sandboxing
    (nix.useSandbox on NixOS,
    or option build-use-sandbox in nix.conf
    on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • Linux
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@mention-bot
Copy link

@matthewbauer, thanks for your PR! By analyzing the history of the files in this pull request, we identified @copumpkin and @LnL7 to be potential reviewers.

@LnL7 LnL7 added 6.topic: darwin Running or building packages on Darwin 8.has: package (update) labels Jan 14, 2017
@LnL7
Copy link
Member

LnL7 commented Jan 14, 2017

The linux build fails for me. https://gist.github.com/LnL7/c6c772261a6e87805eada2ad2c736ef8

@matthewbauer
Copy link
Member Author

Fixed with flag. See facebookarchive/xcbuild#221

@LnL7 LnL7 changed the title xcbuild: update to 0.1.1 xcbuild: 0ab861ab -> 0.1.1 Jan 19, 2017
@globin globin merged commit 10cfe94 into NixOS:master Jan 19, 2017
@LnL7
Copy link
Member

LnL7 commented Jan 19, 2017

There also seems to be a problem with some of the darwin packages that depend on it https://gist.github.com/LnL7/8f6b4714b587549ba23b06c614f6cd32

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.topic: darwin Running or building packages on Darwin 8.has: package (update)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants