Padraig O'Briain wrote:
I have prepared another webrev, http://cr.opensolaris.org/~padraig/ips-7852-v2/.

I think this addresses the concerns raised.

Instead of just hardcoding /var/tmp, why not use tempfile.mkdtemp?

Then just make sure that gets removed on exit.

Cheers,
--
Shawn Walker
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to