[kteam-tools] maint-getabis: ABI sorting with C locale

Whenever different people update a kernel ABI, it's possible a different locale
is used, and ABI is changed, though nothing has really changed. This causes
some spurios ABI changes that make it more difficult to review differentes
between releases.

Signed-off-by: Thadeu Lima de Souza Cascardo <cascardo@canonical.com>
---
 maintscripts/maint-getabis | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

On Thu, Feb 09, 2017 at 12:24:47PM -0200, Thadeu Lima de Souza Cascardo wrote:
> Whenever different people update a kernel ABI, it's possible a different locale
> is used, and ABI is changed, though nothing has really changed. This causes
> some spurios ABI changes that make it more difficult to review differentes
> between releases.
> 
> Signed-off-by: Thadeu Lima de Souza Cascardo <cascardo@canonical.com>
> ---
>  maintscripts/maint-getabis | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/maintscripts/maint-getabis b/maintscripts/maint-getabis
> index f375cc1..2defb70 100755
> --- a/maintscripts/maint-getabis
> +++ b/maintscripts/maint-getabis
> @@ -213,7 +213,8 @@ def DownloadPackage(pkgname, srcpkgname):
>  # remove the package on success.
>  #------------------------------------------------------------------------------
>  def ExtractPackage(pkg, tmpdir, abiversion, flavour):
> -	cmd  = "tmpdir=" + tmpdir + "\n"
> +	cmd  = "export LC_ALL=C.UTF-8\n"
> +	cmd += "tmpdir=" + tmpdir + "\n"
>  	cmd += "pkg=" + pkg + "\n"
>  	cmd += "abiver=" + abiversion + "\n"
>  	cmd += "flavour=" + flavour + "\n"

If we need this here, do we not want this in "getabis" in the trees as
well and for the same reason.

-apw

On Sat, Feb 11, 2017 at 04:15:31PM +0000, Andy Whitcroft wrote:
> On Thu, Feb 09, 2017 at 12:24:47PM -0200, Thadeu Lima de Souza Cascardo wrote:
> > Whenever different people update a kernel ABI, it's possible a different locale
> > is used, and ABI is changed, though nothing has really changed. This causes
> > some spurios ABI changes that make it more difficult to review differentes
> > between releases.
> > 
> > Signed-off-by: Thadeu Lima de Souza Cascardo <cascardo@canonical.com>
> > ---
> >  maintscripts/maint-getabis | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> > 
> > diff --git a/maintscripts/maint-getabis b/maintscripts/maint-getabis
> > index f375cc1..2defb70 100755
> > --- a/maintscripts/maint-getabis
> > +++ b/maintscripts/maint-getabis
> > @@ -213,7 +213,8 @@ def DownloadPackage(pkgname, srcpkgname):
> >  # remove the package on success.
> >  #------------------------------------------------------------------------------
> >  def ExtractPackage(pkg, tmpdir, abiversion, flavour):
> > -	cmd  = "tmpdir=" + tmpdir + "\n"
> > +	cmd  = "export LC_ALL=C.UTF-8\n"
> > +	cmd += "tmpdir=" + tmpdir + "\n"
> >  	cmd += "pkg=" + pkg + "\n"
> >  	cmd += "abiver=" + abiversion + "\n"
> >  	cmd += "flavour=" + flavour + "\n"
> 
> If we need this here, do we not want this in "getabis" in the trees as
> well and for the same reason.
> 
> -apw

You are right. As I changed my locale, I did not realize changing that portion
of the script was not enough. In fact, it has not effect in most trees. Though
this change should still apply anyway.

I will send patches for the trees we care while I work at it.

Thanks.
Cascardo.

Needed in debian/scripts/misc/getabis as well