Skip to content

[backport] Fix no_std MSRV#986

Merged
apoelstra merged 2 commits intorust-bitcoin:rust-bitcoin-0.28from
apoelstra:2022-05--backport-690
May 4, 2022
Merged

[backport] Fix no_std MSRV#986
apoelstra merged 2 commits intorust-bitcoin:rust-bitcoin-0.28from
apoelstra:2022-05--backport-690

Conversation

@apoelstra
Copy link
Copy Markdown
Member

Backport of #690 to 0.28

@@ -1,5 +1,6 @@
target
Cargo.lock
dep_test
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I mean no reason to add this without adding the test crate, but of course also doesn't hurt anything, so....

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I don't get why you mention the test crate. This directory is created in the test.sh script. This chage, AFAICT, helps devs running the test script with AS_DEPENDENCY set to true.

@apoelstra
Copy link
Copy Markdown
Member Author

Oops, it looks like #985 beat me to the punch ... @tcharding should we close this one or yours? I believe they are identical except for the name of the branch that we're merging into.

Why is yours marked DO NOT MERGE?

@tcharding
Copy link
Copy Markdown
Member

Why is yours marked DO NOT MERGE?

I wrote that when the target branch was master before I realised I could target 0.28.x.

Copy link
Copy Markdown
Member

@tcharding tcharding left a comment

Choose a reason for hiding this comment

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

ACK e45f6c7

@apoelstra apoelstra merged commit 219aa59 into rust-bitcoin:rust-bitcoin-0.28 May 4, 2022
@apoelstra apoelstra deleted the 2022-05--backport-690 branch May 4, 2022 17:56
@apoelstra
Copy link
Copy Markdown
Member Author

Merged, tagged and published.

@devrandom
Copy link
Copy Markdown
Contributor

used in lightningdevkit/rust-lightning#1389

ChallengeDev210 pushed a commit to ChallengeDev210/rust-bitcoin that referenced this pull request Aug 1, 2022
e45f6c7 bump version to 0.28.1 (Andrew Poelstra)
eaaa3d0 [backport] Fix `no_std` MSRV (mcroad)

Pull request description:

  Backport of #690 to 0.28

ACKs for top commit:
  tcharding:
    ACK e45f6c7

Tree-SHA512: 5e1459b83044acb2a628e612a45f5be5b05067429fcc7b4c4c46713bd61ee43032bbe52a80a2fc9a3ebe7cb649c2554c6500e7f7d6ea3da35e97150ca6431fc1
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.

5 participants