doc/default.nix: make the manual build on more than one core

[ghost]

Let's build the manual with more than one core. Maybe people will take better care of it now that it is less painful to build.

[Artturin]

Can't test

(before and after this pr)

nix build ".#htmlDocs.nixpkgsManual"

nixpkgs-manual> cp -r doc-support/result/highlightjs/highlight.pack.js highlightjs/
nixpkgs-manual> cp: cannot stat 'doc-support/result/highlightjs/highlight.pack.js': No such file or directory

[ghost]

nix build ".#htmlDocs.nixpkgsManual"

Try

nix build -f doc

But yeah we should give the manual an entry in pkgs/top-level/all-packages.nix. Right now it has its own wonky entry point.

[ghost]

But yeah we should give the manual an entry in pkgs/top-level/all-packages.nix

• all-packages.nix: add nixpkgsManual #226064

[fricklerhandwerk]

@pennae you're certainly more qualified than me to assess this.

[pennae]

this looks like a good thing, but unfortunately it changes the output of the rendering process for some reason. the makefile must be missing some dependencies to make that happen, but for the md rendering part specifically we can do this.

let's do this instead to get most of the benefits until the rendering infra can be migrated away from pandoc:

diff --git a/doc/Makefile b/doc/Makefile
index ee98a3ee921..208f23f5023 100644
--- a/doc/Makefile
+++ b/doc/Makefile
@@ -19,6 +19,9 @@ pandoc_flags = --extract-media=$(pandoc_media_dir) \
 .PHONY: all
 all: validate format out/html/index.html out/epub/manual.epub
 
+.PHONY: render-md
+render-md: ${MD_TARGETS}
+
 .PHONY: debug
 debug:
        nix-shell --run "xmloscopy --docbook5 ./manual.xml ./manual-full.xml"
diff --git a/doc/default.nix b/doc/default.nix
index ac405c37903..4f55c95a04c 100644
--- a/doc/default.nix
+++ b/doc/default.nix
@@ -20,6 +20,10 @@ in pkgs.stdenv.mkDerivation {
     ln -s ${doc-support} ./doc-support/result
   '';
 
+  preBuild = ''
+    make -j$NIX_BUILD_CORES render-md
+  '';
+
   installPhase = ''
     dest="$out/share/doc/nixpkgs"
     mkdir -p "$(dirname "$dest")"

[ghost]

Latest push implements @pennae's suggestion.

[fricklerhandwerk]

Thanks a lot, everyone!