Skip to content

Bump bip39 crate to v2.0.0#875

Merged
notmandatory merged 1 commit intobitcoindevkit:release/0.27from
tnull:2023-02-bump-bip39
Mar 26, 2023
Merged

Bump bip39 crate to v2.0.0#875
notmandatory merged 1 commit intobitcoindevkit:release/0.27from
tnull:2023-02-bump-bip39

Conversation

@tnull
Copy link
Contributor

@tnull tnull commented Feb 28, 2023 •
edited
Loading

Description

The updated version of rust-bip39 was just released. It also bumped the pinned version of unicode-normalization, which previously had lead to some incompatibilities.

Notes to the reviewers

Changelog notice

Checklists

All Submissions:

  • I've signed all my commits
  • I followed the contribution guidelines
  • I ran cargo fmt and cargo clippy before committing

@tnull
Copy link
Contributor Author

tnull commented Feb 28, 2023

Not sure if #561 is addressed with this for the time being.

@tnull
Copy link
Contributor Author

tnull commented Feb 28, 2023

CI is failing due to an unrelated MSRV violation: "error: package base64ct v1.6.0 cannot be built because it requires rustc 1.60 or newer, while the currently active rustc version is 1.57.0"

We had already seen this issue in LDK (cf. lightningdevkit/rust-lightning#2055), I now opened an issue towards electrsd: RCasatta/electrsd#56

@notmandatory notmandatory mentioned this pull request Feb 28, 2023
Closed
21 tasks
@notmandatory
Copy link
Member

notmandatory commented Mar 7, 2023

The CI issue if fixed in the release/0.27 branch, please rebase this PR to be merged to that branch. We're going to use release/0.27 for maintenance fixes while the new 1.0.0 stuff is being merged to master, starting with #793.

@tnull tnull force-pushed the 2023-02-bump-bip39 branch from d48a286 to a4647cf Compare March 7, 2023 18:56
@tnull
Copy link
Contributor Author

tnull commented Mar 7, 2023

Rebased on release/0.27.

@notmandatory notmandatory changed the base branch from master to release/0.27 March 8, 2023 04:00
rajarshimaitra
rajarshimaitra approved these changes Mar 15, 2023
View reviewed changes
Copy link
Contributor

@rajarshimaitra rajarshimaitra left a comment

Choose a reason for hiding this comment

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

tACK a4647cf

notmandatory
notmandatory approved these changes Mar 16, 2023
View reviewed changes
Copy link
Member

@notmandatory notmandatory left a comment

Choose a reason for hiding this comment

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

tACK a4647cf

Verified creating and restoring wallet from mnemonic manually with bdk-cli .

@notmandatory
Copy link
Member

notmandatory commented Mar 16, 2023

I also did a quick test with bdk-ffi using this PR branch version of bdk and works fine.

CLASSPATH=./tests/jna/jna-5.8.0.jar cargo test --features uniffi/bindgen-tests,uniffi/cli

@notmandatory notmandatory mentioned this pull request Mar 16, 2023
Closed
8 tasks
@thunderbiscuit thunderbiscuit mentioned this pull request Mar 24, 2023
Closed
@notmandatory notmandatory added this to the 0.27.3 milestone Mar 26, 2023
@notmandatory notmandatory merged commit 10608af into bitcoindevkit:release/0.27 Mar 26, 2023
@notmandatory
Copy link
Member

notmandatory commented Mar 26, 2023

Note, this was already bumped to 2.0.0 in the master branch so we don't need a corresponding PR there.

@notmandatory notmandatory mentioned this pull request Apr 11, 2023
Closed
11 tasks
@tnull tnull mentioned this pull request Jan 7, 2024
Merged
notmandatory added a commit that referenced this pull request Jan 7, 2024
@notmandatory
Merge #1259: Bump bip39 dependency to v2.0
1ff9d5c 
8694624 Bump `bip39` to v2.0 (Elias Rohrer)

Pull request description:

  We previously bumped the `bip39` version to 2.0 [in the 0.2X release branch](#875). Back then, bumping it in `master` was [assumed unnecessary](#875 (comment)). It seems this was erroneous, as the current `1.0.1` dependency is present since #793, which was merged before the bump in `release/0.27`.

  Here, we therefore bump the crate version in `master` after all.

ACKs for top commit:
  notmandatory:
    ACK 8694624

Tree-SHA512: a109219bc97bb8e965e8b10e72439aa898b710d1d1a154801ce499ad47475a6b23448d85e0de3f306f990573d1fccdae7d587ed41676a01f91d66a719782eae1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@notmandatory notmandatory notmandatory approved these changes

+1 more reviewer

@rajarshimaitra rajarshimaitra rajarshimaitra approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@tnull tnull

Labels

None yet

Projects

None yet

Milestone

0.28.0

Development

Successfully merging this pull request may close these issues.

3 participants

@tnull @notmandatory @rajarshimaitra