|Anonymous | Login | Signup for a new account||2017-05-25 12:44 CEST|
|Main | My View | View Issues | Change Log | Roadmap|
|View Issue Details|
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0005492||OCaml||-for Camlp4 use https://github.com/ocaml/camlp4/issues||public||2012-01-23 16:09||2016-12-07 18:27|
|Priority||normal||Severity||feature||Reproducibility||have not tried|
|Target Version||Fixed in Version|
|Summary||0005492: testsuite does not check camlp4o compatibility|
|Description||Since any new construct in the language should be tested in the testsuite/, we should always verify that camlp4o generates correct code for these new constructs, to be sure that a new release is not shipped with no support for some new construct in camlp4o.|
A patch to the testsuite/ makefile is provided.
|Attached Files||testsuite_camlp4.patch [^] (1,322 bytes) 2012-01-23 16:09 [Show Content]|
The patch looks very incomplete to me, as it applies only to tests of the "several" kind.
It might be worthwhile to have a way to run the testsuite using camlp4o, but I wouldn't want this to be done at every run of the testsuite. One reason is that we need to be able to test new features before camlp4 is updated to support them. Another reason is that several of us developers configure with -no-camlp4 to speed up compiles during development.
Camlp4 is now a separate project. Please re-open a ticket on github  if you are still interested in seeing the original issue fixed.
 https://github.com/ocaml/camlp4/issues [^]
|2012-01-23 16:09||lefessan||New Issue|
|2012-01-23 16:09||lefessan||File Added: testsuite_camlp4.patch|
|2012-03-14 09:38||xleroy||Note Added: 0007069|
|2012-03-14 09:38||xleroy||Severity||minor => feature|
|2012-03-14 09:38||xleroy||Status||new => acknowledged|
|2012-03-14 09:38||xleroy||Category||OCaml documentation => Misc|
|2013-11-29 13:03||doligez||Category||Misc => Camlp4|
|2013-11-29 13:03||doligez||Tag Attached: patch|
|2016-02-08 12:49||doligez||Category||Camlp4 => for Camlp4 use https://github.com/ocaml/camlp4/issues/102 [^]|
|2016-02-08 12:50||doligez||Category||for Camlp4 use https://github.com/ocaml/camlp4/issues/102 [^] => for Camlp4 use https://github.com/ocaml/camlp4/issues [^]|
|2016-12-07 18:27||dim||Note Added: 0016772|
|2016-12-07 18:27||dim||Status||acknowledged => closed|
|2017-02-23 16:43||doligez||Category||for Camlp4 use https://github.com/ocaml/camlp4/issues [^] => -for Camlp4 use https://github.com/ocaml/camlp4/issues [^]|
|Copyright © 2000 - 2011 MantisBT Group|