Messages in this thread | ![/](/images/icornerl.gif) | | Date | Wed, 22 Mar 2023 19:14:58 +0100 | Subject | Re: [PATCH v1 1/1] scripts: merge_config: Fix typo in variable name. | From | Mirsad Goran Todorovac <> |
| |
On 22. 03. 2023. 16:20, Randy Dunlap wrote: > > > On 3/22/23 01:51, Mirsad Goran Todorovac wrote: >> ${WARNOVERRIDE} was misspelled as ${WARNOVVERIDE}, which caused a shell >> syntax error in certain paths of the script execution. >> > > Fixes: 46dff8d7e381e ("scripts: merge_config: Add option to suppress warning on overrides") > > Acked-by: Randy Dunlap <rdunlap@infradead.org> > > Thanks.
Hi, Randy,
I think this time thanks go to the Heavens, for preserving my physical integrity.
Though I don't know of a bike with ABS.
>> Cc: Mark Brown <broonie@kernel.org> >> Cc: Masahiro Yamada <masahiroy@kernel.org> >> Cc: linux-kbuild@vger.kernel.org >> Cc: linux-kernel@vger.kernel.org >> Signed-of-by: Mirsad Goran Todorovac <mirsad.todorovac@alu.unizg.hr> >> --- >> scripts/kconfig/merge_config.sh | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/scripts/kconfig/merge_config.sh b/scripts/kconfig/merge_config.sh >> index 32620de473ad..902eb429b9db 100755 >> --- a/scripts/kconfig/merge_config.sh >> +++ b/scripts/kconfig/merge_config.sh >> @@ -145,7 +145,7 @@ for ORIG_MERGE_FILE in $MERGE_LIST ; do >> NEW_VAL=$(grep -w $CFG $MERGE_FILE) >> BUILTIN_FLAG=false >> if [ "$BUILTIN" = "true" ] && [ "${NEW_VAL#CONFIG_*=}" = "m" ] && [ "${PREV_VAL#CONFIG_*=}" = "y" ]; then >> - ${WARNOVVERIDE} Previous value: $PREV_VAL >> + ${WARNOVERRIDE} Previous value: $PREV_VAL >> ${WARNOVERRIDE} New value: $NEW_VAL >> ${WARNOVERRIDE} -y passed, will not demote y to m >> ${WARNOVERRIDE}
Best regards, Mirsad
-- Mirsad Goran Todorovac Sistem inženjer Grafički fakultet | Akademija likovnih umjetnosti Sveučilište u Zagrebu System engineer Faculty of Graphic Arts | Academy of Fine Arts University of Zagreb, Republic of Croatia The European Union
| ![\](/images/icornerr.gif) |