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

nixos: doc: implement related packages in the manual #32424

Merged
merged 6 commits into from Dec 23, 2017

Conversation

oxij
Copy link
Member

@oxij oxij commented Dec 7, 2017

Motivation for this change

A rebase and a partial rewrite of #14860 which I can't reopen for some reason that implements #9306 (finally!).

This version doesn't define any new types or mkOptions. It's super-uber-simple.

It does this
screenshot
(that's an old screenshot from #14860, this version looks a tiny bit different, but I'm too lazy to make a new one, its close enough)
when you write f56b582.

Adding relatedPackages to options puts a pressure on package maintainers to maintain good package descriptions or else the manual would look very ugly. Its a good thing.

Things done

…f their option group

Why? Because this way configuration.nix(5) can be read linearly.

Before:

> virtualisation.xen.bootParams
> ...
> virtualisation.xen.enable
> ...
> virtualisation.xen.package
> ...

After:

> virtualisation.xen.enable
> virtualisation.xen.package
> virtualisation.xen.bootParams
> ...
This allows one to specify "related packages" in NixOS that get rendered into
the configuration.nix(5) man page. The interface philosophy is pretty much
stolen from TeX bibliography.
@oxij oxij changed the title nixos: implement related packages nixos: implement related packages in the manual Dec 7, 2017
@oxij oxij changed the title nixos: implement related packages in the manual nixos: doc: implement related packages in the manual Dec 7, 2017
@jagajaga
Copy link
Member

jagajaga commented Dec 8, 2017

Awesome PR!

@oxij
Copy link
Member Author

oxij commented Dec 22, 2017 via email

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

Successfully merging this pull request may close these issues.

None yet

4 participants