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

Typos causing errors in https://nix.dev/tutorials/working-with-local-files #1002

Closed
biscotty666 opened this issue Jun 18, 2024 · 1 comment · Fixed by #1017
Closed

Typos causing errors in https://nix.dev/tutorials/working-with-local-files #1002

biscotty666 opened this issue Jun 18, 2024 · 1 comment · Fixed by #1017
Labels
good first issue site Improvements to the site infrastructure or content presentation

Comments

@biscotty666
Copy link

Thank you for some very good docs/tutorials.

I just went through the above tutorial, and wanted to let you know of a couple of typos in the code which naturally cause errors.

  1. The command nix-shell -p npins --run "npins init --bare; npins add github nixos nixpkgs --branch nixos-23.11" creates a directory name npins. Later, in the sections on using unions, the build.nix file is (silently) altered to include the directory ./nix. This should be ./npins (and the addition to the file might be pointed out).

  2. In the section on Matching files tracked by Git, the code block includes git reset src/select.o result. select.o should be select.c.

NixOS is often criticized, with some justification, for its documentation. This site is a counter-example, with top-class tutorials. Thank you again.

@biscotty666 biscotty666 added the site Improvements to the site infrastructure or content presentation label Jun 18, 2024
@fricklerhandwerk
Copy link
Collaborator

Thanks for catching these details!

2. In the section on Matching files tracked by Git, the code block includes git reset src/select.o result. select.o should be select.c.

This is not correct. The command removes select.o from the staged files, which is what we want here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue site Improvements to the site infrastructure or content presentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants