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

cmus: fix darwin #22116

Merged
merged 7 commits into from Feb 20, 2017
Merged

cmus: fix darwin #22116

merged 7 commits into from Feb 20, 2017

Conversation

LnL7
Copy link
Member

@LnL7 LnL7 commented Jan 24, 2017

Motivation for this change

Fixes cmus for darwin based on the changes from #14204

Things done
  • Tested using sandboxing / without xcode
    (nix.useSandbox on NixOS,
    or option build-use-sandbox in nix.conf
    on non-NixOS)
  • Built on platform(s)
    • macOS
    • No Changes
  • 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.

@LnL7 LnL7 added the 6.topic: darwin Running or building packages on Darwin label Jan 24, 2017
@mention-bot
Copy link

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

@LnL7 LnL7 mentioned this pull request Jan 24, 2017
7 tasks
@globin
Copy link
Member

globin commented Jan 25, 2017

The travis failure looks relevant. (disclaimer I have no idea of darwin stuff)

@LnL7
Copy link
Member Author

LnL7 commented Jan 25, 2017

Indeed, that package didn't work before because libcdio was linux only.

@LnL7
Copy link
Member Author

LnL7 commented Jan 25, 2017

I also fixed the build to use clang, perhaps there should be a top level package with the clang based gcc binaries?

@joachifm
Copy link
Contributor

joachifm commented Feb 9, 2017

@LnL7 is this ready for integration?

@LnL7
Copy link
Member Author

LnL7 commented Feb 9, 2017

Should be fine, unless we want a top level drv for the clang gcc wrapper.

@globin globin merged commit dcba6ea into NixOS:master Feb 20, 2017
@LnL7 LnL7 deleted the darwin-cmus branch February 20, 2017 21:13
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants