ACK: [Plucky/Unstable PATCH] UBUNTU: [Packaging] replace $(DROOT) with debian
You-Sheng Yang
vicamo.yang at canonical.com
Thu Oct 31 10:09:00 UTC 2024
Per offline conversation, there are already tons of appearances of
string "debian/", so it's meaningless to define a variable here.
Acked-by: You-Sheng Yang <vicamo.yang at canonical.com>
On Wed, Oct 30, 2024 at 12:19 PM Masahiro Yamada
<masahiro.yamada at canonical.com> wrote:
>
> If DROOT is not set to 'debian', our debian scripts do not work at all.
> This variable is therefore not useful.
>
> Signed-off-by: Masahiro Yamada <masahiro.yamada at canonical.com>
> ---
> debian/rules | 33 +++++++++++++++------------------
> debian/rules.d/1-maintainer.mk | 10 +++++-----
> debian/rules.d/2-binary-arch.mk | 20 ++++++++++----------
> debian/rules.d/4-checks.mk | 8 ++++----
> 4 files changed, 34 insertions(+), 37 deletions(-)
>
> diff --git a/debian/rules b/debian/rules
> index 48298523431c..76b8fe1ec964 100755
> --- a/debian/rules
> +++ b/debian/rules
> @@ -26,15 +26,12 @@ unexport LDFLAGS
> export LC_ALL=C
> export SHELL=/bin/bash -e
>
> -# Where do we find the common configuration.
> -export DROOT=debian
> -
> # Temporary workaround to fix build issues if only python2 runtime is installed
> # (without python3-dev).
> export PYTHON=python3
>
> # Common variables for all architectures
> -include $(DROOT)/rules.d/0-common-vars.mk
> +include debian/rules.d/0-common-vars.mk
>
> # Pull in some arch specific stuff
> -include $(DEBIAN)/rules.d/$(arch).mk
> @@ -43,7 +40,7 @@ include $(DROOT)/rules.d/0-common-vars.mk
> -include $(DEBIAN)/rules.d/hooks.mk
>
> # Maintainer targets
> -include $(DROOT)/rules.d/1-maintainer.mk
> +include debian/rules.d/1-maintainer.mk
>
> do_linux_tools=$(sort $(filter-out false,$(do_tools_usbip) $(do_tools_cpupower) $(do_tools_perf) $(do_tools_bpftool) $(do_tools_x86)))
> do_cloud_tools=$(sort $(filter-out false,$(do_tools_hyperv)))
> @@ -149,22 +146,22 @@ clean: debian/control debian/canonical-certs.pem debian/canonical-revoked-certs.
> [ -f $(DEBIAN)/reconstruct ] && bash $(DEBIAN)/reconstruct || true
>
> # Remove generated intermediate files
> - rm -f $(DROOT)/control.stub $(DEBIAN)/control.stub
> - rm -f $(DROOT)/scripts/fix-filenames
> + rm -f debian/control.stub $(DEBIAN)/control.stub
> + rm -f debian/scripts/fix-filenames
>
> .PHONY: distclean
> distclean: clean
> - rm -rf $(DROOT)/control debian/changelog \
> + rm -rf debian/control debian/changelog \
> debian/control debian/control.stub debian/copyright
>
> # Builds the image, arch headers and debug packages
> -include $(DROOT)/rules.d/2-binary-arch.mk
> +include debian/rules.d/2-binary-arch.mk
>
> # Builds the source and linux-headers indep packages
> -include $(DROOT)/rules.d/3-binary-indep.mk
> +include debian/rules.d/3-binary-indep.mk
>
> # Various checks to be performed on builds
> -include $(DROOT)/rules.d/4-checks.mk
> +include debian/rules.d/4-checks.mk
>
> control_files := $(DEBIAN)/control.stub.in
> ifeq ($(src_pkg_name),linux)
> @@ -182,10 +179,10 @@ UBUNTU_COMPATIBLE_SIGNING=$(shell grep -qx ' *Subject: C = GB, ST = Isle of Man,
> # Misc stuff
> .PHONY: $(DEBIAN)/control.stub
> $(DEBIAN)/control.stub: \
> - $(DROOT)/scripts/control-create \
> + debian/scripts/control-create \
> $(control_files) \
> debian/canonical-revoked-certs.pem \
> - $(DROOT)/control.d/flavour-module.stub \
> + debian/control.d/flavour-module.stub \
> $(DEBIAN)/changelog \
> $(wildcard $(DEBIAN)/control.d/*)
> for i in $(control_files); do \
> @@ -201,7 +198,7 @@ $(DEBIAN)/control.stub: \
> > $(DEBIAN)/control.stub;
> flavours="$(sort $(wildcard $(DEBIAN)/control.d/vars.*))";\
> for i in $$flavours; do \
> - $(SHELL) $(DROOT)/scripts/control-create $$i "$(any_signed)" | \
> + $(SHELL) debian/scripts/control-create $$i "$(any_signed)" | \
> sed -e 's/PKGVER/$(release)/g' \
> -e 's/ABINUM/$(abinum)/g' \
> -e 's/SRCPKGNAME/$(src_pkg_name)/g' \
> @@ -214,10 +211,10 @@ $(DEBIAN)/control.stub: \
> debian/control: $(DEBIAN)/control.stub
> cp $(DEBIAN)/control.stub debian/control
>
> -debian/canonical-certs.pem: $(wildcard $(DROOT)/certs/*-all.pem) $(wildcard $(DROOT)/certs/*-$(arch).pem) $(wildcard $(DEBIAN)/certs/*-all.pem) $(wildcard $(DEBIAN)/certs/*-$(arch).pem)
> +debian/canonical-certs.pem: $(wildcard debian/certs/*-all.pem) $(wildcard debian/certs/*-$(arch).pem) $(wildcard $(DEBIAN)/certs/*-all.pem) $(wildcard $(DEBIAN)/certs/*-$(arch).pem)
> for cert in $(sort $(notdir $^)); \
> do \
> - for dir in $(DEBIAN) $(DROOT); \
> + for dir in $(DEBIAN) debian; \
> do \
> if [ -f "$$dir/certs/$$cert" ]; then \
> cat "$$dir/certs/$$cert"; \
> @@ -226,10 +223,10 @@ debian/canonical-certs.pem: $(wildcard $(DROOT)/certs/*-all.pem) $(wildcard $(DR
> done; \
> done >"$@"
>
> -debian/canonical-revoked-certs.pem: $(wildcard $(DROOT)/revoked-certs/*-all.pem) $(wildcard $(DROOT)/revoked-certs/*-$(arch).pem) $(wildcard $(DEBIAN)/revoked-certs/*-all.pem) $(wildcard $(DEBIAN)/revoked-certs/*-$(arch).pem)
> +debian/canonical-revoked-certs.pem: $(wildcard debian/revoked-certs/*-all.pem) $(wildcard debian/revoked-certs/*-$(arch).pem) $(wildcard $(DEBIAN)/revoked-certs/*-all.pem) $(wildcard $(DEBIAN)/revoked-certs/*-$(arch).pem)
> for cert in $(sort $(notdir $^)); \
> do \
> - for dir in $(DEBIAN) $(DROOT); \
> + for dir in $(DEBIAN) debian; \
> do \
> if [ -f "$$dir/revoked-certs/$$cert" ]; then \
> cat "$$dir/revoked-certs/$$cert"; \
> diff --git a/debian/rules.d/1-maintainer.mk b/debian/rules.d/1-maintainer.mk
> index db2d7b99a312..f26a93fe1113 100644
> --- a/debian/rules.d/1-maintainer.mk
> +++ b/debian/rules.d/1-maintainer.mk
> @@ -29,7 +29,7 @@ configs-targets := updateconfigs defaultconfigs genconfigs editconfigs
> $(configs-targets):
> dh_testdir
> kmake='$(kmake)' skip_checks=$(do_skip_checks) conc_level=$(conc_level) \
> - $(SHELL) $(DROOT)/scripts/misc/kernelconfig $@
> + $(SHELL) debian/scripts/misc/kernelconfig $@
>
> .PHONY: printenv
> printenv:
> @@ -102,11 +102,11 @@ printchanges:
> gawk '/UBUNTU:\s*Ubuntu-.*$$/ { print $$1; exit }'); \
> fi; \
> git log "$$baseCommit"..HEAD | \
> - $(DROOT)/scripts/misc/git-ubuntu-log
> + debian/scripts/misc/git-ubuntu-log
>
> .PHONY: insertchanges
> insertchanges: autoreconstruct finalchecks
> - $(DROOT)/scripts/misc/insert-changes $(DROOT) $(DEBIAN)
> + debian/scripts/misc/insert-changes debian $(DEBIAN)
>
> .PHONY: autoreconstruct
> autoreconstruct:
> @@ -115,12 +115,12 @@ autoreconstruct:
> if grep -q "^EXTRAVERSION = -rc[0-9]\+$$" Makefile; then \
> echo "exit 0" >$(DEBIAN)/reconstruct; \
> else \
> - $(DROOT)/scripts/misc/gen-auto-reconstruct $(upstream_tag) $(DEBIAN)/reconstruct $(DROOT)/source/options; \
> + debian/scripts/misc/gen-auto-reconstruct $(upstream_tag) $(DEBIAN)/reconstruct debian/source/options; \
> fi
>
> .PHONY: finalchecks
> finalchecks: debian/control
> - $(DROOT)/scripts/checks/final-checks "$(DEBIAN)" "$(prev_fullver)" $(do_skip_checks)
> + debian/scripts/checks/final-checks "$(DEBIAN)" "$(prev_fullver)" $(do_skip_checks)
>
> .PHONY: compileselftests
> compileselftests:
> diff --git a/debian/rules.d/2-binary-arch.mk b/debian/rules.d/2-binary-arch.mk
> index 94c9cd5c6b89..6ba3904c0980 100644
> --- a/debian/rules.d/2-binary-arch.mk
> +++ b/debian/rules.d/2-binary-arch.mk
> @@ -54,14 +54,14 @@ define build_dkms_sign =
> )
> endef
> define build_dkms =
> - rc=0; unset MAKEFLAGS; ARCH=$(build_arch) CROSS_COMPILE=$(CROSS_COMPILE) $(SHELL) $(DROOT)/scripts/dkms-build $(dkms_dir) $(abi_release)-$* '$(call build_dkms_sign,$(builddir)/build-$*)' $(1) $(2) $(3) $(4) $(5) || rc=$$?; if [ "$$rc" = "9" -o "$$rc" = "77" ]; then echo do_$(4)_$*=false >> $(builddir)/skipped-dkms.mk; rc=0; fi; if [ "$$rc" != "0" ]; then exit $$rc; fi
> + rc=0; unset MAKEFLAGS; ARCH=$(build_arch) CROSS_COMPILE=$(CROSS_COMPILE) $(SHELL) debian/scripts/dkms-build $(dkms_dir) $(abi_release)-$* '$(call build_dkms_sign,$(builddir)/build-$*)' $(1) $(2) $(3) $(4) $(5) || rc=$$?; if [ "$$rc" = "9" -o "$$rc" = "77" ]; then echo do_$(4)_$*=false >> $(builddir)/skipped-dkms.mk; rc=0; fi; if [ "$$rc" != "0" ]; then exit $$rc; fi
> endef
>
> define install_control =
> for which in $(3); \
> do \
> - template="$(DROOT)/templates/$(2).$$which.in"; \
> - script="$(DROOT)/$(1).$$which"; \
> + template="debian/templates/$(2).$$which.in"; \
> + script="debian/$(1).$$which"; \
> sed -e 's/@abiname@/$(abi_release)/g' \
> -e 's/@localversion@/-$*/g' \
> -e 's/@image-stem@/$(instfile)/g' \
> @@ -191,7 +191,7 @@ ifeq ($(do_extras_package),true)
> mkdir -p $(pkgdir_ex)/lib/modules/$(abi_release)-$*; \
> mv $(pkgdir)/lib/modules/$(abi_release)-$*/kernel \
> $(pkgdir_ex)/lib/modules/$(abi_release)-$*/kernel; \
> - $(SHELL) $(DROOT)/scripts/module-inclusion --master \
> + $(SHELL) debian/scripts/module-inclusion --master \
> $(pkgdir_ex)/lib/modules/$(abi_release)-$*/kernel \
> $(pkgdir)/lib/modules/$(abi_release)-$*/kernel \
> $(DEBIAN)/control.d/$(target_flavour).inclusion-list \
> @@ -221,7 +221,7 @@ endif
> $(pkgdir)/lib/modules/$(abi_release)-$*/initrd/; \
> fi
>
> - echo "interest linux-update-$(abi_release)-$*" >"$(DROOT)/$(bin_pkg_name)-$*.triggers"
> + echo "interest linux-update-$(abi_release)-$*" >"debian/$(bin_pkg_name)-$*.triggers"
> install -d $(pkgdir_bin)/usr/lib/linux/triggers
> $(call install_control,$(bin_pkg_name)-$*,image,postinst postrm preinst prerm)
> install -d $(pkgdir)/usr/lib/linux/triggers
> @@ -295,7 +295,7 @@ endif
> # Copy over scripts/module.lds for building external modules
> cp $(builddir)/build-$*/scripts/module.lds $(hdrdir)/scripts
> # Script to symlink everything up
> - $(SHELL) $(DROOT)/scripts/link-headers "$(hdrdir)" "$(indeppkg)" "$*"
> + $(SHELL) debian/scripts/link-headers "$(hdrdir)" "$(indeppkg)" "$*"
> # The build symlink
> install -d debian/$(basepkg)-$*/lib/modules/$(abi_release)-$*
> $(LN) /usr/src/$(basepkg)-$* \
> @@ -313,7 +313,7 @@ endif
> PREV_REVISION="$(prev_revision)" ABI_NUM="$(abinum)" \
> PREV_ABI_NUM="$(prev_abinum)" BUILD_DIR="$(builddir)/build-$*" \
> INSTALL_DIR="$(pkgdir)" SOURCE_DIR="$(CURDIR)" \
> - run-parts -v $(DROOT)/tests-build
> + run-parts -v debian/tests-build
>
> #
> # Remove files which are generated at installation by postinst,
> @@ -472,8 +472,8 @@ endif
> install -m644 $(abidir)/$*.fwinfo.builtin \
> $(pkgdir_bldinfo)/usr/lib/linux/$(abi_release)-$*/fwinfo.builtin; \
> fi
> - install -m644 $(DROOT)/canonical-certs.pem $(pkgdir_bldinfo)/usr/lib/linux/$(abi_release)-$*/canonical-certs.pem
> - install -m644 $(DROOT)/canonical-revoked-certs.pem $(pkgdir_bldinfo)/usr/lib/linux/$(abi_release)-$*/canonical-revoked-certs.pem
> + install -m644 debian/canonical-certs.pem $(pkgdir_bldinfo)/usr/lib/linux/$(abi_release)-$*/canonical-certs.pem
> + install -m644 debian/canonical-revoked-certs.pem $(pkgdir_bldinfo)/usr/lib/linux/$(abi_release)-$*/canonical-revoked-certs.pem
>
> # Get rid of .o and .cmd artifacts in headers
> find $(hdrdir) -name \*.o -or -name \*.cmd -exec rm -f {} \;
> @@ -484,7 +484,7 @@ endif
> install -d -m755 $(rustdir)
> mv $(hdrdir)/rust $(rustdir)
> # Generate symlink for Rust lib directory in headers
> - $(SHELL) $(DROOT)/scripts/link-lib-rust "$(hdrdir)" "$(indeppkg)" "$*"
> + $(SHELL) debian/scripts/link-lib-rust "$(hdrdir)" "$(indeppkg)" "$*"
>
> ifneq ($(do_full_build),false)
> # Clean out this flavours build directory.
> diff --git a/debian/rules.d/4-checks.mk b/debian/rules.d/4-checks.mk
> index 7cb84e9f71e5..9c840af3698e 100644
> --- a/debian/rules.d/4-checks.mk
> +++ b/debian/rules.d/4-checks.mk
> @@ -1,9 +1,9 @@
> # Check the signature of staging modules
> module-signature-check-%: $(stampdir)/stamp-install-%
> @echo Debug: $@
> - $(DROOT)/scripts/checks/module-signature-check "$*" \
> - "$(DROOT)/$(mods_pkg_name)-$*" \
> - "$(DROOT)/$(mods_extra_pkg_name)-$*" \
> + debian/scripts/checks/module-signature-check "$*" \
> + "debian/$(mods_pkg_name)-$*" \
> + "debian/$(mods_extra_pkg_name)-$*" \
> $(do_skip_checks)
>
> checks-%: module-signature-check-%
> @@ -13,6 +13,6 @@ checks-%: module-signature-check-%
> config-prepare-check-%: $(stampdir)/stamp-prepare-tree-%
> @echo Debug: $@
> ifneq ($(do_skip_checks),true)
> - python3 $(DROOT)/scripts/misc/annotations -f $(commonconfdir)/annotations \
> + python3 debian/scripts/misc/annotations -f $(commonconfdir)/annotations \
> --arch $(arch) --flavour $* --check $(builddir)/build-$*/.config
> endif
> --
> 2.43.0
>
>
> --
> kernel-team mailing list
> kernel-team at lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team
--
Regards,
You-Sheng Yang
More information about the kernel-team
mailing list