Skip to content

Fix over-eager rebuilding of rosrust_msg when nothing changed#201

Open
DomWilliamsEE wants to merge 1 commit into
adnanademovic:masterfrom
DomWilliamsEE:master
Open

Fix over-eager rebuilding of rosrust_msg when nothing changed#201
DomWilliamsEE wants to merge 1 commit into
adnanademovic:masterfrom
DomWilliamsEE:master

Conversation

@DomWilliamsEE

Copy link
Copy Markdown

Fixes #176, where all files including inside .git are tracked by build.rs and triggers rebuilds too often.

@Alby407

Alby407 commented Sep 26, 2023

Copy link
Copy Markdown

Why has this not been merged so far?

@Alby407

Alby407 commented Feb 15, 2024

Copy link
Copy Markdown

@adnanademovic Could you check this PR and merge it?

@adnanademovic

adnanademovic commented Feb 16, 2024

Copy link
Copy Markdown
Owner

Wouldn't this introduce the opposite problem where updates to the .msg file contents won't trigger an update?

I'd be OK with explicitly filtering out any .git folder, as that would make sense.

@DomWilliamsEE

Copy link
Copy Markdown
Author

How about skipping hidden dirs then? Includes .idea and .git etc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

rosrust_msg takes a lot of time on each cargo build while messages definitions have not changed

3 participants