Bug#669271: [PATCH] set HOME to a tmpdir before running 'unopkg list'

[Rene Engelhard <rene@debian.org>]

Hi,

On Thu, May 10, 2012 at 05:14:31PM +0200, Andreas Beckmann wrote:
> diff --git a/shell-lib-extensions.sh b/shell-lib-extensions.sh
> index c0e74bd..41619aa 100644
> --- a/shell-lib-extensions.sh
> +++ b/shell-lib-extensions.sh
> @@ -12,7 +12,7 @@ sync_extensions() {
>    else
>  	d=/usr/lib/libreoffice
>    fi
> -  if /usr/lib/libreoffice/program/unopkg list --bundled >/dev/null 2>/dev/null; then
> +  if HOME=$INSTDIR /usr/lib/libreoffice/program/unopkg list --bundled >/dev/null 2>/dev/null; then
>      if [ -e /usr/lib/libreoffice/share/prereg/bundled ] && readlink /usr/lib/libreoffice/share/prereg/bundled 2>&1 >/dev/null && [ -L $d/ure-link ]; then
>        HOME=$INSTDIR \
>  	/usr/lib/libreoffice/program/unopkg sync -v --bundled \
> @@ -20,8 +20,8 @@ sync_extensions() {
>          "-env:UserInstallation=file://$INSTDIR" \
>          "-env:UNO_JAVA_JFW_INSTALL_DATA=file://$d/share/config/javasettingsunopkginstall.xml" \
>          "-env:JFW_PLUGIN_DO_NOT_CHECK_ACCESSIBILITY=1"
> -      rm -rf $INSTDIR
>      fi
>    fi
> +  rm -rf $INSTDIR
>  }

Interesting. I initially also thought --list could be a problem too but the results in
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=669271#10 suggested otherwise.

But the patch shouldn't do harm, so will commit when I am back home...

Regards,

Rene