| Message ID | 20241022162500.121276-1-andrew.cooper3@citrix.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | tools/pvhsim: Inherit the tools debug= setting | expand |
diff --git a/tools/firmware/xen-dir/Makefile b/tools/firmware/xen-dir/Makefile index 6f5e208ab413..e19916f76722 100644 --- a/tools/firmware/xen-dir/Makefile +++ b/tools/firmware/xen-dir/Makefile @@ -1,4 +1,5 @@ XEN_ROOT = $(CURDIR)/../../.. +include $(XEN_ROOT)/tools/Rules.mk all: xen-shim @@ -75,6 +76,8 @@ $(D): linkfarm.stamp $(D)/xen/.config: $(D) $(MAKE) -C $(@D) KBUILD_DEFCONFIG=pvshim_defconfig defconfig + echo "CONFIG_DEBUG=$(if $(debug),y,n)" >> $@ + $(MAKE) -C $(@D) olddefconfig xen-shim: $(D)/xen/.config $(MAKE) -C $(<D) build
Right now, builds of pvshim are non-debug, because of pvshim_defconfig. However, this means that Gitlab CI testing is less effective than it wants to be. Include Rules.mk, and feed the tools-wide debug= setting. Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com> --- CC: Jan Beulich <JBeulich@suse.com> CC: Roger Pau Monné <roger.pau@citrix.com> CC: Anthony PERARD <anthony.perard@vates.tech> CC: Marek Marczykowski-Górecki <marmarek@invisiblethingslab.com> I don't particualrly like this, because while it ends up getting the appropriate choice of CONFIG_DEBUG, all other symbols with a default dependent on CONFIG_DEBUG get their non-debug form. --- tools/firmware/xen-dir/Makefile | 3 +++ 1 file changed, 3 insertions(+)