Skip to content

Commit

Permalink
ntopng: fix missing static assets
Browse files Browse the repository at this point in the history
Instead of building the `dist` make target in the derivation (requires npm and
all that), we're using the upstream pre-built static assets. Upstream refers to
these using a sub-module so we need to fetch that.

Without this patch the web UI loads the html, but 404s on all CSS and JS assets
because the `$out/share/ntopng/httpdocs/dist` directory is empty
  • Loading branch information
samhug authored and bjornfor committed Jun 17, 2023
1 parent 835cf96 commit 1f07bba
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion pkgs/tools/networking/ntopng/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,8 @@ stdenv.mkDerivation rec {
owner = "ntop";
repo = "ntopng";
rev = "refs/tags/${version}";
hash = "sha256-iGqrS0AneKYwGMEpbKy9if8bnaEu6aEV+QaH+JrF9xs=";
hash = "sha256-pIm0C1+4JLVDdXxSaQtd6ON8R2l6KG8ZXuDDuRd6dQI=";
fetchSubmodules = true;
};

preConfigure = ''
Expand Down

0 comments on commit 1f07bba

Please sign in to comment.