tarides / opam-monorepo

Assemble dune workspaces to build your project and its dependencies as a whole
ISC License
130 stars 27 forks source link

Populate `pin-depends` only with the packages that needs to be pinned in the switch #400

Open samoht opened 1 year ago

samoht commented 1 year ago

This is a tentative fix for https://github.com/mirage/mirage/issues/1476

I guess we could want to add the switch pins to that pin-depends field for reproducibility.

samoht commented 1 year ago

This is ready to review.

This is a breaking change:

Note that I am still a bit confused with the expected workflow to use when using x-opam-monorepo-provided. Maybe we do not need all of this, but unless I'm mistaken, I don't see how support for pinned packages in the switch is working today.