[Ltp-cvs] utils/analysis/lcov/bin geninfo,1.122,1.123

[Peter O. <obe...@us...>]

Update of /cvsroot/ltp/utils/analysis/lcov/bin
In directory sfp-cvs-1.v30.ch3.sourceforge.com:/tmp/cvs-serv28926

Modified Files:
	geninfo 
Log Message:
geninfo: Fix handling of non-english locales

geninfo expects gcov output in the default C locale. This isn't always
given, for example when running in an environment where variable
LANGUAGE is set to a non-english locale. In such cases gcov output
cannot be correctly parsed, resulting for example in the absence of
branch coverage data.

gcov uses gettext() for writing internationalized messages. The info
page for gettext mentions the order in which locale-defining
environment variables are evaluated:

LANGUAGE
LC_ALL
LC_MESSAGES
LANG

In addition, gettext implements special handling where LC_ALL=C takes
precedence over LANGUAGE.

geninfo currently only specifies LANG=C. Fix the issue by specifying
LC_ALL=C instead.

Based on fix suggestion by Sebastian Stigler.

Reported-by: Sebastian Stigler <s_s...@gm...>


Index: geninfo
===================================================================
RCS file: /cvsroot/ltp/utils/analysis/lcov/bin/geninfo,v
retrieving revision 1.122
retrieving revision 1.123
diff -C2 -d -r1.122 -r1.123
*** geninfo	2 Aug 2013 07:29:20 -0000	1.122
--- geninfo	12 Dec 2013 14:58:44 -0000	1.123
***************
*** 267,272 ****
  $lcov_version =~ s/\$\s*Revision\s*:?\s*(\S+)\s*\$/$1/;
  
! # Set LANG so that gcov output will be in a unified format
! $ENV{"LANG"} = "C";
  
  # Check command line for a configuration file name
--- 267,272 ----
  $lcov_version =~ s/\$\s*Revision\s*:?\s*(\S+)\s*\$/$1/;
  
! # Set LC_ALL so that gcov output will be in a unified format
! $ENV{"LC_ALL"} = "C";
  
  # Check command line for a configuration file name