From: "Kim Nguyễn" <Kim.Nguyen@lri.fr>
To: Jeremie Dimino <jdimino@janestreet.com>
Cc: "Richard W.M. Jones" <rich@annexia.org>,
"caml-list@inria.fr" <caml-list@inria.fr>
Subject: Re: [Caml-list] Bug in camlp4 4.02 ? [ was: cduce on ocaml 4.02 ]
Date: Fri, 1 Aug 2014 18:31:22 +0200 [thread overview]
Message-ID: <CAJCP7wGBsgLgU1RAzJ+tm3U5j_dBDQQ118_aTHLjC2YkCf7V8A@mail.gmail.com> (raw)
In-Reply-To: <CANhEzE6g40QmueV1_c66t4bXDGVoW9U01A56UCsrKmNoEAj7rA@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 454 bytes --]
Hi
On Fri, Aug 1, 2014 at 5:11 PM, Jeremie Dimino <jdimino@janestreet.com> wrote:
> Could you fill an issue there: https://github.com/ocaml/camlp4 ?
Done, this is issue #42. As explained in the bug report, using the
ocamlfind package "camlp4.extend" fixes the issue for us.
For those interested I'm attaching a patch that allows one to compile
cduce with the current ocaml trunk (I only tested using the
4.02.0+trunk switch of opam).
Cheers,
--
Kim
[-- Attachment #2: cduce_ocaml_4.02.0.patch --]
[-- Type: text/x-patch, Size: 1209 bytes --]
diff -uprNZ cduce-0.6.0/Makefile.distrib cduce-0.6.1/Makefile.distrib
--- cduce-0.6.0/Makefile.distrib 2014-03-05 14:49:17.000000000 +0100
+++ cduce-0.6.1/Makefile.distrib 2014-08-01 18:15:14.606679963 +0200
@@ -17,7 +17,7 @@ ifeq ($(NATIVE),true)
all: cduce_lib.cmxa
endif
-PACKAGES = dynlink camlp4 ulex pcre num netstring
+PACKAGES = dynlink camlp4 camlp4.extend ulex pcre num netstring
# Call make with VERBOSE=true to get a trace of commands
@@ -41,14 +41,14 @@ SYNTAX_PARSER = -syntax camlp4o $(SYNTAX
CAMLC_P = ocamlc -g
DEPEND_OCAMLDEP = misc/q_symbol.cmo
ifeq ($(PROFILE), true)
- CAMLOPT_P = ocamlopt -p -inline 10000
+ CAMLOPT_P = ocamlopt -p -inline 100
ifeq ($(NATIVE), false)
CAMLC_P = ocamlcp -p a
SYNTAX_PARSER =
DEPEND_OCAMLDEP =
endif
else
- CAMLOPT_P = ocamlopt -inline 10000
+ CAMLOPT_P = ocamlopt -inline 100
endif
OPT = -warn-error FPSXY
diff -uprNZ cduce-0.6.0/parser/parser.ml cduce-0.6.1/parser/parser.ml
--- cduce-0.6.0/parser/parser.ml 2014-03-05 14:49:17.000000000 +0100
+++ cduce-0.6.1/parser/parser.ml 2014-08-01 18:14:52.822680449 +0200
@@ -1,4 +1,3 @@
-#load "pa_extend.cmo";;
open Cduce_loc
(* let raise = Pervasives.raise *)
next prev parent reply other threads:[~2014-08-01 16:32 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-08-01 11:03 Kim Nguyễn
2014-08-01 14:12 ` Richard W.M. Jones
2014-08-01 15:11 ` Jeremie Dimino
2014-08-01 16:31 ` Kim Nguyễn [this message]
2014-08-02 12:35 ` Richard W.M. Jones
2014-08-02 13:11 ` Kim Nguyễn
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CAJCP7wGBsgLgU1RAzJ+tm3U5j_dBDQQ118_aTHLjC2YkCf7V8A@mail.gmail.com \
--to=kim.nguyen@lri.fr \
--cc=caml-list@inria.fr \
--cc=jdimino@janestreet.com \
--cc=rich@annexia.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).