Re: pkg/45181 (the new patch uses mktemp with the -p option, which is not portable.)

[Alan Barrett <apb%cequrux.com@localhost>]

The following reply was made to PR pkg/45181; it has been noted by GNATS.

From: Alan Barrett <apb%cequrux.com@localhost>
To: gnats-bugs%NetBSD.org@localhost
Cc: 
Subject: Re: pkg/45181 (the new patch uses mktemp with the -p option, which
 is not portable.)
Date: Wed, 27 Jul 2011 18:46:20 +0200

 On Wed, 27 Jul 2011, Tim Zingelman wrote:
 > Also, the patch you supplied does not work on Solaris at least... it
 > results in errors like:
 >
 > pdfroff: $(TMPDIR=/tmp mktemp -d -t
 > groff-pdfroff.XXXXXXXXXX)/pdf27836.in: cannot create
 
 If you use the REPLACE_SH mechanism, then #!/bin/sh can be 
 changed to something appropriate for the target system.  If 
 I am reading pkgsrc/mk/configure/replace-interpreter.mk and 
 pkgsrc/mk/tools/tools.SunOS.mk correctly, then it looks as though 
 REPLACE_SH would use #!/bin/ksh on Solaris.
 
 --apb (Alan Barrett)