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

patch ca16adaaef0d048f216a7ae1939a098872645860
Author: Ben Franksen <benjamin.franksen@helmholtz-berlin.de>
Date:   Sun Jun 28 18:23:56 CEST 2015
  * resolve issue2461: darcs log --repo=remoterepo creates and populates _darcs
  
  The solution is to make a case distinction between remote and local usage.
  If we are working in a remote location, check that file arguments are
  relative paths and then do the log action inside a temporary directory.

Attachments

• patch-preview.txt
• resolve-issue2461_-darcs-log-__repo_remoterepo-creates-and-populates-_darcs.dpatch
• unnamed

I haven't run the tests, but in screened, "darcs log
--repo=http://darcs.net" still creates a local _darcs directory with
this patch. Also in branch 2.10 + this patch.

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

patch 807dfd7195309514598f28292691629d5efa1331
Author: Ben Franksen <benjamin.franksen@helmholtz-berlin.de>
Date:   Wed Jul  1 21:10:23 CEST 2015
  * resolve issue2461, this time also when no file names are specified
  
  Also slightly improve the error message when invalid arguments (non-relative
  paths) are given for remote repos.

Attachments

• patch-preview.txt
• resolve-issue2461_-this-time-also-when-no-file-names-are-specified.dpatch
• unnamed

You should run the tests, your last patch prevents "darcs log" without
arguments from working inside of a local repository.

Sorry, stupid mistake. Will amend & re-send.

Amended (also slightly re-factored) version of the latest fix of the fix.
Ran the tests this time before sending ;)

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

patch 47911083e7f069401697df70ecb0d2540f7a3597
Author: Ben Franksen <benjamin.franksen@helmholtz-berlin.de>
Date:   Wed Jul  1 21:10:23 CEST 2015
  * resolve issue2461, this time also when no file names are specified
  
  Also slightly improve the error message when invalid arguments (non-relative
  paths) are given for remote repos.

Attachments

• patch-preview.txt
• resolve-issue2461_-this-time-also-when-no-file-names-are-specified.dpatch
• unnamed

Thanks, it works now, nice fix for 2.10.1!

There seems to be yet another possibility for this to fail. I added
these lines to tests/network/log.sh:

# also test that it works without any filename arguments
darcs log --repo=http://darcs.net --last 1

(Note that we are inside "/" at this moment.) This fails with

| darcs failed:  Not a repository: http://darcs.net (.:
openBinaryTempFile: permission denied (Permission denied))
| 
| HINT: Do you have the right URI for the repository?

However, when I try this outside of the test harness it works!

ben@sarun[1]: / > pwd
/
ben@sarun[1]: / > darcs log --repo=http://darcs.net --last 1
patch 050d575189bd701f7a214b7e1d01aebe17ea2ad9
Author: Ben Franksen <benjamin.franksen@helmholtz-berlin.de>
Date:   Thu Jul  2 15:06:28 CEST 2015
  * simplified code in D.R.Ssh by using withMVar

False alarm. I ran the tests with a stale darcs.