Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change to use Picos for scheduler interop #136

Closed
wants to merge 1 commit into from

Conversation

polytypic
Copy link
Contributor

@polytypic polytypic commented Oct 2, 2023

This PR changes Kcas to use Picos instead of DLA and DLT for scheduler interop.

⚠️ This must obviously not be merged before at least a first release of Picos has been published on opam, but this should be ready enough for preliminary review.

@polytypic polytypic force-pushed the change-to-use-picos branch 4 times, most recently from a1ee8e0 to f43c98d Compare October 2, 2023 17:34
@polytypic polytypic force-pushed the change-to-use-picos branch 5 times, most recently from fe7be98 to 62ed6e4 Compare October 24, 2023 12:18
@polytypic polytypic changed the title WIP: Change to use Picos for scheduler interop Change to use Picos for scheduler interop Oct 24, 2023
@polytypic polytypic marked this pull request as ready for review October 24, 2023 12:24
@@ -1 +1,4 @@
doc: "https://ocaml-multicore.github.io/kcas/doc/kcas/Kcas/"
pin-depends: [
[ "picos.dev" "git+https://github.com/ocaml-multicore/picos#2f2af8b2f170a38889211662fdc79bd160d82a69" ]
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This pin-depend must be removed before merging.

@@ -16,8 +16,7 @@ depends: [
"dune" {>= "3.8"}
"ocaml" {>= "4.13.0"}
"backoff" {>= "0.1.0"}
"domain-local-await" {>= "1.0.0"}
"domain-local-timeout" {>= "1.0.0"}
"picos"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should specify a version number.

@polytypic polytypic force-pushed the change-to-use-picos branch 2 times, most recently from f33bd10 to f3405ce Compare October 25, 2023 10:30
@polytypic polytypic force-pushed the change-to-use-picos branch 12 times, most recently from fc4542e to 82dd5b4 Compare December 9, 2023 21:15
@polytypic
Copy link
Contributor Author

This has been replaced by work in #204.

@polytypic polytypic closed this Sep 20, 2024
@polytypic polytypic deleted the change-to-use-picos branch September 20, 2024 11:23
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.

1 participant