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

terraform: use generic builder and add 0.9.0 #23978

Closed
wants to merge 106 commits into from
Closed

Conversation

peterhoeg
Copy link
Member

Motivation for this change

terraform is project that changes quite quickly and some people may be needing older versions (@shlevy) so this PR introduces a generic builder that allows us to easily have multiple versions.

We also lock terragrunt to v0.8 due to the breaking changes in 0.9.

Supersedes #23977

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.

ljli and others added 30 commits March 16, 2017 10:52
Fixes duplicate linkings issue for Qt-Frameworks provided by qtbase
during configurePhase.
corngood and others added 27 commits March 16, 2017 10:52
[Bjørn: change fetchurl indent 4 -> 2 spaces.]
We should really move to a source build of openjdk on Darwin, but that's
tricky so for now we continue to hack around it and do stuff like this,
even though crypto export polices are stupid and make no sense. No, I'm
not bitter at all that I need to do this 😩
Rresult is an OCaml module for handling computation results and errors in an
explicit and declarative manner without resorting to exceptions.

Homepage: http://erratique.ch/software/rresult
CLIPS is a language for building expert systems.
also added myself as a maintainer
@mention-bot
Copy link

@peterhoeg, thanks for your PR! By analyzing the history of the files in this pull request, we identified @FRidh to be a potential reviewer.

@peterhoeg peterhoeg closed this Mar 17, 2017
@peterhoeg peterhoeg deleted the u/tf branch March 17, 2017 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet