Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tyxml, ocsigenserver and eliom are incompatible with OCaml 4.00.0 #64

Closed
0branch opened this issue Sep 21, 2012 · 8 comments
Closed

tyxml, ocsigenserver and eliom are incompatible with OCaml 4.00.0 #64

0branch opened this issue Sep 21, 2012 · 8 comments

Comments

@0branch
Copy link
Contributor

0branch commented Sep 21, 2012

These packages reference Netstring_pcre which is no longer provided by ocamlnet.3.6. Given that ocamlnet.3.5.1 is incompatible with OCaml 4.00.0, these packages ought to declare ocaml-version: [ < "4.00.0" ].

Related question: any suggestions for the best patching strategy? One option would be to substitute Netstring_pcre with Netstring_str where applicable, repackage and distribute with a modified version number e.g., tyxml.2.1p. (I still need to test this for ocsigenserver and eliom, works for tyxml.) An alternative would be tracking the development heads of each project (via a Github mirror as they're versioned with darcs).

@benozol
Copy link

benozol commented Sep 21, 2012

Hello,
ocamlnet.3.6 still supports PCRE it is just not enabled by default anymore; from the ocamlnet's ChangeLog:

2012-07-19 Gerd Stolpmann gerd@samsung

* The default is now -disable-pcre

Frederic has a fork of opam-repository with a package ocamlnet-all which provides ocamlnet, version 3.6, with PCRE, though, waiting to be pulled.

Does this fix?

@0branch
Copy link
Contributor Author

0branch commented Sep 21, 2012

Aha, thanks for the info Benedikt—I overlooked that. Will try Frederic's repository now.

@samoht
Copy link
Member

samoht commented Sep 21, 2012

Yes @tuong repository contains some important fixes, we'll try to merge it today or at the beginning of next week.

@0branch
Copy link
Contributor Author

0branch commented Sep 21, 2012

Hmm, can't see ocamlnet-all in @tuong's repo (I'm probably doing something dumb). Instead, I added @benozol's fork as a remote and successfully built tyxml against the PCRE enabled ocamlnet.

Next problem: ocsigenserver requires Dbm (--with-dbm is a default configure flag). To build with 4.00.0, either:

  1. A dependency on CamlDBM should be added or
  2. --without-dbm should be supplied in ocsigenserver.2.1.opam

I think (1) is preferable?

@tuong
Copy link
Contributor

tuong commented Sep 21, 2012

It is the branch frederic-master : https://github.com/tuong/opam-repository/tree/frederic-master .
You can then opam install ocsigen_tutorial .

@0branch
Copy link
Contributor Author

0branch commented Sep 21, 2012

Thanks @tuong. I see the dbm stuff is addressed too—perhaps we should close this ticket?

@0branch
Copy link
Contributor Author

0branch commented Sep 21, 2012

(See also #59)

@tuong
Copy link
Contributor

tuong commented Sep 27, 2012

fixed by ed172cc

@tuong tuong closed this as completed Sep 27, 2012
gasche added a commit to gasche/opam-repository that referenced this issue Nov 24, 2017
  * Fix ppx_deriving_yojson.runtime META file
    (ocaml-ppx/ppx_deriving_yojson/ocaml#47)
    Étienne Millon
  * Support for inline records in variant types
    (ocaml-ppx/ppx_deriving_yojson/ocaml#50)
    Gerd Stolpmann
  * OCaml 4.06 compatibility
    (ocaml-ppx/ppx_deriving_yojson/ocaml#64, ocaml-ppx/ppx_deriving_yojson/ocaml#66)
    Leonid Rozenberg, Gabriel Scherer
ejgallego added a commit to ejgallego/opam-repository that referenced this issue Sep 23, 2018
- [serlib] Be more lenient when parsing back `Id.t` as to accommodate
  hacks in the Coq AST (ocaml#64) Thanks to @palmskog for the report.
NathanReb pushed a commit to NathanReb/opam-repository that referenced this issue Nov 26, 2021
add asn1/mirage-entropy-xen/nocrypto/tls/x509 which might work out of the box
NathanReb pushed a commit to NathanReb/opam-repository that referenced this issue Nov 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants