Add OCaml support - Githubissues

replit / prybar

Pry open those interpreters.

GNU General Public License v2.0

253 stars 52 forks source link

Add OCaml support #11

Closed ryyppy closed 5 years ago

ryyppy commented 5 years ago

[X] Make prybar-ocaml buildable
[X] Can output version string
[x] Can run script via -e arg
[x] Can run code without output via -c arg
[x] Can start simple repl via -i flag
[x] Will output ocaml version on startup, unless the -q flag is used
[x] Can use PS1 & PS2 flags for first-line prompt / continuation prompt
[x] Add tests
[x] Update Dockerfile

My strategy is to have a repl.ml script which decides on either eval'ing given input string or running a repl in interactive mode. So far the script will just be a standalone ocaml script without any deps.

basicer commented 5 years ago

Looking pretty good.

Can't wait for the tests / updated docker image : )

ryyppy commented 5 years ago

@basicer Fixed the few issues you found, except the bashrc one, since I don't know exactly how to resolve this until we get more concrete on the use cases of the container.

Also we now rely on ocaml 4.07.1 and dropped out the extra ocaml switch install, which makes the docker build for the ocaml part twice as fast

ryyppy commented 5 years ago

As discussed in our last hangout:

[x] Remove Reason dependency (for now)
[x] Use apt-get install ocaml and use 4.05.0 instead

ryyppy commented 5 years ago

Alright, so I added the changes, currently just commented out the Reason specific parts, since I am confident we can add Reason support rather quickly in the next steps :-)