Skip to content

Conversation

Sjors
Copy link
Contributor

@Sjors Sjors commented Aug 21, 2025

bitcoin/bitcoin#31802 builds libmultiprocess and enables IPC by default, but my understanding is that this breaks oss-fuzz. One proposed solution is #13018.

This PR builds depends with NO_IPC=1, which disables the feature, for now while we work on an actual fix.

Copy link

Sjors is a new contributor to projects/bitcoin-core. The PR must be approved by known contributors before it can be merged. The past contributors are: fanquake, hebasto, maflcko, marcofleon, guidovranken

@Sjors
Copy link
Contributor Author

Sjors commented Aug 21, 2025

Marked draft pending feedback for other Bitcoin Core contributors.

@Sjors
Copy link
Contributor Author

Sjors commented Aug 21, 2025

Switched to using NO_IPC=1 in the depends build step.

Copy link
Contributor

@maflcko maflcko left a comment

Choose a reason for hiding this comment

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

lgtm. Makes sense to temporarily disable to get a green build again. Another workaround or fix can be done in the future.

@Sjors Sjors marked this pull request as ready for review August 21, 2025 11:23
Copy link
Contributor

@maflcko maflcko left a comment

Choose a reason for hiding this comment

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

lgtm

This is fine to merge after a passing CI

@DavidKorczynski DavidKorczynski merged commit 6f8f24b into google:master Aug 21, 2025
15 checks passed
@Sjors Sjors deleted the 2025/08/ipc branch August 22, 2025 06:18
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.

3 participants