This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author giampaolo.rodola
Recipients Jeffrey.Kintscher, giampaolo.rodola, josh.r, riccardomurri, tarek
Date 2019-06-29.11:57:20
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1561809441.1.0.660823135731.issue36422@roundup.psfhosted.org>
In-reply-to
Content
> in case a filesystem has been mounted on the temporary directory, this can lead to the entire filesystem being removed

-1

That is expected behavior and the use case looks pretty unusual. Such a new parameter wouldn't even be supported by other "batteries" since there's no portable/standard way to either mount or unmount a directory (in fact you had to use a subprocess in your unit-tests).

A `delete=bool` parameter would be a more reasonable proposal in principle but:
1) if you want to keep the directory around then you can just use tempfile.mkdtemp() (see "there should preferably be only one way to do it")
2) it would conflict with the context manager usage which is expected to delete the dir on ctx manager exit

In summary, I think this would over-complicate the API for no good reason. 
I'm closing this out as rejected.
History
Date User Action Args
2019-06-29 11:57:21giampaolo.rodolasetrecipients: + giampaolo.rodola, tarek, riccardomurri, josh.r, Jeffrey.Kintscher
2019-06-29 11:57:21giampaolo.rodolasetmessageid: <1561809441.1.0.660823135731.issue36422@roundup.psfhosted.org>
2019-06-29 11:57:21giampaolo.rodolalinkissue36422 messages
2019-06-29 11:57:20giampaolo.rodolacreate