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

installing tree-sitter.dev~ is super slow #166

Closed
aryx opened this issue Feb 24, 2021 · 3 comments
Closed

installing tree-sitter.dev~ is super slow #166

aryx opened this issue Feb 24, 2021 · 3 comments
Assignees
Labels
priority:low not on the roadmap for now

Comments

@aryx
Copy link
Collaborator

aryx commented Feb 24, 2021

make setup, and some of our jobs in CI take a long time partly because of the 'install tree-sitter.dev~' step.
Is there something we can do to speedup that step? Why is it so slow?

@mjambon mjambon added the priority:medium to do, not blocking users label Mar 29, 2021
@mjambon mjambon self-assigned this Mar 29, 2021
@mjambon
Copy link
Member

mjambon commented Mar 29, 2021

Is this still the case? Are you referring to ocaml-tree-sitter CI or semgrep CI? Which jobs?

@aryx aryx added priority:low not on the roadmap for now and removed priority:medium to do, not blocking users labels Mar 30, 2021
@aryx
Copy link
Collaborator Author

aryx commented Mar 30, 2021

I was referring to 'make setup' when done locally on my linux (or macbook) machine in semgrep, or semgrep CI.

If you do 'make setup' in semgrep/, the step below takes more than 5 minutes (all the rest is pretty fast):

tree-sitter.~dev] no changes from git+file:///home/pad/github/semgrep/ocaml-tree-sitter#HEAD
The following actions will be performed:
  ∗ install tree-sitter ~dev*
Processing  1/1: [tree-sitter.~dev: git]

Not sure what is doing OPAM here. Seems busy with calls to 'git' and 'git-remote-http' when using top to see the list of processes.
Maybe we could use again some git shallow clone somewhere to speedup things.

In fact it is so slow, and because it does not even show any progress bad, I think a user could be very tempted to interrupt 'make setup' (I know I did in the past).

@mjambon
Copy link
Member

mjambon commented Jun 29, 2021

This problem will go away when I'm done with #204

@mjambon mjambon closed this as completed Sep 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority:low not on the roadmap for now
Projects
None yet
Development

No branches or pull requests

2 participants