Changes between Version 1 and Version 2 of Ticket #2432, comment 3
- Timestamp:
- 2015-05-26T23:58:15Z (10 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #2432, comment 3
v1 v2 1 The one thing I don't like about `mktemp()` is that the path it generates is not deterministic, due to its use of `mkdtemp` ([https://twistedmatrix.com/trac/browser/tags/releases/twisted-15.2.0/twisted/trial/_synctest.py#L1221 implementation in Twisted v15.2.0]). What I actually want is `os.path.dirname(os.path.dirname(self.mktemp())`, I think. So, maybe it would be better to have a mixin to shorten that, so that if we want to change it, we only need to do so in one place.1 The one thing I don't like about `mktemp()` is that the path it generates is not deterministic, due to its use of `mkdtemp` ([https://twistedmatrix.com/trac/browser/tags/releases/twisted-15.2.0/twisted/trial/_synctest.py#L1221 implementation in Twisted v15.2.0]). What I actually want is `os.path.dirname(os.path.dirname(self.mktemp())`, I think. Maybe it would be better to have a mixin to shorten that, so that if we want to change it, we only need to do so in one place.