1 patch for repository http://darcs.net/screened:

patch 31f2953b33a4a713544c22eac8c09b3446f9e91a
Author: Ben Franksen <ben.franksen@online.de>
Date:   Mon May 18 10:58:02 CEST 2020
  * fix creation of temporary directories
  
  The algorithm we used to generate a unique name is prone to race conditions
  and I occasionally got random test failures because of this. The new
  implementation uses the temporary package.

Attachments

• fix-creation-of-temporary-directories.dpatch
• patch-preview.txt
• unnamed

Looks like a good simplification

Unfortunately this is causing a test failure on Windows in diff.sh:

| diff: darcs-diff-f5276236205c89d1\old-temp1: No such file or directory
| diff: darcs-diff-f5276236205c89d1\new-temp1: No such file or directory

I haven't investigated the details at all yet.

I think this change should fix the problem you noticed on Windows. I am
pretty sure this is not Windows specific per se, rather I suspect that
timing differences have uncovered a bug here.

1 patch for repository http://darcs.net/screened:

patch 97dfa3830cac2b55f8745b9c35321fd512c73762
Author: Ben Franksen <ben.franksen@online.de>
Date:   Wed Jun 24 15:36:07 CEST 2020
  * diff command: withTempDir->withDelayedDir

Attachments

• diff-command_-withtempdir__withdelayeddir.dpatch
• patch-preview.txt
• unnamed

As processes that run darcs commands are supposed to be short-lived, 
perhaps we should make withDelayedDir the standard way to handle 
temporary directories. The problem here is that we usually read hashed 
files lazily, which makes it pretty hard to reason about when it is 
safe to finally remove the directory and its content.

I have pushed the (hopefully) fix for your test failure to screened.

Unfortunately it's still failing (I double-checked I have the
"diff command: withTempDir->withDelayedDir" patch) - full output
attached.

Attachments

• diff.txt

> Unfortunately it's still failing (I double-checked I have the
> "diff command: withTempDir->withDelayedDir" patch) - full output
> attached.

Hmm. The output says

| diff: darcs-diff-e7b8195ee68f0c35\old-temp1: No such file or directory

Note the Windows-like '\' path separator here. Could it be that this is
a similar problem to one we had previously where a native Windows
version of an external command is used instead of the cygwin one?

Can you add --debug to the first 'darcs diff' command in the test and 
place an 'exit 1' after that, then attach the output?

Hi Ganesh, I think this patch should fix the problem on Windows.

1 patch for repository http://darcs.net/screened:

patch 5a047f6168e6c682bbf49609d9bd5d6aa0c77da8
Author: Ben Franksen <ben.franksen@online.de>
Date:   Fri Jul 10 13:11:04 CEST 2020
  * fix in diff command
  
  We must not use the native System.FilePath's </> operator and then call
  System.FilePath.Posix.takeFileName on the result.

Attachments

• fix-in-diff-command.dpatch
• patch-preview.txt
• unnamed

That works. Thanks a lot for fixing this even when I have been so
unresponsive about debugging it.