Skip to content

Commit 235470d

Browse files
committed
fixup! refactor(ros-z-msgs): remove the path of local roslibrust
1 parent 6a770d5 commit 235470d

2 files changed

Lines changed: 6 additions & 21 deletions

File tree

flake.nix

Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -282,24 +282,6 @@
282282
}) distros
283283
));
284284

285-
# Package outputs
286-
packages =
287-
let
288-
basePackage = mkRustPackage {
289-
pname = "ros-z";
290-
buildAndTestSubdir = "ros-z";
291-
description = "Native Rust ROS 2 implementation using Zenoh - Core";
292-
};
293-
294-
# Generate packages for all distros
295-
allDistroPackages = builtins.foldl' (acc: distro: acc // (mkRosPackages distro)) { } distros;
296-
in
297-
{
298-
ros-z = basePackage;
299-
default = basePackage;
300-
}
301-
// allDistroPackages;
302-
303285
formatter = pkgs.nixfmt-rfc-style;
304286
}
305287
);

ros-z-msgs/build.rs

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -250,7 +250,10 @@ fn find_roslibrust_assets() -> PathBuf {
250250
}
251251
}
252252

253-
// Fallback: return the local path anyway, it will fail gracefully if it doesn't exist
254-
println!("cargo:warning=Could not find roslibrust assets, packages may not be available");
255-
local_path
253+
// Fallback: panic with helpful error message
254+
panic!(
255+
"Could not find roslibrust assets directory!\n\
256+
Make sure roslibrust is specified as a git dependency in Cargo.toml.\n\
257+
The build system searches: ~/.cargo/git/checkouts/roslibrust-*/*/assets/"
258+
);
256259
}

0 commit comments

Comments
 (0)