opam-repository
opam-repository copied to clipboard
ocaml
post-messages for OCaml pre-release packages
This PR seeks both to thank users for trying out OCaml pre-releases and also to advertise more loudly that alpha releases may require the addition of a testing repository.
The intention is that the message about the alpha repository would disappear either with the beta or rc release. At the actual release, the message for the trunk package would change to reflect the fact it's no longer a pre-release.
It is possible to include a package in opam-alpha-repository to customise the message based on whether the repository has been enabled or not (using exactly the same mechanism as we do for ocaml-beta). This adds a small bit of complexity (one extra package appears in the switch) but it does have the benefit that we could warn users to remove opam-alpha-repository when they test beta and rc releases, etc. We could even go further and prevent the use of opam-alpha-repository with a released compiler (if we wanted).
This PR also proposes removing the parallelism failure message, at least from pre-releases, where that should be reported as a bug and arguably from all new releases, as it's a bug which should be reported and fixed.
Note that changing post-messages doesn't trigger a switch rebuild.
LGTM. One question before merging: are you sure the line breaks are taken into account with the simple double quotes " (instead of """)?
We need to move opam-alpha-repository (or update the URL in this one) also before merging (sorry, I should have said that in the PR!)
Yeah - the triple quoted version just allows you to use " marks unescaped in the text, it doesn't affect newlines.
This PR isn't relevant anymore since the release of 5.0.0. We could discuss it again for 5.1 but I don't think the opam-alpha-repository will be required the same way it was during the 5.0 alpha cycle.
Agreed - let's remember to bring this up when 5.1 is branched in April (it might be worth having a post-messages for the releases regardless, even if it doesn't need to advertise the alpha repo)