Skip to content

Commit

Permalink
rerun: move wasm app build to preBuild hook instead of `configurePh…
Browse files Browse the repository at this point in the history
…ase`
  • Loading branch information
OlivierLDff committed Jan 24, 2025
1 parent da83b17 commit 4d0503e
Showing 1 changed file with 1 addition and 5 deletions.
6 changes: 1 addition & 5 deletions pkgs/by-name/re/rerun/package.nix
Original file line number Diff line number Diff line change
Expand Up @@ -57,17 +57,13 @@ rustPlatform.buildRustPackage rec {
# The command is taken from https://github.com/rerun-io/rerun/blob/dd025f1384f9944d785d0fb75ca4ca1cd1792f17/pixi.toml#L198C72-L198C187
# Note that cargoBuildFeatures reference what buildFeatures is set to in stdenv.mkDerivation,
# so that user can easily create an overlay to set cargoBuildFeatures to what he needs
configurePhase = ''
runHook preConfigure
preBuild = ''
if [[ " $cargoBuildFeatures " == *" web_viewer "* ]]; then
echo "Building with web_viewer feature..."
cargo run -p re_dev_tools -- build-web-viewer --no-default-features --features analytics,grpc,map_view --release -g
else
echo "web_viewer feature not enabled, skipping web viewer build."
fi
runHook postConfigure
'';

nativeBuildInputs = [
Expand Down

0 comments on commit 4d0503e

Please sign in to comment.