On Mon, Jan 19, 2015 at 21:01:18 +0100, Paul Gevers wrote:

> On 18-01-15 22:42, Julien Cristau wrote:
> > Please don't use /tmp for this.  Use some place under the build tree, or
> > use mkdtemp.
> 
> I assume you mean mktemp here. So is the following better (and

No, I did mean mkdtemp(3).

> acceptable for the RT for jessie?)
> 
> Paul
> 
> --- a/debian/rules
> +++ b/debian/rules
> @@ -19,7 +19,8 @@ override_dh_auto_build-arch:
>       mv winff winff-qt
> 
>  override_dh_auto_build-indep:
> -     soffice --headless --convert-to pdf --outdir docs docs/WinFF.*.od?
> +# bug 768733: if $HOME does not exist, soffice fails to build pdf's
> +     HOME=$(shell mktemp -d) soffice --headless --convert-to pdf --outdir
> docs docs/WinFF.*.od?
>  # The language files are searched only for the first two characters of the
>  # language code, and we don't have a proper Spanish translation yet.
>       if [ ! -f docs/WinFF.es.pdf ] ; then \
> 
Now you're missing a rm -rf.  That was the point of the "somewhere under
the build tree" option.

Still wondering how it's 2015 and people still can't use temporary files
correctly.

Cheers,
Julien

Attachment: signature.asc
Description: Digital signature

Reply via email to