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

Makefile enhancements #43

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

zklinger
Copy link

The Motivation

To make building the parser for generating documentation for telescope.nvim on a local machine a touch simpler.

Currently there are some manual steps required to set up the parser directory before generating the parser and later copying/sym linking the parser.so to the parser directory.

This PR is trying to streamline that.

The Changes

  • make node tree-sitter a build dependency and install it if it's not installed yet
  • create the parser directory if it does not exist yet
  • after building the parser, copy the generated parser.so to the parser/lua.so

Notes

I have removed the following lines from the Makefile

ifeq (, ${ts})
    ts := $(shell which tree-sitter 2> /dev/null)
endif

ifeq (, ${ts})
    ts := $(shell which tree-sitter-cli 2> /dev/null)
endif

as the above conditionals always evaluate to false because ${ts} is never empty (it has been set to "./node_modules/tree-sitter-cli/tree-sitter" on line 1).

@zklinger zklinger changed the title Make file enhancements Makefile enhancements Nov 15, 2021
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.

1 participant