On 5/14/21 1:54 PM, Iain Buclaw wrote:
>
> Oops, sorry I meant src/MERGE, druntime has nothing to do with it.
>
> So yes, please add 63f4caa900e17c541042617b2fa187059b86bf88 to
> libphobos/src/MERGE and it's good to go.
>
Okay, so this is what I've pushed now:
commit cb787efa45782adab764575a2efc356e082828b6
Author: Bernd Edlinger <bernd.edlin...@hotmail.de>
Date: Fri May 14 07:10:59 2021 +0200
Cleanup temp files in libphobos unittest at src/std/process.d
2021-05-14 Bernd Edlinger <bernd.edlin...@hotmail.de>
* src/std/process.d (unittest): Remove tmpname on exit.
* src/MERGE: Merge upstream phobos 63f4caa90.
diff --git a/libphobos/src/MERGE b/libphobos/src/MERGE
index 49622c5..ac709f9 100644
--- a/libphobos/src/MERGE
+++ b/libphobos/src/MERGE
@@ -1,4 +1,4 @@
-32cfe9b61570d52d9885b0208fd20de0d351b51e
+63f4caa900e17c541042617b2fa187059b86bf88
The first line of this file holds the git revision number of the last
merge done from the dlang/phobos repository.
diff --git a/libphobos/src/std/process.d b/libphobos/src/std/process.d
index 63ec493..1e977aa 100644
--- a/libphobos/src/std/process.d
+++ b/libphobos/src/std/process.d
@@ -2581,6 +2581,7 @@ private auto executeImpl(alias pipeFunc, Cmd,
ExtraPipeFuncArgs...)(
ReturnType!executeShell r;
auto tmpname = uniqueTempPath;
+ scope(exit) if (exists(tmpname)) remove(tmpname);
auto t = stderr;
// Open a new scope to minimize code ran with stderr redirected.
{
Thanks
Bernd.
