Improved Ocaml checker to support OcamlC and Jane Street Core #327

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
2 participants
@adinapoli
Contributor

adinapoli commented Aug 23, 2012

Hi,

I'm pumped up the OCaml checker. Now it's possible to set three new different flags:

let g:syntastic_ocaml_use_ocamlc = 1

Tells syntastic to switch over ocamlc as preferred checker.

let g:syntastic_ocaml_use_janestreet_core = 1
let g:syntastic_ocaml_janestreet_core_dir = <path>

These two are very handy and extends the checking to modules imported from Jane Street's Core, a replacement for the standard library.

Cheers,
Alfredo

Alfredo Di Napoli added some commits Aug 23, 2012

@scrooloose

This comment has been minimized.

Show comment Hide comment
@scrooloose

scrooloose Aug 31, 2012

Collaborator

I have pulled these commits and added another on top to do some refactoring - i.e. to clean up the makeprg generation code.

Unfortunetly, i dont actually use ocaml :) and didnt check that my refactoring didnt break anything. Want to test it out for me?

here is the commit: c5b985b

Collaborator

scrooloose commented Aug 31, 2012

I have pulled these commits and added another on top to do some refactoring - i.e. to clean up the makeprg generation code.

Unfortunetly, i dont actually use ocaml :) and didnt check that my refactoring didnt break anything. Want to test it out for me?

here is the commit: c5b985b

@scrooloose scrooloose closed this Aug 31, 2012

@adinapoli

This comment has been minimized.

Show comment Hide comment
@adinapoli

adinapoli Sep 1, 2012

Contributor

Sure, I'll test that everything is ok and then I'll commit again if necessary :)

Contributor

adinapoli commented Sep 1, 2012

Sure, I'll test that everything is ok and then I'll commit again if necessary :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment