You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As suggested here, let's make ocaml-tree-sitter the main command for everything, including:
what ocaml-tree-sitter currently does: generate ocaml code from grammar.json
simplify-grammar
gen-ocaml, a bash script
gen-c, a bash script
possibly a combination of the above that does everything
Tasks include:
finding good names for the subcommands
locating the external executables to be called (the bash scripts): should we rely on $PATH to locate them or something else?
adding subcommands to the cmdliner interface. See cmdliner cheatsheet for how to add subcommands.
Compatibility with the previous ocaml-tree-sitter command line is not important, since as far as I know, the only user is semgrep and all that code exists in the ocaml-tree-sitter repo (the semgrep build does not call ocaml-tree-sitter).
The text was updated successfully, but these errors were encountered:
Some of this work was done (#202) already: simplify is now a subcommand of ocaml-tree-sitter. What's left I think is to add subcommands for gen-c and gen-ocaml, which would call the commands ocaml-tree-sitter-gen-c and ocaml-tree-sitter-gen-ocaml respectively.
mjambon
changed the title
Create subcommands for 'ocaml-tree-sitter'
Create subcommands for 'ocaml-tree-sitter-gen-c' and ''ocaml-tree-sitter-gen-ocaml'
Jun 21, 2021
As suggested here, let's make
ocaml-tree-sitter
the main command for everything, including:ocaml-tree-sitter
currently does: generate ocaml code fromgrammar.json
simplify-grammar
gen-ocaml
, a bash scriptgen-c
, a bash scriptTasks include:
$PATH
to locate them or something else?Compatibility with the previous
ocaml-tree-sitter
command line is not important, since as far as I know, the only user is semgrep and all that code exists in the ocaml-tree-sitter repo (the semgrep build does not callocaml-tree-sitter
).The text was updated successfully, but these errors were encountered: