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

[new release] websocket, websocket-lwt-unix and websocket-async (2.13) #13681

Merged
merged 2 commits into from
Apr 8, 2019

Conversation

vbmithr
Copy link
Contributor

@vbmithr vbmithr commented Mar 17, 2019

Websocket library

CHANGES:
  • Async: use logs_async
  • Async: upgrade_connection API (@kkazuo)
  • Lwt: fix upgrade_connection for TLS (API change)
  • Upgrade Base64 support to 3.1.0

@camelus
Copy link
Contributor

camelus commented Mar 17, 2019

☀️ All lint checks passed db7d5af
  • These packages passed lint tests: gemini.0.1, gemini.0.2.0, websocket-async.2.13, websocket-lwt-unix.2.13, websocket.2.13

☀️ Installability check (10930 → 10933)
  • new installable packages (3): websocket.2.13 websocket-async.2.13 websocket-lwt-unix.2.13

"org:mirage"
"org:xapi-project"
]
build: [ "dune" "build" "-j" jobs "-p" name "@install" ]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not a blocker at all but just for the information "@install" is not necessary

build: [ "dune" "build" "-j" jobs "-p" name "@install" ]
depends: [
"dune" {build & >= "1.1.1"}
"websocket" {= "2.12"}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this doesn't seem to be the right version number

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you can now use {=version} to make the constraint track the current package version

@kit-ty-kate
Copy link
Member

websocket-async.2.13 fails with:

#=== ERROR while compiling websocket-async.2.13 ===============================#
# context              2.0.3 | linux/x86_64 | ocaml-base-compiler.4.07.1 | pinned(https://github.com/vbmithr/ocaml-websocket/releases/download/2.13/websocket-2.13.tbz)
# path                 ~/.opam/4.07/.opam-switch/build/websocket-async.2.13
# command              ~/.opam/4.07/bin/dune build -j 71 -p websocket-async @install
# exit-code            1
# env-file             ~/.opam/log/websocket-async-20-d2e822.env
# output-file          ~/.opam/log/websocket-async-20-d2e822.out
### output ###
# File "async/dune", line 13, characters 25-44:
# 13 |  (libraries uri.services logs-async-reporter websocket-async))
#                               ^^^^^^^^^^^^^^^^^^^
# Error: Library "logs-async-reporter" not found.
# Hint: try: dune external-lib-deps --missing -j 71 -p websocket-async @install

And websocket-lwt.2.13 is also missing from this PR:

$ opam depext -uivyj 2 websocket-lwt-unix.2.13
# Detecting depexts using vars: arch=x86_64, os=linux, os-distribution=debian, os-family=debian
[ERROR] No solution for websocket-lwt-unix.2.13: The following dependencies couldn't be met:
          - websocket-lwt-unix -> websocket-lwt >= 2.13
              no matching version

@kit-ty-kate
Copy link
Member

There is an issue on 4.04 and 4.05:

#=== ERROR while compiling websocket-lwt-unix.2.13 ============================#
# context              2.0.3 | linux/x86_64 | ocaml-base-compiler.4.04.2 | pinned(https://github.com/vbmithr/ocaml-websocket/releases/download/2.13/websocket-2.13.tbz)
# path                 ~/.opam/4.04/.opam-switch/build/websocket-lwt-unix.2.13
# command              ~/.opam/4.04/bin/dune build -j 71 -p websocket-lwt-unix
# exit-code            1
# env-file             ~/.opam/log/websocket-lwt-unix-20-3730ca.env
# output-file          ~/.opam/log/websocket-lwt-unix-20-3730ca.out
### output ###
#     ocamldep lwt/.websocket_lwt_unix.objs/websocket_lwt_unix.mli.d (exit 2)
# (cd _build/default && /home/opam/.opam/4.04/bin/ocamldep.opt -modules -intf lwt/websocket_lwt_unix.mli) > _build/default/lwt/.websocket_lwt_unix.objs/websocket_lwt_unix.mli.d
# File "lwt/websocket_lwt_unix.mli", line 25, characters 20-22:
# Error: Syntax error

@kit-ty-kate
Copy link
Member

ping @vbmithr

@vbmithr vbmithr force-pushed the release-websocket-2.13 branch from 7a607ce to 9a4c062 Compare March 27, 2019 16:40
@kit-ty-kate
Copy link
Member

Could you have a look at the revdeps failures for disml and gimini? Those two were not failing before

CHANGES:

- Async: use logs_async
- Async: upgrade_connection API (@kkazuo)
- Lwt: fix upgrade_connection for TLS (API change)
- Upgrade Base64 support to 3.1.0
@vbmithr vbmithr force-pushed the release-websocket-2.13 branch from 9a4c062 to 5beed09 Compare April 3, 2019 09:45
@mseri
Copy link
Member

mseri commented Apr 5, 2019

@kit-ty-kate disml is fixed no, gimini is still failing with some async-related error. @vbmithr, do you know if this is related to this PR? Maybe it can be enough to add an upper bound to gimini opam files.

@vbmithr
Copy link
Contributor Author

vbmithr commented Apr 5, 2019

Yes gemini needs an upper bound…

@mseri
Copy link
Member

mseri commented Apr 5, 2019

Would you mind adding it to this PR? Otherwise we could merge and add it separately

@vbmithr
Copy link
Contributor Author

vbmithr commented Apr 6, 2019

Done… hope this is fine this time…

@kit-ty-kate
Copy link
Member

Thanks a lot!

@kit-ty-kate kit-ty-kate merged commit 65c1562 into ocaml:master Apr 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants