-
Notifications
You must be signed in to change notification settings - Fork 373
2025 Developer Meetings
Next meeting: 23/06/2025 13:00 CET
Agenda:
- look at new issues/PRs in:
- opam
- opam-file-format
- github.com/ocaml-opam/*
- discuss plans for a public meeting
- discuss how to handle the
next
/3.0
branch - triage old issues/PRs in the above listed repositories
Present: David (@dra27), Kate (@kit-ty-kate), Raja (@rjbou)
The issues up to #6548 and PRs up to #6552 didn't require further discussions
- #6541 we don't need more than 8 or 16 anyway since ocaml will not compile any faster
TODO:
- Debug GHA: kate
- review #6544: raja
- review #6520: david
- review #6549: kate
- make sure the test in #6539 matches the actual behaviour of opam-repo-ci: raja
- once done, review it: kate
- review #6550 after the other PRs are merge and this one is rebased: kate
- release: kate
Present: Artemiy (@arozovyk), David (@dra27), Kate (@kit-ty-kate), Raja (@rjbou)
The issues up to #6536 and PRs up to #6531 didn't require further discussions
- #6535: we need to flip the new switch state field from the overwritten opams to the simulated opams
- #6528, #6520, etc...: sum up of the current statuses of the different experiments. Solution chosen: We merge #6520 in 2.4 and work on a better solution in 2.5.
- ocurrent/opam-repo-ci#441: looked at the issue. It could be an issue with either #6436 or the recent changes to pinned packages. Kate was unable to reproduce locally but we'll try again.
In order to have the intuitive workflow for users we decided to change the whole behaviour as described in #6520 (no self-conflict) and then take time to do a more fine grained specification for 2.5 (remove only if it is accurate the removal of installed package).
We're hoping to start the release by the end of the week. TODO:
- Rebase and merge #6520: Raja
- Open a PR to enable the embedded cygwin setup on Windows in the release script: Kate
- Debug ocurrent/opam-repo-ci#441: Raja
- Test memory usage of large updates (e.g. from the before-phase1 tag): Kate
Present: Artemiy (@arozovyk), David (@dra27), Kate (@kit-ty-kate), Raja (@rjbou)
The issues up to #6521 and PRs up to #6523 didn't require further discussions
-
#6513: the fix is in
patch
, it has been released and PR will follow to updte external sources patch - #6504: Kate will finish the PR, then a review
- #6501: deps-only behaviour, we need to test the new behaviour and add some tests
Present: Artemiy (@arozovyk), David (@dra27), Kate (@kit-ty-kate), Raja (@rjbou)
The issues up to #6521 and PRs up to #6523 didn't require further discussions
We discussed needed PRs for the release, the current plan is : beta1 02/06, rc 09/06, final 16/06
Present: Artemiy (@arozovyk), David (@dra27), Raja (@rjbou)
The issues up to #6514 and PRs up to #6516 didn't require further discussions
Release PRs & issues:
- #6501 : dps-onyl issue, Raja reproduced, she will look at it
-
#6513 : stack overflow, guess that it is an issue on
patch
- #6498 : windows / setup.exe, Raja will give it a try
The current plan is to have the final on 10/06
Present: Artemiy (@arozovyk), David (@dra27), Raja (@rjbou)
The issues up to #6514 and PRs up to #6515 didn't require further discussions
- #6514 (source cache shouldn't be indexed by the checksum only) : it is definitively something todo, see comment
Present: Artemiy (@arozovyk), Kate (@kit-ty-kate)
Discussed implementation of #6461 Move the depexts availability calculation at opam update time
Present: Artemiy (@arozovyk), David (@dra27), Kate (@kit-ty-kate), Raja (@rjbou)
The issues up to #6491 and PRs up to #6490 didn't require further discussions
There is a view to try to enforce MSYS2 support for 2.5 release.
Instead of having setup.exe in cache in opam.ocaml.org, we could add setup.exe embed in Windows opam binary. It adds only 2M and resolve all network issue related to that (plus Cygwin website seems to be no more reachable in some countries). The licensing seems good (GPL).
Related issue: #6497
Currently, ocaml-system
is the compiler installed by default is there is an
ocaml binary on the system. This is very useful to have a quickly set switch,
but with the time there was several issue encountered:
- some system have the compiler installed but not the
compiler-libs
- some system installs several version of some libraries confusing opam install actions
Several proposition were discussed:
- check if the install is consistent and the one needed by opam: it would result into too much work to keep synchronised over several distributions.
- remove completely
ocaml-system
from repository, but it is not what is wanted: it would break too much process, especially CIs - remove compiler flag from
ocaml-system
: it would break opam 2.0 users, where invariant was not yet introduced -
#6465 (use
default-compiler
flag to select invariant): used with other repositories, it would need to keepdefault-compiler
up-to-date with newly added compilers -
#6464 (remove
eval-variables
from invariant computation): it would remove the ability to useeval-variables
for other repositories to compute invariant
We tend for a solution like #6464
with an option in the global config to re-enable eval-variables
in invariant
computation. But this won't resolve having ocaml-system
selected when install
local switchs, or install a switch with the compiler ocaml
(which ends up as
an invariant).
We finally agreed on a solution of having an explicit
flag for ocaml-system
that would make it installable only if requested explicitly, like that it is
won't be installed by default on switches with an invariant, and won't be
included on computed invariants. It is possible with mccs, we need to check if
it is possible with 0install.
The general behaviour of explicit
flag would be: the package won't be
installed unless explicitly required via CLI or in the invariant, not if it is
in package dependency.
There is no (simple) way to keep a complete working history, and it is not
required for other plugins. The solution we would go for is to just keep the
history and make plugins changes on top of that, like that we have at least a
working git blame
. Everything is ready for the split, we pause that until we
see more with opam admin
users.
Present: David (@dra27), Kate (@kit-ty-kate), Raja (@rjbou), Ryan (@RyanGibb)
The issues up to #6374 and PRs up to #6377 didn't require further discussions
Related PR: #5982
The current implementation generates a Nix derivation at each install of a
depext that contains all needed depexts by the switch: new ones (opam packages
to install) and old ones (already installed opam packages). This leads to a
rework of the API for handling the current switch state. A proposal for discussion
was done to keep persistent the Nix derivation file in the switch hierarchy and
only update it when a new package is added; but it needs a removal mechanism to
remove already stored Nix packages from that internal install (there is no
simple way to update that internal install, it would be an update of the file).
It is a very delicate issue to remove depexts in other distribution (user
managed system), but for internal ones (Nix, Cygwin) is doable but have its pro
and cons: some think it is preferable as the user has no way to interact with
the internal install, some think that it is as delicate a non internal install
as user may change it. It is
possible to add this functionality at opam remove
as it is done now at opam install
for system packages install. After some discussions and projections,
we end up by keeping the way it is done currently in the PR (propagate the
information of all needed depext at current state), it may be reused by other
futur depext support, and it won't add the removal complexity (handling
multiswitch system dependencies removal with its own command).
Another review of the PR will be done with that in mind.
No meeting
Present: David (@dra27), Kate (@kit-ty-kate), Raja (@rjbou)
The issues up to #6366 and PRs up to #6367 didn't require further discussions
- Alpha release of ocaml-patch soon, to integrate to opam for 2.4 if released for opam 2.4 release process
- Some exploratory work for reftest splitting, not so trivial for dune
- On
opam admin
split, we'll try to keep git history, and need to manage how to move issues (different orgs) & PRs (reopen?)
No meeting
Present: David (@dra27), Kate (@kit-ty-kate)
The issues up to #6357 and PRs up to #6343 didn't require further discussions