Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

msvc: Compile test\fuzz\bitdeque.cpp #29983

Merged
merged 2 commits into from May 1, 2024

Conversation

hebasto
Copy link
Member

@hebasto hebasto commented Apr 28, 2024

This PR resolves one point from the #29774 (comment):

What is the issue with the bitdeque... ?

@DrahtBot
Copy link
Contributor

DrahtBot commented Apr 28, 2024

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Code Coverage

For detailed information about the code coverage, see the test coverage report.

Reviews

See the guideline for information on the review process.

Type Reviewers
ACK maflcko, dergoegge, sipa, achow101

If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

@hebasto hebasto changed the title build, msvc: Compile test\fuzz\bitdeque.cpp msvc: Compile test\fuzz\bitdeque.cpp Apr 28, 2024
@hebasto
Copy link
Member Author

hebasto commented Apr 28, 2024

cc @sipa

@maflcko
Copy link
Member

maflcko commented Apr 29, 2024

lgtm ACK 774359b

Copy link
Member

@dergoegge dergoegge left a comment

Choose a reason for hiding this comment

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

utACK 774359b

@sipa
Copy link
Member

sipa commented Apr 30, 2024

utACK 774359b

@achow101
Copy link
Member

achow101 commented May 1, 2024

ACK 774359b

Tested on windows, the bitdeque fuzz target is indeed now available.

@achow101 achow101 merged commit 4df2d0c into bitcoin:master May 1, 2024
16 checks passed
@hebasto hebasto deleted the 240428-msvc-fuzz-bitdeque branch May 1, 2024 06:18
@hebasto
Copy link
Member Author

hebasto commented May 1, 2024

Ported to the CMake-based build system in hebasto#182.

hebasto added a commit to hebasto/bitcoin that referenced this pull request May 3, 2024
fc91bfe fixup! cmake: Add fuzzing options (Hennadii Stepanov)
5927338 fixup! cmake: Add platform-specific definitions and properties (Hennadii Stepanov)
dcf2e66 refactor: Make 64-bit shift explicit (Hennadii Stepanov)
a7edea8 refactor: Fix "error C2248: cannot access private member" on MSVC (Hennadii Stepanov)

Pull request description:

  This PR ports bitcoin#29774 after the recent sync/rebase [PR](#179).

  Also included changes from bitcoin#29983.

Top commit has no ACKs.

Tree-SHA512: 562919ec9b89401d193c6d59a8634f5086a1a063b6ee075c465c5dbc9bf59da4bb1eb7b826deac91f470e22fe9bf049b43b7106bdc66c5d26c715d68a1008f4d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants