Skip to content

Remove extern crate hashbrown#1717

Merged
apoelstra merged 1 commit intorust-bitcoin:masterfrom
tcharding:03-21-rm-hashbrown
Mar 21, 2023
Merged

Remove extern crate hashbrown#1717
apoelstra merged 1 commit intorust-bitcoin:masterfrom
tcharding:03-21-rm-hashbrown

Conversation

@tcharding
Copy link
Copy Markdown
Member

(Merge candidate only after release of 0.30.0)

We no longer have a "hashbrown" feature, the feature gated pub extern crate hashbrown should have been removed when we removed the feature.

We no longer have a "hashbrown" feature, the feature gated `pub extern
crate hashbrown` should have been removed when we removed the feature.
@apoelstra
Copy link
Copy Markdown
Member

I don't think this needs to be gated on 0.30.0, AFIACT it has no user-visible effects and is very unlikely to cause merge conflicts with anything else.

Copy link
Copy Markdown
Member

@apoelstra apoelstra left a comment

Choose a reason for hiding this comment

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

ACK 2d23e11

Copy link
Copy Markdown
Collaborator

@Kixunil Kixunil left a comment

Choose a reason for hiding this comment

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

ACK 2d23e11

@Kixunil
Copy link
Copy Markdown
Collaborator

Kixunil commented Mar 21, 2023

Indeed, feel free to merge it into 0.30.

@Kixunil Kixunil added the trivial Obvious, easy and quick to review (few lines or doc-only...) label Mar 21, 2023
@tcharding
Copy link
Copy Markdown
Member Author

tcharding commented Mar 21, 2023

As for the other one, I don't want rebasing #1699 and waiting for timezones and acks to hold up the release.

@tcharding tcharding marked this pull request as ready for review March 21, 2023 21:15
@tcharding tcharding mentioned this pull request Mar 21, 2023
8 tasks
@apoelstra apoelstra merged commit 86142f1 into rust-bitcoin:master Mar 21, 2023
apoelstra added a commit that referenced this pull request Mar 22, 2023
ffee8ad Bump version to v0.30.0 (Tobin C. Harding)

Pull request description:

  Add changelog notes and bump the version number to v0.30.0.

  ## TODO - pre-merge

  - [x] Release `bitcoin_hashes` 0.12: #1694
  - [x] Release secp 0.27: rust-bitcoin/rust-secp256k1#588
    - rust-bitcoin/rust-secp256k1#590
  - [x] Update `secp256k1` dependency to use newly released v0.27: #1714
  - [x] Merge
    - ~#1696
    - #1695
    -  #1111
  - [x] If time permits merge these:
    - #1710
    - #1705
    - #1713
  - [x] Set the release date in changelog header
  - [x] And merge these:
    - #1721
    - #1720
    - #1719
    - #1717

  ## TODO  - post release
  - [ ] Release the blogpost: rust-bitcoin/www.rust-bitcoin.org#2
     - ~Set the date in the blog post to match the date 0.30 is released~

ACKs for top commit:
  sanket1729:
    reACK ffee8ad
  Kixunil:
    ACK ffee8ad
  apoelstra:
    ACK ffee8ad

Tree-SHA512: b0ea113ee1726fd9b263d0e01fe14bd544c007c05a9ac43b6c2d4edbeef3bb3ad456b061ef086626e1e1b27a0cda49cb6bc28aac3ad1691d72ffe00400ed5b45
@tcharding tcharding deleted the 03-21-rm-hashbrown branch March 22, 2023 02:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

trivial Obvious, easy and quick to review (few lines or doc-only...)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants