Closed schrockwell closed 8 months ago
@schrockwell Thank you for this alternative implementation! It makes a lot of sense to me and you've tested it well. I'm going to test it in a few more places and hopefully we can get this in.
Thanks again for fixing this!
This is a follow-up to #778 and a proposed fix for #781.
Here's what I know:
mktemp
on macOS 13 does not support the-p
option (as reported in #781)-t
option is problematic-t
option is deprecatedmktemp -d -t 'merge-squashfs-tmp'
doesn't even work on Linux; the template string requires "X" characters as placeholdersmktemp
does not consistently honor$TMPDIR
mktemp
honors$TMPDIR
by defaultmktemp
does NOT honor$TMPDIR
by defaultSo, my proposed solution avoids
-t
and-p
and is about as simple and explicit as possible, and it has been tested as working equally well on macOS 13, 14, and Ubuntu 18, and with or without$TMPDIR
being set.