Apparantly tmpFileName() is broken. That would be the code to post. Be careful with this, there is a race if two of these get the same $tmpfile.

You're not really using File::Copy. Why not mv instead of cp? That would save cleanup.

After Compline,
Zaxo