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

docs/community-builder: add example NixOS configuration for remote builds #1552

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JohnRTitor
Copy link
Contributor

Added example NixOS configuration for remote builds using nix-build and nix build commands.

Inspired from https://github.com/NixOS/aarch64-build-box?tab=readme-ov-file#configuring-your-computer-for-remote-builds

Tested in my configuration, now finally nixpkgs-review works!

@github-actions github-actions bot added the docs label Nov 4, 2024
@JohnRTitor
Copy link
Contributor Author

Added a warning after discussing in the matrix channel.
I can't seem to fix the treefmt error in the CI though?

@zowoq zowoq marked this pull request as draft November 4, 2024 22:29
@zowoq
Copy link
Contributor

zowoq commented Nov 4, 2024

Added example NixOS configuration for remote builds using nix-build and nix build commands.

Inspired from https://github.com/NixOS/aarch64-build-box?tab=readme-ov-file#configuring-your-computer-for-remote-builds

Added a warning after discussing in the matrix channel.

I intentionally didn't add this, didn't want to have an "example" distributedBuilds config, even with a warning.

now finally nixpkgs-review works!

Maybe we could show an example of passing builders to nixpkgs-review instead.

@JohnRTitor
Copy link
Contributor Author

JohnRTitor commented Nov 5, 2024

Maybe we could show an example of passing builders to nixpkgs-review instead.

I am not sure how nixpkgs-review works with remote builders actually.

But I thought it would be safe if we configure it where NixOS system platform != Builder platform

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants