[bug#64209] [PATCH v2 00/14] Reorder commits and better descriptions

[Benjamin <benjamin@uvy.fr>]

Hi

I’am updating, my first series of patches.

I bumped ocaml-lsp-server to latest patch

> There seems to be an issue with the ordering of the patches. I can not
> build ocaml-dune-rpc, as it requires ocaml-ordering which is only
> available a couple of commits later.

I reordered commits so that each package can be built for each commit

> Could you explain why the 'remove-vendor phase is needed? It seems like
> if code is vendored it should maybe also be removed in the `dune`
> package?

I did this because I saw that opam build is doing this way :
https://github.com/ocaml/dune/blob/main/dyn.opam#L21-L22

If those vendor files are not remove, there was some errors when
building ocaml-stdune

```
File "otherlibs/stdune/path.mli", line 1:
Error: The files otherlibs/stdune/.stdune.objs/byte/stdune__Path_intf.cmi
       and /gnu/store/wig75gdqm284z6y89dcd75fmxzim307a-ocaml5.0-pp-1.1.2/lib/ocaml/site-lib/pp/pp.cmi
       make inconsistent assumptions over interface Pp
error: in phase 'build': uncaught exception:
%exception #<&invoke-error program: "dune" arguments: ("build" "@install" "-p" "stdune") exit-status: 1 term-signal: #f stop-signal: #f>
phase `build' failed after 1.8 seconds
command "dune" "build" "@install" "-p" "stdune" failed with status 1
builder for `/gnu/store/cb2n85al5bqn6q3pcaiil01p9wvfbpvw-ocaml5.0-stdune-3.6.1.drv' failed with exit code 1
build of /gnu/store/cb2n85al5bqn6q3pcaiil01p9wvfbpvw-ocaml5.0-stdune-3.6.1.drv failed
```

Because it has been repeated for each dune sublibraries, it probably
make sense to share this part of the code.
Though I don't know if we can directly do that for dune as those
vendoring are not removed here https://github.com/ocaml/dune/blob/main/dune.opam

> We need a more descriptive description.

> I guess the descriptions come from OPAM. Unfortunately Guix has stricter
> requirements than OPAM...:)

You are right, I just took descriptions from opam and did not edit them
I  updated some of the description and synopsis. Tell if it is enough.
As I don't exactly know what packages are doing, I just picked up descriptions
from source files comments

Tell me if you have other comments on those patches

Best

Benjamin

Benjamin (14):
  gnu: Fix ocaml-dune-build-info
  gnu: Add ocaml-pp.
  gnu: Add ocamlformat-rpc-lib.
  gnu: Add ocaml-ppx-yojson-conv-lib.
  gnu: Add ocaml-ordering.
  gnu: Add ocaml-dyn.
  gnu: Add ocaml-stdune.
  gnu: Add ocaml-chrome-trace.
  gnu: Add ocaml-xdg.
  gnu: Add ocamlc-loc.
  gnu: Add ocaml-fiber.
  gnu: Add ocaml-dune-rpc.
  gnu: Add ocaml-merlin-lib-4.9.
  gnu: Add ocaml-lsp-server.

 gnu/packages/ocaml.scm | 295 ++++++++++++++++++++++++++++++++++++++++-
 1 file changed, 294 insertions(+), 1 deletion(-)


base-commit: b20e5bcafd9654a767ffb78eefd7494f73ca67c8
-- 
2.41.0