Skip to content

Commit aef788b

Browse files
committed
Merge rust-bitcoin/rust-bitcoin#690: BUG: Does not work with no_std under 1.29 (MSRV)
7854bd7 Fix `no_std` MSRV Fixes #690, #947 (mcroad) Pull request description: `rust-bitcoin` does not work with rust 1.29 under a `no_std` environment. This could be considered a bug. However, `no_std` support is a recent addition and this is likely not breaking anyone's builds. A decision needs to be made, either `no_std` MSRV is the current stable version while keeping the `std` MSRV as 1.29, or it needs to be fixed. This pr adds `no_std` to the 1.29 test suite. This came as I try to get rust-bitcoin/rust-miniscript#277 working and got stuck on the issue of testing `no_std` under 1.29. ACKs for top commit: Kixunil: ACK 7854bd7 tcharding: ACK 7854bd7 sanket1729: ACK 7854bd7 apoelstra: ACK 7854bd7 Tree-SHA512: 1614fb2193f760ed340592bdb94d076066f6f783bc1dc2b145d97f7151a28316e56b1975f1ad948460eb26db04e7e9382e60076686a681e46dcf33521fda5fca
2 parents 006b53e + a0f1886 commit aef788b

File tree

4 files changed

+9
-2
lines changed

4 files changed

+9
-2
lines changed

.github/workflows/rust.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,10 @@ jobs:
2828
- rust: 1.41.1
2929
env:
3030
AS_DEPENDENCY: true
31+
- rust: 1.47
32+
env:
33+
AS_DEPENDENCY: true
34+
DO_NO_STD: true
3135
steps:
3236
- name: Checkout Crate
3337
uses: actions/checkout@v2

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
target
22
Cargo.lock
3+
dep_test
34

45
#fuzz
56
fuzz/hfuzz_target

README.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,8 @@ please join us in
7777

7878
## Minimum Supported Rust Version (MSRV)
7979

80-
This library should always compile with any combination of features on **Rust 1.29**.
80+
This library should always compile with any combination of features (minus
81+
`no-std`) on **Rust 1.41.1** or **Rust 1.47** with `no-std`.
8182

8283
Because some dependencies have broken the build in minor/patch releases, to
8384
compile with 1.29.0 you will need to run the following version-pinning command:

src/util/bip32.rs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,8 @@ impl fmt::Debug for ExtendedPrivKey {
7474
.field("parent_fingerprint", &self.parent_fingerprint)
7575
.field("child_number", &self.child_number)
7676
.field("chain_code", &self.chain_code)
77-
.finish_non_exhaustive()
77+
.field("private_key", &"[SecretKey]")
78+
.finish()
7879
}
7980
}
8081

0 commit comments

Comments
 (0)