-
Notifications
You must be signed in to change notification settings - Fork 7
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
Expose a variant of connect
that takes offset and length as argument
#6
Comments
can i work on this? do i write the code to this file https://github.com/reynir/mirage-block-partition/blob/main/lib/mirage_block_partition.ml ? or as a comment here ? |
Yes, you may work on this. It would be great if you could submit it as a pull request. You need to update lib/mirage_block_partition.ml but as well the interface file lib/mirage_block_partition.mli to expose the new function. |
@reynir , are there more issues coming for mirage-block-partition, or can everyone try this issue first? |
If you like you can work on this issue |
@AryanGodara I would look at the definitions of |
Got it! |
@reynir , I just updated the PR, kindly review and advise further. Thanks |
@reynir please check my new comment on my PR, to advise further. thanks |
@0x0god You are welcome to keep working on this. Note however that the Outreachy contribution period is over and further contributions will not be considered for you application. I will as well have less time to help and guide you. So expect to be more self-reliant if you want to continue working on this. |
Thanks,...Yes, I understand the outreachy contribution period is over :) I'm doing this personally to improve my OCaml skills as I develop a passion for OCaml and also to contribute more to the opensource community. :) |
The current
connect
interface has the nice property that it is difficult to create overlapping partitions. However, in the context of Mirage the Functoria API doesn't have a concept of pairs. It is not clear to me how or if the current interface can be integrated into the Mirage tool.Implement a variant of
connect
(could be namedconnect'
) that takes and offset and a length and returns a single partition.The text was updated successfully, but these errors were encountered: