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

Package ocsigenserver.2.15.0 #14685

Merged
merged 2 commits into from
Aug 28, 2019

Conversation

jrochel
Copy link
Contributor

@jrochel jrochel commented Aug 16, 2019

ocsigenserver.2.15.0

A full-featured and extensible Web server
Ocsigen Server implements most features of the HTTP protocol, and has a very powerful extension mechanism that makes it very easy to plug your own OCaml modules for generating pages. Many extensions are already implemented, like a reverse proxy, content compression, access control, authentication, etc.



🐫 Pull-request generated by opam-publish v2.0.0

@avsm
Copy link
Member

avsm commented Aug 18, 2019

Is there a missing dependency here? Looks like upstream packages are breaking due to

# ocamlfind: Package `lwt_ssl' not found - required by `ocsigenserver.http'

@jrochel
Copy link
Contributor Author

jrochel commented Aug 19, 2019

@avsm : In the logs of bitcoin, it seems to me that the problematic line would be

- remove  lwt           4.2.1   [conflicts with bitcoin]

So the relevant dependency chain for bitcoin look like this: bitcoin →? ocsigenserver → lwt_ssl → lwt
At the same time bitcoin is in conflict with lwt.
I'm curious to know what @darioteixeira has to say about this.

@kit-ty-kate
Copy link
Member

The problem with the bitcoin package seems to be the same than the revdeps issue with eliom (i.e. the package is not uninstalled properly):

#=== ERROR while installing ocsigenserver.2.15.0 ==============================#
# context              2.0.5 | linux/x86_64 | ocaml-base-compiler.4.08.1 | pinned(https://github.com/ocsigen/ocsigenserver/archive/2.15.0.tar.gz)
# path                 ~/.opam/4.08/.opam-switch/build/ocsigenserver.2.15.0
# command              /usr/bin/make install
# exit-code            2
# env-file             ~/.opam/log/ocsigenserver-17-0b5740.env
# output-file          ~/.opam/log/ocsigenserver-17-0b5740.out
### output ###
# /usr/bin/make -C src install
# make[1]: Entering directory '/home/opam/.opam/4.08/.opam-switch/build/ocsigenserver.2.15.0/src'
# # Server binaries
# install -m 755 -d /home/opam/.opam/4.08/bin
# install -m 755 server/ocsigenserver server/ocsigenserver.opt /home/opam/.opam/4.08/bin/
# # Server libraries
# install -m 755 -d /home/opam/.opam/4.08/lib
# ocamlfind install ocsigenserver \
#    -destdir /home/opam/.opam/4.08/lib \
#    -patch-version 2.9 \
#    baselib/ocsigen_cache.cmi baselib/ocsigen_lib_base.cmi baselib/ocsigen_lib.cmi baselib/ocsigen_config.cmi baselib/ocsigen_messages.cmi baselib/ocsigen_stream.cmi baselib/ocsigen_loader.cmi baselib/polytables.cmi http/http_headers.cmi http/ocsigen_http_frame.cmi http/ocsigen_headers.cmi http/framepp.cmi http/ocsigen_http_com.cmi http/ocsigen_charset_mime.cmi http/ocsigen_senders.cmi http/ocsigen_cookies.cmi server/ocsigen_extensions.cmi server/ocsigen_parseconfig.cmi server/ocsigen_http_client.cmi server/ocsigen_local_files.cmi server/ocsigen_server.cmi server/ocsigen_request_info.cmi baselib/ocsigen_getcommandline.cmi baselib/ocsigen_cache.cmx baselib/ocsigen_lib_base.cmx baselib/ocsigen_lib.cmx baselib/ocsigen_config.cmx baselib/ocsigen_messages.cmx baselib/ocsigen_stream.cmx baselib/ocsigen_loader.cmx baselib/polytables.cmx http/http_headers.cmx http/ocsigen_http_frame.cmx http/ocsigen_headers.cmx http/framepp.cmx http/ocsigen_http_com.cmx http/ocsigen_charset_mime.cmx http/ocsigen_senders.cmx http/ocsigen_cookies.cmx server/ocsigen_extensions.cmx server/ocsigen_parseconfig.cmx server/ocsigen_http_client.cmx server/ocsigen_local_files.cmx server/ocsigen_server.cmx server/ocsigen_request_info.cmx baselib/ocsigen_lib_base.cmo baselib/baselib.cma baselib/parsecommandline.cma baselib/donotparsecommandline.cma baselib/polytables.cmo http/ocsigen_cookies.cmo http/http.cma server/ocsigenserver.cma server/server_main.cmo  baselib/ocsigen_lib_base.cmx baselib/polytables.cmx http/ocsigen_cookies.cmx server/server_main.cmx baselib/ocsigen_lib_base.o   baselib/polytables.o   http/ocsigen_cookies.o   server/server_main.o   baselib/baselib.cmxa baselib/parsecommandline.cmxa baselib/donotparsecommandline.cmxa http/http.cmxa server/ocsigenserver.cmxa baselib/baselib.a baselib/parsecommandline.a baselib/donotparsecommandline.a http/http.a server/ocsigenserver.a  baselib/parsecommandline.cmxs baselib/donotparsecommandline.cmxs baselib/polytables.cmxs baselib/ocsigen_cache.mli baselib/ocsigen_lib_base.mli baselib/ocsigen_lib.mli baselib/ocsigen_config.mli baselib/ocsigen_messages.mli baselib/ocsigen_stream.mli baselib/ocsigen_loader.mli baselib/polytables.mli http/http_headers.mli http/ocsigen_http_frame.mli http/ocsigen_headers.mli http/framepp.mli http/ocsigen_http_com.mli http/ocsigen_charset_mime.mli http/ocsigen_senders.mli http/ocsigen_cookies.mli server/ocsigen_extensions.mli server/ocsigen_parseconfig.mli server/ocsigen_http_client.mli server/ocsigen_local_files.mli server/ocsigen_server.mli server/ocsigen_request_info.mli baselib/ocsigen_getcommandline.mli files/META
# ocamlfind: Package ocsigenserver is already installed
#  - (file /home/opam/.opam/4.08/lib/ocsigenserver/META already exists)
# make[1]: *** [Makefile:120: raw_install] Error 2
# make[1]: Leaving directory '/home/opam/.opam/4.08/.opam-switch/build/ocsigenserver.2.15.0/src'
# make: *** [Makefile:51: install.META] Error 2

However that should not happen and I'm very surprised about this. @rjbou @AltGr would you know why opam didn't uninstalled ocsigenserver properly?

@darioteixeira
Copy link
Contributor

This may be tangential to the issue at hand, but I intend to split the bitcoin monolith into the new OPAM convention of separate packages for each of the engines (bitcoin-cohttp, bitcoin-ocurl, etc). I will be away on a conference followed by holidays, so this won't happen before the 2nd week of September, though.

@rjbou
Copy link
Contributor

rjbou commented Aug 20, 2019

@kit-ty-kate For an obscure reason, ocsigenserver files have changed since last install (from eliom log. As opam revert its install file operations (changes files), if files/directories have changed since install, it keeps them as is.

  - recompile ocsigenserver                 2.15.0*    [uses tyxml]
[...]
-> removed   ocsigenserver.2.15.0
[WARNING] While removing ocsigenserver.2.15.0: not removing files that changed since:
            - man/man1/ocsigenserver.1
            - lib/ocsigenserver/var/www/ocsigenstuff/video.png
            - lib/ocsigenserver/var/www/ocsigenstuff/unknown.png
[...]
            - lib/ocsigenserver/baselib.cmxa
            - lib/ocsigenserver/baselib.cma
            - lib/ocsigenserver/baselib.a
            - lib/ocsigenserver/META

[NOTE] While removing ocsigenserver.2.15.0: not removing non-empty directories:
         - lib/ocsigenserver/var/www/ocsigenstuff
         - lib/ocsigenserver/var/run
         - lib/ocsigenserver/extensions
         - lib/ocsigenserver/etc/ocsigenserver

@camelus
Copy link
Contributor

camelus commented Aug 21, 2019

☀️ All lint checks passed 64e29b2
  • These packages passed lint tests: ocsigenserver.2.15.0

☀️ Installability check (+1)
  • new installable packages (1): ocsigenserver.2.15.0

@kit-ty-kate
Copy link
Member

@jrochel would you happen to know why this happens?

@kit-ty-kate
Copy link
Member

I can't reproduce it locally (with or without docker), I really don't understand what's going on here. Let's just merge it and maybe fix it later if somebody comes up with an explanation.

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

Successfully merging this pull request may close these issues.

6 participants