Handle relationship between boxo and go-car and go-merkledag #218
Description
Current
Done criteria
boxo/car is removed. boxo will use ipld/go-car.
vvv OLD vvv
Done Criteria
Decision is made on the relation to be had between ipfs/boxo and ipld/go-car and ipfs/go-merkledag. This could mean having a dependency on ipfs/boxo (e.g., ipfs/go-merkledag#100 ) or boxo taking a fork of the relevant functionality it needs and maintaining it.
Why Important
The #196 effort (and particularly #206) can't be completed without decisions here, and that work is blocking other refactoring work like #176.
Resulting Actions
Below are the actions that are being taken as a result of the 2023-03-21 meeting:
✅ Item 1: Get go-car decoupled:
- @Jorropo (with @rvagg review) revert deprecation to ipfs/go-block-format that was done in refactor: deprecate and add stub types to go-libipfs/blocks go-block-format#37
- @Jorropo release ipfs/go-block-format
- @Jorropo (with @rvagg review) replace go-libipfs/blocks with ipfs/go-block-format in ipld/go-car. (Includes removing dependency in ipld/go-car on ipfs/go-libipfs)
- @Jorropo replace go-libipfs/blocks with ipfs/go-block-format in ipfs/go-unixfs
- @Jorropo remove dependency in ipfs/go-unixfs on ipfs/go-libipfs: remove go-libipfs dependency go-unixfs#146
✅ Item 2: copy in go-merkledag to boxo
This will get handled as part of #202
A row will be added for go-merkledag.
No deprecated types will be added.
A CODOWNERS will be added.
ipfs/go-merkledag#101
Additional items that have come up since>
- ✅ Surface this with maintainers (folks in PL EngRes #bedrock): https://filecoinproject.slack.com/archives/C03FK0JJ10E/p1680108077842429
- Clarify that boxo/ipld/car is not intended as a replacement for ipld/go-car: Clarify ipld/car relatioship with ipld/go-car #244