* [PATCH] kbuild: fix adjust_autoksyms.sh for modules that need only one symbol @ 2016-04-28 21:33 Nicolas Pitre 2016-04-28 21:49 ` Arnd Bergmann 2016-05-10 15:25 ` Michal Marek 0 siblings, 2 replies; 3+ messages in thread From: Nicolas Pitre @ 2016-04-28 21:33 UTC (permalink / raw) To: Michal Marek; +Cc: Arnd Bergmann, linux-kbuild When only one symbol was listed and therefore the line didn't contain any space to separate multiple symbols, that symbol got ignored. Reported-by: Arnd Bergmann <arnd@arndb.de> Signed-off-by: Nicolas Pitre <nico@linaro.org> diff --git a/scripts/adjust_autoksyms.sh b/scripts/adjust_autoksyms.sh index 5bf538f1ed..8dc1918b67 100755 --- a/scripts/adjust_autoksyms.sh +++ b/scripts/adjust_autoksyms.sh @@ -59,7 +59,7 @@ cat > "$new_ksyms_file" << EOT */ EOT -sed -ns -e '3s/ /\n/gp' "$MODVERDIR"/*.mod | sort -u | +sed -ns -e '3{s/ /\n/g;/^$/!p;}' "$MODVERDIR"/*.mod | sort -u | while read sym; do if [ -n "$CONFIG_HAVE_UNDERSCORE_SYMBOL_PREFIX" ]; then sym="${sym#_}" ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] kbuild: fix adjust_autoksyms.sh for modules that need only one symbol 2016-04-28 21:33 [PATCH] kbuild: fix adjust_autoksyms.sh for modules that need only one symbol Nicolas Pitre @ 2016-04-28 21:49 ` Arnd Bergmann 2016-05-10 15:25 ` Michal Marek 1 sibling, 0 replies; 3+ messages in thread From: Arnd Bergmann @ 2016-04-28 21:49 UTC (permalink / raw) To: Nicolas Pitre; +Cc: Michal Marek, linux-kbuild On Thursday 28 April 2016 17:33:38 Nicolas Pitre wrote: > When only one symbol was listed and therefore the line didn't contain > any space to separate multiple symbols, that symbol got ignored. > > Reported-by: Arnd Bergmann <arnd@arndb.de> > Signed-off-by: Nicolas Pitre <nico@linaro.org> > > Tested-by: Arnd Bergmann <arnd@arndb.de> ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] kbuild: fix adjust_autoksyms.sh for modules that need only one symbol 2016-04-28 21:33 [PATCH] kbuild: fix adjust_autoksyms.sh for modules that need only one symbol Nicolas Pitre 2016-04-28 21:49 ` Arnd Bergmann @ 2016-05-10 15:25 ` Michal Marek 1 sibling, 0 replies; 3+ messages in thread From: Michal Marek @ 2016-05-10 15:25 UTC (permalink / raw) To: Nicolas Pitre; +Cc: Arnd Bergmann, linux-kbuild On Thu, Apr 28, 2016 at 05:33:38PM -0400, Nicolas Pitre wrote: > When only one symbol was listed and therefore the line didn't contain > any space to separate multiple symbols, that symbol got ignored. > > Reported-by: Arnd Bergmann <arnd@arndb.de> > Signed-off-by: Nicolas Pitre <nico@linaro.org> Applied to kbuild.git#kbuild. Michal ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-05-10 15:25 UTC | newest] Thread overview: 3+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2016-04-28 21:33 [PATCH] kbuild: fix adjust_autoksyms.sh for modules that need only one symbol Nicolas Pitre 2016-04-28 21:49 ` Arnd Bergmann 2016-05-10 15:25 ` Michal Marek
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox