Skip to content
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

add cargo watch or bacon to build tools #808

Closed
ilrudie opened this issue Feb 9, 2024 · 2 comments
Closed

add cargo watch or bacon to build tools #808

ilrudie opened this issue Feb 9, 2024 · 2 comments

Comments

@ilrudie
Copy link
Contributor

ilrudie commented Feb 9, 2024

This is to build consensus about adding cargo watch or bacon to our build tools image.

I personally like to work inside of make shell when possible for consistency. I also like using cargo watch and/or bacon to track changes and re-test/re-lint/rebuild automatically.

Upsides:

  • handy
  • easy to get with native rust tooling cargo install cargo-watch or cargo install --locked bacon

Downsides:

  • not strictly required for builds, possibly out of scope as a result
  • building native rust tooling like this takes time
  • adds ~5mb (bacon) or ~7mb (cargo-watch) to image
@howardjohn
Copy link
Member

Can you just install in a volume that is mounted via CONDITIONAL_HOST_MOUNTS? Then you can install the kitchen sink of whatever you want?

@ilrudie
Copy link
Contributor Author

ilrudie commented Feb 9, 2024

That's fair. Adding it into the image is not the only way

@ilrudie ilrudie closed this as completed Feb 19, 2024
@ilrudie ilrudie closed this as not planned Won't fix, can't repro, duplicate, stale Feb 19, 2024
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

No branches or pull requests

2 participants