-
Notifications
You must be signed in to change notification settings - Fork 437
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
lake: could not materialize package #5330
Comments
@jcommelin, I took the liberty of rewriting. :-) |
As an aside, in regards to this particular error, note that a package can use both an explicit source (i.e., require "leanprover-community" / "LeanSearchClient"
from git "https://github.com/leanprover-community/LeanSearchClient.git" @ "main" |
github-merge-queue bot
pushed a commit
that referenced
this issue
Nov 27, 2024
This PR improves the errors Lake produces when it fails to fetch a dependency from Reservoir. If the package is not indexed, it will produce a suggestion about how to require it from GitHub. Closes #5330.
kim-em
pushed a commit
that referenced
this issue
Nov 27, 2024
This PR improves the errors Lake produces when it fails to fetch a dependency from Reservoir. If the package is not indexed, it will produce a suggestion about how to require it from GitHub. Closes #5330.
kim-em
pushed a commit
that referenced
this issue
Nov 29, 2024
This PR improves the errors Lake produces when it fails to fetch a dependency from Reservoir. If the package is not indexed, it will produce a suggestion about how to require it from GitHub. Closes #5330.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
While working on mathlib branch bump/nightly-2024-09-11 (see leanprover-community/mathlib4#16716) we used the new Reservoir dependency syntax, but received the error message:
This error is both going to be reasonably common (whenever someone doesn't understand how reservoir works), and is not actionable.
Could we please add:
"
require
line with"
If the first bullet point is complicated to implement, we should implement the 2nd and 3rd bullet points in the meantime.
We could do even more here:
The text was updated successfully, but these errors were encountered: