Skip to content

deps/refactor: port to iroh 0.91 #47

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

Merged
merged 26 commits into from
Aug 1, 2025
Merged

deps/refactor: port to iroh 0.91 #47

merged 26 commits into from
Aug 1, 2025

Conversation

Frando
Copy link
Member

@Frando Frando commented Jul 1, 2025

Description

Ports iroh-docs to iroh 0.91 and from quic-rpc to irpc.

TODOs / Missing features vs iroh-docs 0.35:

  • Doc::export_file and Doc::import_file are missing. Would need some more work to port them, I didn't get around to that yet. But it's not super critical, as you can import via iroh-blobs and then use Doc::set_hash.
  • Document all the API changes (this is quite a bit of work - best way likely to compare the docs for old and new version)
  • Maybe: Convert errors to snafu. This is still using anyhow and thiserror. It is quite some work, not sure about it yet.

Status for everything else

  • All APIs converted
  • All tests pass

Breaking Changes

Quite a few. TODO: Document.

Notes & open questions

Change checklist

  • Self-review.
  • Documentation updates following the style guide, if relevant.
  • Tests if relevant.
  • All breaking changes documented.

@n0bot n0bot bot added this to iroh Jul 1, 2025
@github-project-automation github-project-automation bot moved this to 🏗 In progress in iroh Jul 1, 2025
@Frando Frando changed the title wip: port to iroh 0.90 deps/refactor: port to iroh 0.90 Jul 2, 2025
@Frando Frando force-pushed the Frando/iroh-0.90 branch from 09ad4a4 to e2dd5b0 Compare July 2, 2025 09:41
Copy link

github-actions bot commented Jul 3, 2025

Documentation for this PR has been generated and is available at: https://n0-computer.github.io/iroh-docs/pr/47/docs/iroh_docs/

Last updated: 2025-08-01T08:39:38Z

src/engine.rs Outdated
})
})
}
// TODO(Frando): We can't port iroh-docs to 0.90 without something like this.
Copy link
Member

Choose a reason for hiding this comment

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

hey, does this TODO need to be addressed before merging?

Copy link
Member Author

Choose a reason for hiding this comment

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

I removed the comment. This is now just living elsewhere, as the needed iroh_blobs PR was merged.

ramfox pushed a commit to n0-computer/iroh-blobs that referenced this pull request Jul 31, 2025
## Description

This adds back a way to externally protect blobs from garbage
collection. It works like it did in blobs1: Users can provide an (async)
callback to which a `&mut HashSet<Hash>` is passed. The callback is
invoked before each gc run, and all hashes added to the set will be
protected from gc during this run.

Used in n0-computer/iroh-docs#47

## Breaking Changes

<!-- Optional, if there are any breaking changes document them,
including how to migrate older code. -->

## Notes & open questions

<!-- Any notes, remarks or open questions you have to make about the PR.
-->

## Change checklist

- [ ] Self-review.
- [ ] Documentation updates following the [style
guide](https://rust-lang.github.io/rfcs/1574-more-api-documentation-conventions.html#appendix-a-full-conventions-text),
if relevant.
- [ ] Tests if relevant.
- [ ] All breaking changes documented.
@Frando Frando changed the title deps/refactor: port to iroh 0.90 deps/refactor: port to iroh 0.91 Aug 1, 2025
@ramfox ramfox merged commit f3214d0 into main Aug 1, 2025
24 checks passed
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in iroh Aug 1, 2025
@ramfox ramfox deleted the Frando/iroh-0.90 branch August 1, 2025 13:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

2 participants