-
-
Notifications
You must be signed in to change notification settings - Fork 6
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
Git dependency in extra-deps doesn't work if specified commit isn't reachable from remote HEAD #38
Comments
geo2a
pushed a commit
to runtimeverification/hs-backend-booster
that referenced
this issue
Nov 20, 2023
…ps (#376) Temporary fix for cdepillabout/stacklock2nix#38, until this is (hopefully) fixed upstream
cdepillabout
added a commit
that referenced
this issue
Nov 22, 2023
Checkout #38 for an explanation of where this is necessary. This could be slower for large repositories, but it should match `stack`'s default behavior better.
cdepillabout
added a commit
that referenced
this issue
Nov 22, 2023
Checkout #38 for an explanation of where this is necessary. This could be slower for large repositories, but it should match `stack`'s default behavior better.
Thanks for submitting this! I released https://github.com/cdepillabout/stacklock2nix/releases/tag/v4.0.0, which uses your suggested fix from runtimeverification/hs-backend-booster@62139cd. |
jberthold
pushed a commit
to runtimeverification/haskell-backend
that referenced
this issue
Apr 10, 2024
…ps (#376) Temporary fix for cdepillabout/stacklock2nix#38, until this is (hopefully) fixed upstream
jberthold
pushed a commit
to runtimeverification/haskell-backend
that referenced
this issue
Apr 10, 2024
…ps (#376) Temporary fix for cdepillabout/stacklock2nix#38, until this is (hopefully) fixed upstream
jberthold
pushed a commit
to runtimeverification/haskell-backend
that referenced
this issue
Apr 10, 2024
…ps (#376) Temporary fix for cdepillabout/stacklock2nix#38, until this is (hopefully) fixed upstream
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
For example, having something like:
in
extra-deps
fails with an error like:I think this is because of this
fetchGit
call:stacklock2nix/nix/build-support/stacklock2nix/default.nix
Lines 379 to 383 in 10d5316
Adding
allRefs = true
as the error suggests would solve this, but there's probably repos where it's much slower to do that, so maybe there's a cleverer solution, or it can just be made configurable?The text was updated successfully, but these errors were encountered: