Skip to content

Conversation

@dignifiedquire
Copy link
Contributor

breaking change, because this is reexported as part of the public interface

@SeverinAlexB
Copy link
Contributor

SeverinAlexB commented Sep 30, 2025

Hey @dignifiedquire

I looked over n0-computer/iroh#3476 again. How sure are you the dependency upgrades are really a breaking change?

Isn't this just a version conflict in iroh?

You are specifying ed25519-dalek in iroh-base and setting it to version = "2.1.1". With this PR mainline will use ed25519-dalek v3. Therefore, you have a version mismatch and you can't convert the Public Keys anymore.

Please correct me if I am missing something.

@dignifiedquire
Copy link
Contributor Author

The reason why this is a breaking change for the mainline crate iteself, is because of this line https://github.com/pubky/mainline/blob/main/src/lib.rs#L30 as this means the public type reexported is now a new one, incompatible with the old one.

@SeverinAlexB SeverinAlexB merged commit bfaaf81 into pubky:main Sep 30, 2025
1 check passed
@dignifiedquire
Copy link
Contributor Author

any chance I could get a release for this?

@SeverinAlexB SeverinAlexB mentioned this pull request Oct 3, 2025
@SeverinAlexB
Copy link
Contributor

github-merge-queue bot pushed a commit to n0-computer/iroh that referenced this pull request Oct 6, 2025
## Description

- Upgrade to rand@0.9
- Upgrade dalek crates to prereleases


Blocked on 

- [x] RustCrypto/nacl-compat#182
- [x] pubky/mainline#71
- [x] pubky/pkarr#186

Closes #3198 

## Breaking Changes

- interfaces that depend on `rand` now need `0.9`

## Notes & open questions

<!-- Any notes, remarks or open questions you have to make about the PR.
-->
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.

2 participants