Skip to content

enable submodules by default #1962

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

enable submodules by default #1962

wants to merge 1 commit into from

Conversation

domenkozar
Copy link
Member

@domenkozar domenkozar commented Jun 27, 2025

Fixes #1030

 /nix/store/ma2fs2a60gz04xsgxkdxqhqxxvinwn5h-nix-devenv-2.30.0pre20250626_afa41b0/bin/nix --extra-experimental-features flakes --option lazy-trees true --option warn-dirty false --keep-going --max-jobs 8 --option eval-cache false print-dev-env --profile /home/domen/dev/cachix/cdevenv/.devenv/gc/shell -vv --log-format internal-json
@nix {"action":"msg","level":4,"msg":"evaluating file '<nix/derivation-internal.nix>'"}
@nix {"action":"start","id":5924662571696128,"level":4,"parent":0,"text":"evaluating derivation 'git+file:///home/domen/dev/cachix/devenv#devShells.x86_64-linux.default'","type":0}
@nix {"action":"msg","level":4,"msg":"evaluating file '/home/domen/dev/cachix/devenv/.devenv.flake.nix'"}
nix: ../canon-path.cc:56: nix::CanonPath nix::CanonPath::removePrefix(const nix::CanonPath&) const: Assertion `isWithin(prefix)' failed.
Aborted (core dumped)

Copy link

cloudflare-workers-and-pages bot commented Jun 27, 2025

Deploying devenv with  Cloudflare Pages  Cloudflare Pages

Latest commit: b8786c2
Status: ✅  Deploy successful!
Preview URL: https://ec8501a0.devenv.pages.dev
Branch Preview URL: https://submodules.devenv.pages.dev

View logs

@cachix cachix deleted a comment from claude bot Jun 27, 2025
@sandydoo
Copy link
Member

sandydoo commented Jul 1, 2025

Oh, does this work now with the v2 nix? I forgot to re-test this.

@domenkozar
Copy link
Member Author

Oh, does this work now with the v2 nix? I forgot to re-test this.

It does not, Nix fails to find Cargo.toml in the submodule.

@cachix cachix deleted a comment from claude bot Jul 3, 2025
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.

git submodule imports
2 participants