From: Patrick Talbert <ptalbert@xxxxxxxxxx> 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@xxxxxxxxxx> 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 _______________________________________________ kernel mailing list -- kernel@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to kernel-leave@xxxxxxxxxxxxxxxxxxxxxxx Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/kernel@xxxxxxxxxxxxxxxxxxxxxxx Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
