Opened 4 months ago

Closed 4 months ago

#70174 closed defect (fixed)

geneweb: install fails

Reported by: ctreleaven (Craig Treleaven) Owned by: pguyot (Paul Guyot)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc:
Port: geneweb

Description

First, thanks for putting in what must have been a huge amount of work on bringing our Geneweb port up to date. However, Geneweb fails to install on my Intel 10.15.7 system:

...
Building executables
dune build -p geneweb --profile dev
File "bin/ged2gwb/dune", line 5, characters 22-84:
5 |   (preprocess (action (run camlp5o pr_o.cmo pa_extend.cmo q_MLast.cmo %{input-file})))
                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
(cd _build/default && /opt/local/bin/camlp5o pr_o.cmo pa_extend.cmo q_MLast.cmo bin/ged2gwb/ged2gwb.ml) > _build/default/bin/ged2gwb/ged2gwb.pp.ml
Error while loading "/opt/local/lib/ocaml/camlp5/pr_o.cmo": no implementation available for O_keywords.
make: *** [build] Error 1
make: Leaving directory `/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_tarballs_ports_genealogy_geneweb/geneweb/work/geneweb-7.1-beta'
Command failed:  cd "/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_tarballs_ports_genealogy_geneweb/geneweb/work/geneweb-7.1-beta" && /usr/bin/make -w distrib 
Exit code: 2
Error: Failed to build geneweb: command execution failed
DEBUG: Error code: CHILDSTATUS 58567 2

All of the dependencies appear to have installed without issue. I tried to clean the port and then install but got the same error message.

I'll attach the full log.

Attachments (1)

geneweb_install_fail_5Jun2024.txt (131.2 KB) - added by ctreleaven (Craig Treleaven) 4 months ago.

Download all attachments as: .zip

Change History (7)

Changed 4 months ago by ctreleaven (Craig Treleaven)

comment:1 Changed 4 months ago by ctreleaven (Craig Treleaven)

Owner: set to pguyot

comment:2 Changed 4 months ago by pguyot (Paul Guyot)

Thank you for the report. geneweb is indeed broken with latest camlp5. See: https://github.com/geneweb/geneweb/pull/1806

I am still working on a fix (the PR above still fails somehow locally).

comment:3 Changed 4 months ago by pguyot (Paul Guyot)

There is a fix in https://github.com/macports/macports-ports/pull/24362 which works locally and passes CI. Would you mind testing on your system? You might need to reinstall ocaml-ppx_blob and to install the other two ports added by the PR.

comment:4 Changed 4 months ago by ctreleaven (Craig Treleaven)

Thanks for the quick response. Given that the port is broken, would you like me to commit your PR? I've reviewed to the extent that I can (knowing essentially nothing about ocaml) and the proposed changes seem minimal.

BTW, is the main English-based support for Geneweb on Github? It appears that there is a separate mailing list and discussion forum but both appear to be in French. My French is far too weak for that to be a viable option for me.

comment:5 Changed 4 months ago by pguyot (Paul Guyot)

If this fix works for you, please feel free to go ahead and merge the PR. If I got the commit message right it shall close this ticket.

I'm not aware of geneweb support channels. I'm just a user myself and not related to the dev team at all.

There is a bilingual mailing list - https://framalistes.org/sympa/arc/geneweb - but it seems pretty inactive, with only three messages in 2023. The forum is much more active but mostly in French as you noticed. I would raise any issue on GitHub indeed.

comment:6 Changed 4 months ago by pguyot (Paul Guyot)

Resolution: fixed
Status: assignedclosed

In 23728b4ad367361e91a866caf54bf778ec3ff841/macports-ports (master):

Fix geneweb incompatibility with camlp5 8.03.0 (https://github.com/macports/macports-ports/pull/24362)

  • ocaml-ocamlgraph: new port 2.1.0

Signed-off-by: Paul Guyot <pguyot@…>

  • ocaml-not-ocamlfind: new port 0.13

Signed-off-by: Paul Guyot <pguyot@…>

  • ocaml-ppx_blob: increase revision as ocaml-ppxlib was updated

Signed-off-by: Paul Guyot <pguyot@…>

  • geneweb: fix incompatibility with camlp5 8.03.0
  • Add upstream patch in PR1806
  • Add ocaml-notocamlfind as a dependency required by upstream patch

Fixes #70174

Signed-off-by: Paul Guyot <pguyot@…>


Signed-off-by: Paul Guyot <pguyot@…>

Note: See TracTickets for help on using tickets.