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

Release 2.8 #114

Closed
vasilisp opened this issue Nov 8, 2016 · 12 comments
Closed

Release 2.8 #114

vasilisp opened this issue Nov 8, 2016 · 12 comments

Comments

@vasilisp
Copy link
Contributor

vasilisp commented Nov 8, 2016

Let's get it out in the next few days!

Major changes:

  • New Postgres Ocsipersist backend, OR switch to the standalone Ocsipersist (with Postgres added)
  • Compatibility with TyXML 4.0.x.

We may also want to merge #87. Anything else?

@vasilisp
Copy link
Contributor Author

vasilisp commented Nov 8, 2016

I can take care of the Ocsipersist switch, it needs to happen (finally).

@vasilisp
Copy link
Contributor Author

vasilisp commented Nov 8, 2016

We may also want to convert to PPX and eliminate the Camlp4 dependency. We are only using the Lwt Camlp4 syntax, I hope?

@dannywillems
Copy link
Member

dannywillems commented Nov 8, 2016

@vasilisp Seems Lwt Camlp4 syntax is used in:

  • src/baselib/ocsigen_message.ml
  • src/extensions/ocsipersist-psql/ocsipersist.ml
  • src/baselib/ocsigen_lib_base.ml

@vasilisp
Copy link
Contributor Author

vasilisp commented Nov 8, 2016

@dannywillems, I know, PR coming :).

@vasilisp
Copy link
Contributor Author

vasilisp commented Nov 8, 2016

See #115.

@dannywillems
Copy link
Member

@vasilisp Compatibility with 4.04 must be also verified.

@dannywillems
Copy link
Member

dannywillems commented Nov 8, 2016

It seems there is a problem with the OPAM ssl package (which is available only for OCaml < 4.04.0). But I think, as it's not caused by ocsigenserver, it must be OK.

@vasilisp
Copy link
Contributor Author

vasilisp commented Nov 8, 2016

@dannywillems, I have gotten everything (up to ocsigen-start) compiling with an opam-friendly ssl branch: https://github.com/vasilisp/ocaml-ssl/tree/pin-friendly . Haven't tested much.

@dannywillems
Copy link
Member

@vasilisp I think it could be interesting to do a PR.

@vasilisp
Copy link
Contributor Author

vasilisp commented Nov 8, 2016

@dannywillems, it is not done well, I just put the auto-generated files in the repo as a temporary workaround. They are not supposed to be there. This is the standard autotools annoyance...

@vasilisp
Copy link
Contributor Author

vasilisp commented Nov 9, 2016

I think the Ocsipersist split gets postponed, see ocsigen/ocsipersist#4.

@vasilisp
Copy link
Contributor Author

Tagged. opam-repository PR: ocaml/opam-repository#7914

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

2 participants