Thomas Gazagnaire

Yea I think we should have to make sure GC-ed objects are not found in the LRU. As the layers are self-contained, I strongly suspect that only removing the commit...

@patricoferris thanks for your report! A possible workaround is to run your own Git server, and push directory to it from the browser. And then configure your server-side irmin store...

@CraigFe I am also wondering if that's not related to the issue you found with missing custom operators in https://github.com/talex5/irmin-indexeddb/pull/19

I am not sure to understand what you mean: ``` (modules (:standard \ config manifest)) ``` means that `dune` will take all top-level modules apart from `config.ml` and `manifest.ml`. Could...

But it should already added to the `modules` field. That's what `:standard` means: all the top-level modules. Does it fail to be added only for generated files?

So that seems to be a bug with the interaction between `:standard` and `enabled_if`?

> Should we make an issue on ocaml/dune? Yes, please! That definitely looks like a bug in `dune`. (having a small repro that doesn't involve the `mirage` tool would be...

> fix bootvar for all targets This was already broken in MirageOS 3 so it's not a regression. It'd be nice to have consistent semantics for all the targets but...

We also decided to postpone the rest of the device changes to the next (point) release.

Without #6137 ``` ❯ dune exec -- cohttp-lwt-unix/bin/cohttp_curl_lwt.exe --help ❯ codesign -v _build/default/cohttp-lwt-unix/bin/cohttp_curl_lwt.exe ❯ codesign -vv _build/default/cohttp-lwt-unix/bin/cohttp_curl_lwt.exe _build/default/cohttp-lwt-unix/bin/cohttp_curl_lwt.exe: valid on disk _build/default/cohttp-lwt-unix/bin/cohttp_curl_lwt.exe: satisfies its Designated Requirement ```