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

json-readtable-error 47 #39

Closed
Pi-Cla opened this issue Apr 27, 2023 · 8 comments
Closed

json-readtable-error 47 #39

Pi-Cla opened this issue Apr 27, 2023 · 8 comments

Comments

@Pi-Cla
Copy link

Pi-Cla commented Apr 27, 2023

I have installed Lean4 via elan and have set lean-rootdir to home/cla/.elan and then home/cla/.elan/ and both times when I try to use lean-toggle-show-goal with the default Main.lean file generated by lake init foo, it gives me the error: lean get info: (json-readtable-error 47)

@urkud
Copy link
Member

urkud commented Jul 13, 2023

Do you have Lean 3 mode installed?

@kisonecat
Copy link

I also saw this error when I had lean-mode installed; replacing it with lean4-mode fixed this for me.

@collares
Copy link
Contributor

You can have both packages, but the command to see goals in lean4-mode is lean4-toggle-info (bound to C-c TAB). The command you cited is from the lean-mode package (i.e., Lean 3).

@quinn-dougherty
Copy link

Notice: you also need to not have lean-company installed to avoid this error.

@mekeor
Copy link
Collaborator

mekeor commented Jul 21, 2024

You can have both packages, but the command to see goals in lean4-mode is lean4-toggle-info (bound to C-c TAB). The command you cited is from the lean-mode package (i.e., Lean 3).

I agree. lean4-mode does not define a command named lean-toggle-show-goal. The issue-report is just invalid and can be closed, in my opinion.

@mekeor
Copy link
Collaborator

mekeor commented Jul 21, 2024

Notice: you also need to not have lean-company installed to avoid this error.

That'd be because lean-company depends on lean-mode. Only then, the command lean-toggle-show-goal becomes available.

@quinn-dougherty
Copy link

I fixed this by thoroughly purging (with doom purge), since I had previously installed lean-mode and naively removing it didn't purge it thoroughly enough. So I'd expect people to keep running into it if they mistakenly install lean-mode before they realize they need lean4-mode.

mekeor added a commit that referenced this issue Dec 2, 2024
@mekeor
Copy link
Collaborator

mekeor commented Dec 2, 2024

This confusion is due to Lean3-Mode using lean- as prefix for symbols, for historical reasons. I explained this pitfall in the README and thus regard this ticket as resolved from our side. Feel free to reopen if not convinced. 0ad6796

@mekeor mekeor closed this as completed Dec 2, 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

6 participants