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

add guide on post build hook #528

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

fricklerhandwerk
Copy link
Collaborator

Closes #463

originally written under LGPL 2.1 for the Nix reference manual.
authors as recorded in the Nix repository:

git log --pretty="Co-authored-by: %an <%ae>" -- doc/manual/advanced-topics/post-build-hook.xml doc/manual/src/advanced-topics/post-build-hook.md | sort | uniq

relicenced with authors' permission to CC-BY-SA 4.0: #463

@fricklerhandwerk fricklerhandwerk added the guide Topics for a new guides label Feb 7, 2024
@infinisil
Copy link
Member

Now that the CLA is gone, it seems fine to proceed with this. @fricklerhandwerk sound good?

@fricklerhandwerk
Copy link
Collaborator Author

fricklerhandwerk commented Mar 7, 2024

Yeah, it just needs to be finished (i.e. review pass, add links to the Nix manual, check that it actually works, etc.). Free for grabs really, my queue is way too long to ever terminate anyway.

fricklerhandwerk and others added 4 commits May 27, 2024 07:48
originally written under LGPL 2.1 for the Nix reference manual.
authors as recorded in the Nix repository:

    git log --pretty="Co-authored-by: %an <%ae>" -- doc/manual/advanced-topics/post-build-hook.xml doc/manual/src/advanced-topics/post-build-hook.md | sort | uniq

relicenced with authors' permission to CC-BY-SA 4.0: NixOS#463

Co-authored-by: Eelco Dolstra <edolstra@gmail.com>
Co-authored-by: Graham Christensen <graham@grahamc.com>
Co-authored-by: Joe Hermaszewski <git@monoid.al>
Co-authored-by: Kevin Stock <kevin@kevinstock.org>
Co-authored-by: endgame <endgame@users.noreply.github.com>
Co-authored-by: regnat <rg@regnat.ovh>
@proofconstruction
Copy link
Contributor

Rebased this and fixed some of the links (and the errors due to the vale check I didn't notice before), but now linkcheck fails in an unrelated file due to NixOS/nixos-homepage#1448; pausing this while that's sorted.

trusted-public-keys = cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY= example-nix-cache-1:1/cKDz3QCCOmwcztD2eV6Coggp6rqc9DGjWv7C0G+rM=

Machines that build for the cache must sign derivations using the private key.
On those machines, add the path to the key file to the [`secret-key-files`](https://nixos.org/manual/nix/command-ref/conf-file.html#conf-secret-key-files) field in their [`nix.conf`](https://nixos.org/manual/nix/command-ref/conf-file.html):
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
On those machines, add the path to the key file to the [`secret-key-files`](https://nixos.org/manual/nix/command-ref/conf-file.html#conf-secret-key-files) field in their [`nix.conf`](https://nixos.org/manual/nix/command-ref/conf-file.html):
On those machines, add the path to the file with the private key to the [`secret-key-files`](https://nixos.org/manual/nix/command-ref/conf-file.html#conf-secret-key-files) field in their [`nix.conf`](https://nixos.org/manual/nix/command-ref/conf-file.html):


secret-key-files = /etc/nix/key.private

We will restart the Nix daemon in a later step.
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
We will restart the Nix daemon in a later step.
You will restart the Nix daemon in a later step.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
guide Topics for a new guides
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

Nix post-build hooks
3 participants