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

build: add nix-env for building the proxy #1354

Closed
wants to merge 1 commit into from
Closed

Conversation

hawkw
Copy link
Contributor

@hawkw hawkw commented Nov 3, 2021

This branch adds a simple nix-env configuration for building the proxy
locally on NixOS. This does not yet introduce a derivation for
packaging the proxy as a Nix package, just a nix-env for local
development for Nix users.

Actually packaging the proxy for Nix could be
fun, but it's not really necessary, since the proxy is not currently
distributed as a package for other package managers --- it's distributed
as a docker image. If we were going to actually distribute something as a
Nix package, it would be the Linkerd CLI (which could be worth doing!).

This branch does include the necessary configuration to build
cloudflare/boring, which is added as a dependency by @olix0r's
PR #1351. I've confirmed that it is possible to build that branch with
these configs, but I opened this as a separate PR against main so that
we can merge it separately. Currently, we do have to build boringssl
from source, rather than depending on it from nixpkgs, which is kind of
a bummer, but this can be fixed later.

This branch adds a simple nix-env configuration for building the proxy
locally on NixOS. This does *not* yet introduce a derivation for
*packaging* the proxy as a Nix package, just a nix-env for local
development for Nix users.

Actually packaging the proxy for Nix could be
fun, but it's not really necessary, since the proxy is not currently
distributed as a package for other package managers --- it's distributed
as a docker image. If we were going to actually distribute something as a
Nix package, it would be the Linkerd CLI (which could be worth doing!).

This branch *does* include the necessary configuration to build
`cloudflare/boring`, which is added as a dependency by @olix0r's
PR #1351. I've confirmed that it is possible to build that branch with
these configs, but I opened this as a separate PR against `main` so that
we can merge it separately. Currently, we do have to build `boringssl`
from source, rather than depending on it from nixpkgs, which is kind of
a bummer, but this can be fixed later.
@hawkw hawkw requested a review from a team November 3, 2021 18:36
hawkw added a commit that referenced this pull request Nov 6, 2021
@olix0r
Copy link
Member

olix0r commented Jan 19, 2022

Closing this in favor of #1432

@olix0r olix0r closed this Jan 19, 2022
@olix0r olix0r deleted the eliza/nix-env branch January 19, 2022 17:38
@hawkw
Copy link
Contributor Author

hawkw commented Jan 19, 2022

fair enough!

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.

2 participants