Commit 56754f0f authored by Josh Poimboeuf's avatar Josh Poimboeuf
Browse files

objtool: Rename --Werror to --werror 



The objtool --Werror option name is stylistically inconsistent: halfway
between GCC's single-dash capitalized -Werror and objtool's double-dash
--lowercase convention, making it unnecessarily hard to remember.

Make the 'W' lower case (--werror) for consistency with objtool's other
options.

Acked-by: default avatarPetr Mladek <pmladek@suse.com>
Tested-by: default avatarJoe Lawrence <joe.lawrence@redhat.com>
Signed-off-by: default avatarJosh Poimboeuf <jpoimboe@kernel.org>
parent 48f1bbaf

scripts/Makefile.lib

+1 −1
Original line number Diff line number Diff line
@@ -191,7 +191,7 @@ objtool-args-$(CONFIG_HAVE_STATIC_CALL_INLINE) += --static-call 
objtool-args-$(CONFIG_HAVE_UACCESS_VALIDATION)		+= --uaccess

objtool-args-$(or $(CONFIG_GCOV_KERNEL),$(CONFIG_KCOV))	+= --no-unreachable

objtool-args-$(CONFIG_PREFIX_SYMBOLS)			+= --prefix=$(CONFIG_FUNCTION_PADDING_BYTES)

objtool-args-$(CONFIG_OBJTOOL_WERROR)			+= --Werror

objtool-args-$(CONFIG_OBJTOOL_WERROR)			+= --werror



objtool-args = $(objtool-args-y)					\

	$(if $(delay-objtool), --link)					\

scripts/Makefile.vmlinux_o

+1 −1
Original line number Diff line number Diff line
@@ -41,7 +41,7 @@ objtool-enabled := $(or $(delay-objtool),$(CONFIG_NOINSTR_VALIDATION)) 
ifeq ($(delay-objtool),y)

vmlinux-objtool-args-y					+= $(objtool-args-y)

else

vmlinux-objtool-args-$(CONFIG_OBJTOOL_WERROR)		+= --Werror

vmlinux-objtool-args-$(CONFIG_OBJTOOL_WERROR)		+= --werror

endif



vmlinux-objtool-args-$(CONFIG_NOINSTR_VALIDATION)	+= --noinstr \

tools/objtool/builtin-check.c

+1 −1
Original line number Diff line number Diff line
@@ -101,7 +101,7 @@ static const struct option check_options[] = { 
	OPT_BOOLEAN(0,   "sec-address", &opts.sec_address, "print section addresses in warnings"),

	OPT_BOOLEAN(0,   "stats", &opts.stats, "print statistics"),

	OPT_BOOLEAN('v', "verbose", &opts.verbose, "verbose warnings"),

	OPT_BOOLEAN(0,   "Werror", &opts.werror, "return error on warnings"),

	OPT_BOOLEAN(0,   "werror", &opts.werror, "return error on warnings"),



	OPT_END(),

};