Makefile: fix make update-versions.
What the developer wrote
Authored by Rusty Russell
Makefile: fix make update-versions.
1. We need to replace versions in __init__.py
2. We need to run uv after changing versions, so it updates uv.lock
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
What changed, and why it matters
This is a routine build-maintenance patch. It fixes the developer-only 'make update-versions' command so it also updates a Python package version string inside source files and runs the uv lock tool after version changes. There is no user-facing or security-relevant change.
No security action needed. Treat as normal build/maintenance commit.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The Makefile target update-versions now (1) updates version in each pyln subpackage’s init.py and (2) runs ‘uv lock’ after all version updates so the lockfile is refreshed. The diff is purely in the Makefile and only affects release tooling.
Changed components
Makefile release toolingupdate-versions targetupdate-pyln-version-% targetInspect captured patch +2 / −0
diff --git a/Makefile b/Makefile
index c1c027ae..7fa87ea4 100644
--- a/Makefile
+++ b/Makefile
@@ -758,12 +758,14 @@ clean: obsclean
PYLNS=client proto testing
update-versions: update-pyln-versions update-reckless-version update-dot-version update-doc-examples
@uv lock
+
update-pyln-versions: $(PYLNS:%=update-pyln-version-%)
update-pyln-version-%:
@if [ -z "$(NEW_VERSION)" ]; then echo "Set NEW_VERSION!" >&2; exit 1; fi
@echo "Updating contrib/pyln-$* to $(NEW_VERSION)"
@sed -i.bak 's/^version = .*/version = "$(NEW_VERSION)"/' contrib/pyln-$*/pyproject.toml && rm contrib/pyln-$*/pyproject.toml.bak
+ @sed -i.bak 's/^__version__ = .*/__version__ = "$(NEW_VERSION)"/' contrib/pyln-$*/pyln/$*/__init__.py && rm contrib/pyln-$*/pyln/$*/__init__.py.bak
pyln-release: $(PYLNS:%=pyln-release-%)
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.