LGTM

On Feb 24 10:54, Michele Tartara wrote:
> Precise has a different way to handle locales w.r.t. debian distributions.
> This patch adds code to make the distinction.
> 
> Signed-off-by: Michele Tartara <[email protected]>
> ---
>  devel/build_chroot | 17 +++++++++++++----
>  1 file changed, 13 insertions(+), 4 deletions(-)
> 
> diff --git a/devel/build_chroot b/devel/build_chroot
> index a1bba38..104d04b 100755
> --- a/devel/build_chroot
> +++ b/devel/build_chroot
> @@ -320,13 +320,22 @@ EOF
>  ;;
>  esac
>  
> -echo "en_US.UTF-8 UTF-8" >> $CHDIR/etc/locale.gen
> -
>  in_chroot -- \
>    $APT_INSTALL sudo fakeroot rsync locales less socat
>  
> -in_chroot -- \
> -  locale-gen
> +# Configure the locale
> +case $DIST_RELEASE in
> +  precise)
> +    in_chroot -- \
> +      $APT_INSTALL language-pack-en
> +    ;;
> +  *)
> +    echo "en_US.UTF-8 UTF-8" >> $CHDIR/etc/locale.gen
> +
> +    in_chroot -- \
> +      locale-gen
> +    ;;
> +esac
>  
>  in_chroot -- \
>    $APT_INSTALL lvm2 ssh bridge-utils iproute iputils-arping \
> -- 
> 1.9.0.rc1.175.g0b1dcb5
> 

-- 
Jose Antonio Lopes
Ganeti Engineering
Google Germany GmbH
Dienerstr. 12, 80331, München

Registergericht und -nummer: Hamburg, HRB 86891
Sitz der Gesellschaft: Hamburg
Geschäftsführer: Graham Law, Christine Elizabeth Flores
Steuernummer: 48/725/00206
Umsatzsteueridentifikationsnummer: DE813741370

Reply via email to