Re: Removing Camlp4 dependency from OPAM package

Dear Andre,

Thank you very much for you feedback.
I have now fixed configure and make clean.
Please tell me if there is anything wrong left.

Cheers,

Jacques

On 2016/03/08 03:08, Andre Maroneze wrote:
> 
> Thank you!
> 
> I tried the version from github (937f931), and despite a few issues with './configure' (which insisted that camlp4 should be installed, so I commented out a few lines) and 'make clean' (which removed a few extra files), I was able to compile and use it. It worked out of the box without doing ./configure, but because I needed to set the prefix for an opam installation, I had to run it and thus remove the lines related to camlp4.
> 
> Anyway, thank you for your responsiveness.
> 
> 
> On 03/06/2016 03:50 PM, Jacques Garrigue wrote:
>> I have just committed a fix that removes the dependency on camlp4 to the git repository (either github or ocamlforge).
>> This just adds the preprocessed files to the repository, so that one doesn't have to process them locally.
>> Can you give it a try?
>> 
>> Basically, camlp4 is only used for stream parsers, which were available in ocaml before the merge of camlp4
>> (and subsequent removal). It would be nice if they were supported again natively.
>> Note that this is a problem of parsing, so ppx is of no help there.
>> 
>> Cheers,
>> 
>> Jacques