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

chore(deps): Update dependencies #1821

Merged
merged 7 commits into from
Nov 25, 2023
Merged

chore(deps): Update dependencies #1821

merged 7 commits into from
Nov 25, 2023

Conversation

flub
Copy link
Contributor

@flub flub commented Nov 19, 2023

Description

This updates dependencies of iroh 0.4.1 release, most notably quinn fixes platform support for android without patches.

Notes & open questions

  • Not all dependencies are updated. Only a few select ones which make things better
    for deltachat.

  • Note that the base of this PR is maint-0.4 which is a branch currently pointing
    at the 0.4.1 commit.

  • This PR also bumps the version number already to 0.4.2, we'd have to publish a new
    release with this.

  • This ignores a rust advisory, the way iroh uses the code was determined not to be
    problematic and adopting the new API is rather invasive and not suitable for a
    maintenance branch. The main branch has updated this code.

  • Netsim CI is disabled, provides not much value for a maintenance branch and it
    has been updated for newer versions of iroh. Keeping it compatible with this
    old version is not worth it.

  • Only stable rust is tested in CI now. The proc-macro2 crate version used in
    Cargo.lock does not work with current nightly, which is fine.

  • This disables the CI check on the Windows/GNU build. There is a linker problem
    picking up an MSVC library. Rather than debug this I think we care much more
    about MSVC.

Change checklist

  • Self-review.
  • Documentation updates if relevant.
  • Tests if relevant.

This updates dependencies of iroh, most notably quinn fixes platform
support for android without patches.
@flub flub requested a review from dignifiedquire November 19, 2023 19:10
This is also used by deltachat currently.
@dignifiedquire
Copy link
Contributor

ci is very sad, what's the plan there?

@flub
Copy link
Contributor Author

flub commented Nov 20, 2023

to fix it...

flub added 5 commits November 20, 2023 13:01
This is a bit much for a maintenance branch, and current nightly does
not agree with the proc-macro2 version used in the Cargo.lock file.
We don't need this on a maintenance branch and it's been adjusted for
current iroh, not an old version.
Not sure we really care, the linker picks up the wrong library which
is probably rather brittle.  This failure currently stops the msvc CI
task from succeeding which otherwise looks like it might work fine.
That one is much more important.
@flub
Copy link
Contributor Author

flub commented Nov 20, 2023

@dignifiedquire now CI is happy again. Admittedly I disabled some of it... Do you think it would be reasonable to publish 0.4.2?

@flub flub merged commit 9881b78 into maint-0.4 Nov 25, 2023
11 checks passed
@flub flub deleted the flub/maint-0.4-deps branch October 4, 2024 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants