[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#754463: RFS: pdf2htmlex/0.11+ds-1



Hi,

Quoting Jakub Wilk (2014-07-17 13:36:47)
> export HOME=`mktemp --dry-run`
> 
> This sets HOME literally to `mktemp --dry-run`. I think you wanted to 
> say:
> 
> export HOME=$(shell mktemp --dry-run)

oh shoot it's not shell, it's make... while that method will surely also yield
a nonexistant home directory I see why that method is inferior ^^

> But there's a good reason --dry-run is described as “unsafe” in the mktemp
> manpage.

What is the reason? I thought the reason for it being called "unsafe" was that
if you use --dry-run first and then create the directory with that name
yourself then somebody else could hijack that location in the meantime. But
this is no problem for this use case.

> So how about something like this instead:
> 
> export HOME=$(CURDIR)/nonexistent

This is a good solution. As we have control about the directories in $(CURDIR)
we can guarantee that no directory named like that exists during build time.
I'll use it.

cheers, josch


Reply to: