Commit aa1acf59 authored by Daniel Dadap's avatar Daniel Dadap Committed by Kleber Sacilotto de Souza

scripts: override locale from environment when running recordmcount.pl

BugLink: https://bugs.launchpad.net/bugs/1828084

recordmcount.pl uses a set of regular expressions to parse the output of
objdump(1). However, if objdump(1) output is localized, it may not match
the regular expressions, thereby preventing recordmcount.pl from parsing
object files correctly.

In order to allow recordmcount.pl to function correctly regardless of the
current locale settings, set LANG=C when running objdump(1). LC_ALL is
already unset in the top-level Makefile, so it is not necessary to also
override that environment variable.
Signed-off-by: default avatarDaniel Dadap <ddadap@nvidia.com>
Reviewed-by: default avatarRobert Morell <rmorell@nvidia.com>
Signed-off-by: default avatarMasahiro Yamada <yamada.masahiro@socionext.com>
(cherry picked from commit e46b94d2)
Signed-off-by: default avatarKai-Heng Feng <kai.heng.feng@canonical.com>
Acked-by: default avatarStefan Bader <stefan.bader@canonical.com>
Acked-by: default avatarKleber Sacilotto de Souza <kleber.souza@canonical.com>
Signed-off-by: default avatarKleber Sacilotto de Souza <kleber.souza@canonical.com>
parent b05c8216
......@@ -468,7 +468,7 @@ sub update_funcs
#
# Step 2: find the sections and mcount call sites
#
open(IN, "$objdump -hdr $inputfile|") || die "error running $objdump";
open(IN, "LANG=C $objdump -hdr $inputfile|") || die "error running $objdump";
my $text;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment