Skip to content

fix(ci): pin bzip2-sys to 0.1.12#1867

Merged
evanlinjin merged 2 commits intobitcoindevkit:masterfrom
oleonardolima:ci/pin-bzip2-sys-to-0.1.12
Mar 5, 2025
Merged

fix(ci): pin bzip2-sys to 0.1.12#1867
evanlinjin merged 2 commits intobitcoindevkit:masterfrom
oleonardolima:ci/pin-bzip2-sys-to-0.1.12

Conversation

@oleonardolima
Copy link
Collaborator

@oleonardolima oleonardolima commented Mar 3, 2025

Description

It looks like CI is failing due to the latest release of https://github.com/trifectatechfoundation/bzip2-rs/releases/tag/v0.5.2 which is a dependency of zip and therefore from electrsd.

It's failing on MSRV 1.63.0 step raising the following error: error[E0658]: use of unstable library feature 'core_ffi_c'.

This PR adds bzip2-sys to the pinned dependency list, pinning it to 0.1.12.

Notes to the reviewers

Changelog notice

  • Adds bzip2-sys to the pinned dependency list, pinning it to 0.1.12.

Checklists

All Submissions:

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

@oleonardolima
Copy link
Collaborator Author

I don't know why code coverage is failing though, taking a look.

@notmandatory notmandatory moved this to In Progress in BDK Chain Mar 4, 2025
@notmandatory notmandatory added this to the 1.2.0 milestone Mar 4, 2025
@ValuedMammal ValuedMammal force-pushed the ci/pin-bzip2-sys-to-0.1.12 branch from 0cd61be to cd7e358 Compare March 5, 2025 16:17
@ValuedMammal ValuedMammal marked this pull request as ready for review March 5, 2025 16:38
Copy link
Contributor

@LagginTimes LagginTimes left a comment

Choose a reason for hiding this comment

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

ACK 23626fe

@ValuedMammal ValuedMammal moved this from In Progress to Needs Review in BDK Chain Mar 5, 2025
@evanlinjin evanlinjin merged commit 362c3dc into bitcoindevkit:master Mar 5, 2025
22 of 23 checks passed
@github-project-automation github-project-automation bot moved this from Needs Review to Done in BDK Chain Mar 5, 2025
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.

5 participants