From: Patrick Talbert ptalbert@redhat.com
redhat: fix the check for the n option
The current logic is testing for the letter n anywhere in the MAKEFLAGS. Instead we should just check the firstword as described in the make docs section 7.3.
Signed-off-by: Patrick Talbert ptalbert@redhat.com
diff --git a/redhat/Makefile b/redhat/Makefile index blahblah..blahblah 100644 --- a/redhat/Makefile +++ b/redhat/Makefile @@ -13,7 +13,7 @@ SPECRELEASED_KERNEL=$(RELEASED_KERNEL) SPECINCLUDE_FEDORA_FILES=$(INCLUDE_FEDORA_FILES) SPECINCLUDE_RHEL_FILES=$(INCLUDE_RHEL_FILES)
-ifneq (,$(findstring n,$(MAKEFLAGS))) +ifneq (,$(findstring n,$(firstword -$(MAKEFLAGS)))) # Do not set RHTEST on the command line. Use the make command built-in options # -n, --just-print, --dry-run, --recon on the command line. RHTEST=1
-- https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2395
From: Patrick Talbert on gitlab.com https://gitlab.com/cki-project/kernel-ark/-/merge_requests/2395#note_1339459...
That seems to be it. Thanks!
kernel@lists.fedoraproject.org