Skip to content

Revert "update makefile to remove hsm_encryption"#8580

Merged
rustyrussell merged 1 commit into
ElementsProject:masterfrom
rustyrussell:fix-master-fuzzing-build
Oct 2, 2025
Merged

Revert "update makefile to remove hsm_encryption"#8580
rustyrussell merged 1 commit into
ElementsProject:masterfrom
rustyrussell:fix-master-fuzzing-build

Conversation

@rustyrussell
Copy link
Copy Markdown
Contributor

@rustyrussell rustyrussell commented Oct 2, 2025 •
edited
Loading

This reverts commit 084b033.

Fixes the build:

/usr/bin/ld: tests/fuzz/fuzz-hsm_encryption.o: in function `run':
/home/rusty/devel/cvs/lightning/tests/fuzz/fuzz-hsm_encryption.c:28:(.text.run[run]+0x375): undefined reference to `hsm_secret_encryption_key_with_exitcode'
/usr/bin/ld: /home/rusty/devel/cvs/lightning/tests/fuzz/fuzz-hsm_encryption.c:30:(.text.run[run]+0x3ee): undefined reference to `encrypt_hsm_secret'
/usr/bin/ld: /home/rusty/devel/cvs/lightning/tests/fuzz/fuzz-hsm_encryption.c:32:(.text.run[run]+0x459): undefined reference to `decrypt_hsm_secret'
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make: *** [Makefile:703: tests/fuzz/fuzz-hsm_encryption] Error 1

Changelog-None

@rustyrussell rustyrussell added this to the v25.12 milestone Oct 2, 2025
@rustyrussell rustyrussell merged commit 8895ca2 into ElementsProject:master Oct 2, 2025
1 check failed
@Chand-ra
Copy link
Copy Markdown

Chand-ra commented Oct 11, 2025

Thanks for this, a lot of my local branches were failing to compile due to this change. Any idea why it was merged in the first place?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

v25.12

Development

Successfully merging this pull request may close these issues.

2 participants

@rustyrussell @Chand-ra