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

Avoid rebuilds #20

Draft
wants to merge 11 commits into
base: master
Choose a base branch
from
Draft

Avoid rebuilds #20

wants to merge 11 commits into from

Conversation

srid
Copy link
Owner

@srid srid commented Aug 23, 2023

resolves #12

  • nix run is cached
  • nix flake check (clippy) is cached

@srid
Copy link
Owner Author

srid commented Aug 23, 2023

While this does work, it breaks the cargo-leptos generated wasm binary. server fns, and signals are broken. logging doesn't work as well. But route switches work.

@srid srid added the help wanted Extra attention is needed label Sep 5, 2023
@srid
Copy link
Owner Author

srid commented Sep 5, 2023

This approach suffers from another problem. It is not gonna work with projects using cargo workspaces, because the dummy source obviously doesn't include them, yet Cargo.toml can refer to the local crates (such as leptos_extra from https://github.com/juspay/nix-browser)

Ref:

       >
       > Caused by:
       >     `cargo metadata` exited with an error: error: failed to load manifest for dependency `leptos_extra`
       >
       >     Caused by:
       >       failed to read `/private/tmp/nix-build-nix-browser-deps-0.1.0.drv-1/dummy-src/crates/leptos_extra/Cargo.toml`

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Avoid full rebuilds
1 participant