coccinelle: Reuse devel/menhir and also depends on pcre-ocaml

[Leonardo Taccari <leot%NetBSD.org@localhost>]

Module Name:	pkgsrc-wip
Committed By:	Leonardo Taccari <leot%NetBSD.org@localhost>
Pushed By:	leot
Date:		Tue Oct 10 11:42:00 2017 +0200
Changeset:	877bbda310e7574c61d2e57757dbe100cd3ec4dc

Modified Files:
	coccinelle/Makefile
	coccinelle/TODO

Log Message:
coccinelle: Reuse devel/menhir and also depends on pcre-ocaml

- Depends on devel/menhir, it seems that now it works
- Add a dependency on devel/pcre-ocaml, at least with current ocaml it seems
  that the test failed because they needed pcre-ocaml. Now the tests passes but
  we have a bit lower score than the one expected.
- Update TODO accordingly

To see a diff of this commit:
https://wip.pkgsrc.org/cgi-bin/gitweb.cgi?p=pkgsrc-wip.git;a=commitdiff;h=877bbda310e7574c61d2e57757dbe100cd3ec4dc

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

diffstat:
 coccinelle/Makefile | 7 ++++---
 coccinelle/TODO     | 7 ++++---
 2 files changed, 8 insertions(+), 6 deletions(-)

diffs:
diff --git a/coccinelle/Makefile b/coccinelle/Makefile
index 18cb2d514f..cf64d448da 100644
--- a/coccinelle/Makefile
+++ b/coccinelle/Makefile
@@ -12,9 +12,9 @@ LICENSE=	gnu-gpl-v2
 
 BUILD_DEPENDS+=	ocaml-findlib-[0-9]*:../../devel/ocaml-findlib
 
-# the current version rejects this package's .mly file
-#DEPENDS+=	menhir>=20071212:../../devel/menhir
-CONFIGURE_ARGS+=	--without-menhir
+DEPENDS+=	menhir>=20071212:../../devel/menhir
+
+CONFIGURE_ARGS+=	--with-menhir
 CONFIGURE_ARGS+=	--disable-pyml
 
 MAKE_JOBS_SAFE=		no
@@ -45,6 +45,7 @@ TEST_TARGET=	test
 
 BUILDLINK_TRANSFORM+=	opt:-ldl:${BUILDLINK_LDADD.dl:M*}
 
+.include "../../devel/pcre-ocaml/buildlink3.mk"
 BUILDLINK_API_DEPENDS.ocaml+=	ocaml>=3.11
 .include "../../lang/ocaml/buildlink3.mk"
 .include "../../lang/camlp4/buildlink3.mk"
diff --git a/coccinelle/TODO b/coccinelle/TODO
index 0f00de7e1d..ed191503dc 100644
--- a/coccinelle/TODO
+++ b/coccinelle/TODO
@@ -1,7 +1,8 @@
 Still not ready for using it to update devel/coccinelle. The following issues
 should be investigated further and if possible fixed:
 
- - pkgsrc `test' target no longer works (manually invoking that on ${WRKSRC}
-   does the trick, I guess that some naughty things happen when
-   compiling/linking test programs)
  - Update to coccinelle-1.0.6 breaks textproc/coccigrep
+
+   XXXleot: provide a simple command to test that.
+    
+    -- leot (2017/10/10)