[noble PATCH v2 1/3] UBUNTU: SAUCE: make $(stampdir)/stamp-build-perarch depend on build-arch
Masahiro Yamada
masahiro.yamada at canonical.com
Mon Dec 4 01:49:47 UTC 2023
Since commit ("UBUNTU: [Packaging] bpftool: always use vmlinux to
generate headers"), tools/bpf/bpftool/vmlinux must be created before
building the bpftool.
This is not guaranteed because the $(stampdir)/stamp-build-perarch
target does not depend on build-arch.
Running 'debian/rules binary' results in the following build error:
mv [...]/linux/debian/build/tools-perarch/tools/bpf/bpftool/vmlinux [...]/linux/debian/build/tools-perarch/vmlinux
mv: cannot stat '[...]/linux/debian/build/tools-perarch/tools/bpf/bpftool/vmlinux': No such file or directory
This potential issue has been hidden, relying on build drivers such as
dpkg-buildpackage running 'debian/rules build' and 'debian/rules binary'
as separate steps.
Without rootless builds, the (fake) root privilege is required for the
binary target, hence dpkg-buildpackage splits the build process into
three stages:
- fakeroot debian/rules clean
- debian/rules build
- fakeroot debian/rules binary
To support rootless builds, you need to specify all dependencies
correctly without such an assumption.
Once we transition to rootless builds, the 'binary' target will not
require the root privilege any more, so dpkg-buildpackage will merge
'build' and 'binary' into a single step. We need to assume only the
following two will be executed:
- debian/rules clean
- debian/rules binary
Signed-off-by: Masahiro Yamada <masahiro.yamada at canonical.com>
---
Changes in v2:
- new patch
debian/rules.d/2-binary-arch.mk | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/debian/rules.d/2-binary-arch.mk b/debian/rules.d/2-binary-arch.mk
index f89efd270e1c..b281428d3462 100644
--- a/debian/rules.d/2-binary-arch.mk
+++ b/debian/rules.d/2-binary-arch.mk
@@ -695,7 +695,7 @@ ifeq ($(do_any_tools),true)
endif
touch $@
-$(stampdir)/stamp-build-perarch: $(stampdir)/stamp-prepare-perarch install-arch-headers
+$(stampdir)/stamp-build-perarch: $(stampdir)/stamp-prepare-perarch install-arch-headers build-arch
@echo Debug: $@
ifeq ($(do_linux_tools),true)
ifeq ($(do_tools_usbip),true)
--
2.40.1
More information about the kernel-team
mailing list