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

Add tar_kv_ro and tar_kv_rw #1389

Merged
merged 4 commits into from Feb 3, 2023
Merged

Add tar_kv_ro and tar_kv_rw #1389

merged 4 commits into from Feb 3, 2023

Conversation

reynir
Copy link
Member

@reynir reynir commented Feb 3, 2023

In addition, archive is deprecated in favor of tar_kv_ro and tar_kv_rw.

@hannesm
Copy link
Member

hannesm commented Feb 3, 2023

This looks nice in general, but the Tar_mirage.Make_KV_RW functor expects a PCLOCK and a BLOCK device, so we cannot uniformly handle it.

Also, the deprecated attribute shouldn't be hidden in the ocamldoc, but visible to the OCaml compiler so it can emit warnings :)

`Tar_mirage.Make_KV_RW` first appeared in 2.2.0.
@hannesm
Copy link
Member

hannesm commented Feb 3, 2023

looks good, please bow to dune build @fmt --auto

@hannesm hannesm merged commit 3e6ebeb into mirage:main Feb 3, 2023
@reynir reynir deleted the tar-kv-rw branch February 5, 2023 08:28
hannesm added a commit to hannesm/opam-repository that referenced this pull request Feb 23, 2023
… (4.3.4)

CHANGES:

#### Fixed

- Add -f <config> to Makefile output (mirage/mirage#1394 @hannesm)

#### Added

- Add tar_kv_ro and tar_kv_rw (mirage/mirage#1389 @reynir)

#### Changed

- Improved documentation of mirage.mli (mirage/mirage#1390 mirage/mirage#1391 @reynir)
- Deprecate Mirage.archive (use tar_kv_ro, mirage/mirage#1389 @reynir)
- Removed deprecated device git_happy_eyeballs (mirage/mirage#1388 @TaeminHa)
- Adapt bounds: mirage-crypto 0.11.0 (mirage/mirage#1392 @hannesm) git-mirage 2.13.0
  (mirage/mirage#1396 @hannesm) dns 7.0.0 tls 0.16.0 (mirage/mirage#1399 @hannesm)
  mirage-block-ccm (mirage/mirage#1400 @hannesm)
- mirage-runtime: conflict with ppxlib 0.29 (mirage/mirage#1397 @hannesm)
- add upper bound to dune 3.7 until a release of chamelon with fixes lands (see
  mirage/mirage#1401 (comment))
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.

None yet

2 participants