Skip to content

Conversation

@rklaehn
Copy link
Collaborator

@rklaehn rklaehn commented Dec 4, 2025

Description

While this will currently compile to wasm, when running in the browser, any reference to std::time will explode with unsupported platform errors, this fixes that.

Breaking Changes

Notes & open questions

Note: replaces #194 which is obsolete due to the change to n0-error

Change checklist

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

@rklaehn rklaehn changed the title Fix std time deps: Fix deps to actually work, not just compile, in wasm-browser Dec 4, 2025
@github-actions
Copy link

github-actions bot commented Dec 4, 2025

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

Last updated: 2025-12-04T09:52:12Z

@rklaehn rklaehn requested a review from b5 December 4, 2025 09:56
@rklaehn
Copy link
Collaborator Author

rklaehn commented Dec 4, 2025

@b5 I closed your PR and redid it because there were too many conflicts now that blobs is using n0-error. I hope I got all places where std::time::{Time, Duration, Instant} were used, but I don't really know how to check that this actually works...

@rklaehn rklaehn requested a review from Frando December 4, 2025 10:04
@n0bot n0bot bot added this to iroh Dec 4, 2025
@github-project-automation github-project-automation bot moved this to 🏗 In progress in iroh Dec 4, 2025
@rklaehn rklaehn merged commit 6ef70e3 into main Dec 4, 2025
25 of 27 checks passed
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in iroh Dec 4, 2025
@rklaehn rklaehn deleted the fix-std-time branch December 4, 2025 14:24
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