Hello Arthur,

Good idea to use mktemp.  Thanks for your help.  But, I have some
questions about the patch.

On Fri, Mar 17, 2006 at 11:23:13PM +0100, Arthur de Jong wrote:
> Subject: flashplugin-nonfree: please use /var/tmp instead of /var/run

The patch doesn't seem to use /var/tmp.  Would it be OK to create the
temporary directory in /tmp instead ?

> The attached patch uses mktemp to create a temporary directory in the
> default /tmp directory (which can be set by the TMPDIR environment
> variable).
> +             TMPDIR=`mktemp -d flashplugin-nonfree.XXXXXX`

The patch seems to create a temporary directory in the current
directory, not in /tmp.  I guess adding "-p /tmp" should solve this.

I would use UNPACKDIR instead of TMPDIR to store the result of mktemp,
because TMPDIR already has a special meaning to mktemp.

Regards,

Bart Martens

Attachment: signature.asc
Description: Digital signature

Reply via email to