Index: sdk/lib/_internal/pub/lib/src/io.dart |
diff --git a/sdk/lib/_internal/pub/lib/src/io.dart b/sdk/lib/_internal/pub/lib/src/io.dart |
index 41a254392a1c0f75794633ccd82c3b2ca4ed1e31..af8168e461b073ad4ee1c712d40dc5d3e598a894 100644 |
--- a/sdk/lib/_internal/pub/lib/src/io.dart |
+++ b/sdk/lib/_internal/pub/lib/src/io.dart |
@@ -232,12 +232,19 @@ String ensureDir(String dirPath) { |
/// suffix appended to it. If [dir] is not provided, a temp directory will be |
/// created in a platform-dependent temporary location. Returns the path of the |
/// created directory. |
-String createTempDir([dir = '']) { |
- var tempDir = new Directory(dir).createTempSync(); |
+String createTempDir(String base, String prefix) { |
Bob Nystrom
2013/10/01 17:37:03
Is it a prefix or a suffix? The doc says one thing
Bill Hesse
2013/10/02 13:25:05
It is a prefix. Fixed the doc comment.
|
+ var tempDir = new Directory(base).createTempSync(prefix); |
log.io("Created temp directory ${tempDir.path}"); |
return tempDir.path; |
} |
+String createSystemTempDir() { |
Bob Nystrom
2013/10/01 17:37:03
Doc comment. Or just combine this with the above m
Bill Hesse
2013/10/02 13:25:05
Done.
|
+ var tempDir = Directory.createSystemTempSync('pub_'); |
+ log.io("Created temp directory ${tempDir.path}"); |
+ return tempDir.path; |
+} |
+ |
Bob Nystrom
2013/10/01 17:37:03
Remove extra blank line.
Bill Hesse
2013/10/02 13:25:05
Done.
|
+ |
/// Lists the contents of [dir]. If [recursive] is `true`, lists subdirectory |
/// contents (defaults to `false`). If [includeHidden] is `true`, includes files |
/// and directories beginning with `.` (defaults to `false`). |
@@ -648,7 +655,7 @@ Future timeout(Future input, int milliseconds, String description) { |
/// [fn] completes to. |
Future withTempDir(Future fn(String path)) { |
return new Future.sync(() { |
- var tempDir = createTempDir(); |
+ var tempDir = createSystemTempDir(); |
return new Future.sync(() => fn(tempDir)) |
.whenComplete(() => deleteEntry(tempDir)); |
}); |