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
signature.asc
Description: Digital signature

