-
Notifications
You must be signed in to change notification settings - Fork 74
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
rosrust_msg takes a lot of time on each cargo build while messages definitions have not changed #176
Comments
MichiBab
added a commit
to MichiBab/rosrust
that referenced
this issue
Aug 28, 2022
fix from adnanademovic#176
From all of my testing of how this trigger works, the rerun trigger needs to include the hierarchy of folders that we're looking at, otherwise adding a new Overly eager rebuilds would be reduced by filtering out folders that we know won't contain message files, like |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I have using
rosrust
in a catkin workspace and use catkin to build the nodes using these lines in theCMakeList.txt
file, inspired from # 113But after any change in the
main.rs
file, the rust targetrosrust_msg
takes a lot of time to build.With the
-vv
flag ofcargo build
, I have noticed that the list of files printed withcargo:rerun-if-changed=
includes literally all the files of my ROS workspace, including.git
content for each package.Commenting the following lines fixed the issue and only the
.msg
and.srv
files are listed, which avoids therosrust_msg
rebuild after any change.rosrust/rosrust_msg/build.rs
Lines 84 to 86 in 2b75da0
I understand that using
rosrust
with catkin is not yet supported but maybe we can only add folders with namessrv
andmsg
in the list?The text was updated successfully, but these errors were encountered: