Nixpkgs/Contributing: Difference between revisions
imported>Milahu unindent section headings |
imported>Milahu + called without required argument |
||
Line 74: | Line 74: | ||
** Read https://editorconfig.org/#download to configure your editor | ** Read https://editorconfig.org/#download to configure your editor | ||
* [https://github.com/NixOS/ofborg#how-does-ofborg-call-nix-build ofborg-eval] will call <code>nix-build -A $yourpackage</code> | * [https://github.com/NixOS/ofborg#how-does-ofborg-call-nix-build ofborg-eval] will call <code>nix-build -A $yourpackage</code> | ||
==== Called without required argument ==== | |||
This error is produced by <code>nixpkgs-basic-release-checks</code> (Basic evaluation checks) | |||
<pre> | |||
anonymous function at /path/to/your-package.nix called without required argument 'some-dependency' | |||
</pre> | |||
Usually, a dependency (some-dependency) is not available on a certain platform, | |||
for example on <code>aarch64-darwin</code> | |||
Solution: Make it an optional dependency: | |||
<pre> | |||
{ lib | |||
, stdenv | |||
, some-dependency ? null | |||
, another-dependency | |||
}: | |||
stdenv.mkDerivation { | |||
buildInputs = | |||
[ another-dependency ] | |||
++ lib.optionals (!stdenv.isDarwin) [ some-dependency ] | |||
# some-dependency is missing on darwin | |||
; | |||
} | |||
</pre> | |||
=== Push to your remote repository === | === Push to your remote repository === |
Revision as of 21:54, 6 November 2021
Development in NixOS primarily driven by the work in nixpkgs on GitHub. This repository contains both all packages available in your NixOS channel and all the options you can use for configuring your system with your configuration.nix
. To get your text editor to recognize Nix expressions, consider installing a Nix Editor Modes for Nix Files.
Report issues
Any issue can be reported in the nixpkgs issue tracker on GitHub. Keep in mind that all work on nixpkgs is being done by volunteers and you cannot expect a quick response and solution for all problems you may face. In general Pull Requests have a much shorter round-trip-time.
Create pull requests
If you want to see your package being provided by a channel, creating an issue will most likely not enough. It is up to you to create a nix package description in Nixpkgs and create a pull request in the Nixpkgs repository. Pull requests are a way to tell a GitHub project that you've created some changes, which maintainers can easily review, comment on and, and finally merge into the repository.
Here's how to create a pull request on GitHub:
Fork Nixpkgs on Github
Create a GitHub account and fork the nixpkgs repository.
Add a remote
On your host, create a new remote location:
# add your github clone as remote location:
YOUR_GITHUB_NAME=fill-in
git remote add $YOUR_GITHUB_NAME git@github.com:$YOUR_GITHUB_NAME/nixpkgs.git
Hack Nixpkgs
Make any modifications you want to your local copy of the repository, then build the package from the root of the nixpkgs
directory with:
nix-build -A $yourpackage
The output of your build will be located under the result/
subdirectory. Try running the freshly built binaries in result/bin
and check that everything is OK.
To test the changes on a NixOS machine, rebuild the system using your newly hacked Nixpkgs by executing:
sudo nixos-rebuild switch -I nixpkgs=/path/to/local/nixpkgs
Commit your change locally
Commit your change to your local clone of the repository:
git add FILE_LIST # use git add --patch as needed
git commit
=== Check status ===
Verify everything is ok - no unexpected dangling files git does not know about yet:
git status
Run tests locally
Pushing commits to Github will run tests on Github.
We can run these tests locally, to reduce "commit noise" from failing tests
cd nixpkgs # Basic evaluation checks nix-build pkgs/top-level/release.nix -A tarball.nixpkgs-basic-release-checks \ --arg supportedSystems '[ "aarch64-darwin" "aarch64-linux" "x86_64-linux" "x86_64-darwin" ]' # list all derivations nix-env --query --available --out-path --file ./. --show-trace # build nix-build -A $yourpackage
Tests on Github:
- Basic evaluation checks
- Checking EditorConfig
- Read https://editorconfig.org/#download to configure your editor
- ofborg-eval will call
nix-build -A $yourpackage
Called without required argument
This error is produced by nixpkgs-basic-release-checks
(Basic evaluation checks)
anonymous function at /path/to/your-package.nix called without required argument 'some-dependency'
Usually, a dependency (some-dependency) is not available on a certain platform,
for example on aarch64-darwin
Solution: Make it an optional dependency:
{ lib , stdenv , some-dependency ? null , another-dependency }: stdenv.mkDerivation { buildInputs = [ another-dependency ] ++ lib.optionals (!stdenv.isDarwin) [ some-dependency ] # some-dependency is missing on darwin ; }
Push to your remote repository
Submitting your change, push to your repository:
# recommended: create a topic branch, so that this change
# can be submitted independently from other patches:
git checkout -tb submit/your-topic-name
git push $YOUR_GITHUB_NAME submit/your-topic-name
Why create your own branch? You can follow upstream (master) by running git merge master
.
You can git commit --amend
fixes and git push -f
your branch.
You can always switch back to master by git checkout master
.
Create a pull request on GitHub
Go to the Nixpkgs repository and push the create pull request button. Add a description of your work and submit.
Manage your local repository
Tips & tricks for managing your nixpkgs
checkout are kept in the page on git.
Becoming a Nixpkgs maintainer
There are two types of maintainer: Members of the NixOS organization and package/module maintainer.
Members of the NixOS organization have direct access to the nixpkgs and therefore can merge pull requests.
Everybody can become a package maintainer by adding to maintainers
attribute in the meta section of a package. First add your nick handle (preferable your GitHub username) to maintainer-list.nix and reference the said handle within the package:
{ stdenv, lib, fetchurl }:
stdenv.mkDerivation rec {
name = "hello-2.10";
# ....
meta = with lib; {
# ...
maintainers = with maintainers; [ eelco your-nick ]; # replace your-nick with your real nick
platforms = platforms.all;
};
}
There can be multiple maintainers per package. Maintainers will receive emails from Hydra when package builds enters a failed state. They do not have special privileges as everybody can suggest updates and modifications to any package. However they might be consulted by NixOS members for testing and as a domain experts, when somebody else make a change to a package.
Building all of the packages you maintain
nix-build maintainers/scripts/build.nix --argstr maintainer your-nick
Maintain your nixpkgs fork
Update master
Add nixpkgs as a remote called upstream:
git remote add upstream https://github.com/NixOS/nixpkgs.git
You only have to do it once.
git checkout master #1
git fetch upstream
git rebase upstream/master
1. make sure you're on the master branch