Skip to content

make: skip extract-bolt-csv if bolts are absent - #9393

Merged
daywalker90 merged 1 commit into
ElementsProject:masterfrom
daywalker90:skip-check-wire-format
Aug 11, 2026
Merged

make: skip extract-bolt-csv if bolts are absent#9393
daywalker90 merged 1 commit into
ElementsProject:masterfrom
daywalker90:skip-check-wire-format

Conversation

@daywalker90

Copy link
Copy Markdown
Collaborator

check-source -> check-wire-format unconditionally ran
extract-bolt-csv, which failed when ../bolts was not checked out:

/bin/sh: 1: .tmp.lightningrfc/tools/extract-formats.py: not found
make: *** [wire/Makefile:54: wire/peer_wire.csv.raw] Error 127

Skip it via bolt-precheck when .tmp.lightningrfc is missing,
matching the bolt-check pattern.

@daywalker90
daywalker90 force-pushed the skip-check-wire-format branch from 2e9d5d7 to 825c5d7 Compare August 7, 2026 14:44
@daywalker90 daywalker90 changed the title make: skip check-wire-format if bolts are absent make: skip extract-bolt-csv if bolts are absent Aug 7, 2026
`check-source` -> `check-wire-format` unconditionally ran
`extract-bolt-csv`, which failed when `../bolts` was not checked out:

```
/bin/sh: 1: .tmp.lightningrfc/tools/extract-formats.py: not found
make: *** [wire/Makefile:54: wire/peer_wire.csv.raw] Error 127
```

Skip it via `bolt-precheck` when `.tmp.lightningrfc` is missing,
matching the `bolt-check` pattern.

Changelog-None
@daywalker90
daywalker90 force-pushed the skip-check-wire-format branch from 825c5d7 to ba2a0cb Compare August 7, 2026 14:45
@daywalker90 daywalker90 added the Status::Ready for Review The work has been completed and is now awaiting evaluation or approval. label Aug 7, 2026

@Andezion Andezion left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Nice!

@daywalker90
daywalker90 merged commit 2c90d93 into ElementsProject:master Aug 11, 2026
42 of 46 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Status::Ready for Review The work has been completed and is now awaiting evaluation or approval.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants