darcs

Issue 869 push --dry-run => takeFile _darcs/patches/pending.tentative-0 ... (Permission denied) (2.0.0 + 244)

Title push --dry-run => takeFile _darcs/patches/pending.tentative-0 ... (Permission denied) (2.0.0 + 244)
Priority bug Status resolved
Milestone Resolved in
Superseder fix bug where we were calling revertRepository when running with --dry-run.
View: 855
Nosy List darcs-devel, dmitry.kurochkin, jch, kowey, thorkilnaur, tommy
Assigned To
Topics

Created on 2008-05-20.18:22:05 by kowey, last changed 2009-08-27.14:09:58 by admin.

Files
File name Uploaded Type Edit Remove
unnamed dagit, 2008-05-20.18:25:49 text/html
Messages
msg4800 (view) Author: kowey Date: 2008-05-20.18:22:03
Sorry Juliusz, I don't think this issue is completely dead.

We're not trying to take that lock anymore, but it still fails when trying to
create the tentative pending file (why would it be trying to do that? it's a dry
run).

I think this means there's a flaw in my test script.  Assigning to myself to
look at the test :-/
msg4802 (view) Author: dagit Date: 2008-05-20.18:25:49
On Tue, May 20, 2008 at 11:22 AM, Eric Kow <bugs@darcs.net> wrote:

>
> New submission from Eric Kow <eric.kow@gmail.com>:
>
> Sorry Juliusz, I don't think this issue is completely dead.
>
> We're not trying to take that lock anymore, but it still fails when trying
> to
> create the tentative pending file (why would it be trying to do that? it's
> a dry
> run).

In a dry run I might expect to get a summary of what works and what
doesn't.  That is, failure is noted but not fatal.  So I might expect darcs
to tell me that a lock cannot be taken and that I would need it for a wet
run (is that the right term?).

It probably makes your changes more difficult, but I thought I'd mention it.

Jason
Attachments
msg4804 (view) Author: kowey Date: 2008-05-21.10:23:38
Test case re-submitted/corrected.  It now fails like it should, and Eric
understands shell scripts a little bit better.
msg4819 (view) Author: kowey Date: 2008-05-21.13:34:38
Ah! David fixed my fix.
The following patch updated the status of issue855 (and issue864) to be resolved:

* resolve issue855: fix bug where we were calling revertRepository when running
with --dry-run.
History
Date User Action Args
2008-05-20 18:22:05koweycreate
2008-05-20 18:25:52dagitsetfiles: + unnamed
nosy: jch, tommy, beschmi, kowey, dagit
status: unread -> unknown
messages: + msg4802
title: push --dry-run => takeFile _darcs/patches/pending.tentative-0 ... (Permission denied) (2.0.0 + 244) -> push --dry-run => takeFile _darcs/patches/pending.tentative-0 ... (Permission denied) (2.0.0 + 244)
2008-05-21 10:23:40koweysetnosy: jch, tommy, beschmi, kowey, dagit
messages: + msg4804
assignedto: kowey ->
2008-05-21 13:34:40koweysetstatus: unknown -> resolved-in-unstable
nosy: jch, tommy, beschmi, kowey, dagit
messages: + msg4819
2008-09-04 21:33:28adminsetstatus: resolved-in-unstable -> resolved
nosy: jch, tommy, beschmi, kowey, dagit
2009-08-06 21:05:30adminsetnosy: + dmitry.kurochkin, simon, thorkilnaur, - beschmi
2009-08-11 00:14:56adminsetnosy: - dagit
2009-08-25 18:11:15adminsetnosy: + darcs-devel, - simon
2009-08-27 14:09:58adminsetnosy: jch, tommy, kowey, darcs-devel, thorkilnaur, dmitry.kurochkin