Skip to content

Conversation

@Alizter
Copy link
Collaborator

@Alizter Alizter commented Nov 17, 2025

This appears to have snuck in at some point #12623.

We track these libraries, along with re, spawn and uutf in both duneboot.ml and bootstrap_info.ml. They are the vendored libraries we use for bootstrap but not for building the other libraries.

@Sudha247
Copy link
Collaborator

Could you rebase, please? I think this is ready to merge.

This appeared to have snuck in at some point.

We track these libraries, along with re, spawn and uutf in both
duneboot.ml and bootstrap_info.ml. They are the vendored libraries we
use for bootstrap but not for building the other libraries.

Signed-off-by: Ali Caglayan <[email protected]>
@Alizter Alizter enabled auto-merge November 18, 2025 10:28
@Alizter Alizter merged commit 605f300 into ocaml:main Nov 18, 2025
26 checks passed
@Alizter Alizter deleted the push-zvwnnrmwvyus branch November 18, 2025 10:46
@rgrinberg
Copy link
Member

rgrinberg commented Nov 18, 2025

Isn't this commit empty?

@Alizter
Copy link
Collaborator Author

Alizter commented Nov 18, 2025

@rgrinberg Yes, that's my mistake. I rebased on top of #12719 which also accidentally fixed this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants