|
Created on 2006-04-17.19:11:30 by droundy, last changed 2009-08-27.13:47:12 by admin.
msg618 (view) |
Author: droundy |
Date: 2006-04-17.19:11:27 |
|
$ /tmp/darcs-1.0.6-i386-linux changes | head -1
Tue Apr 11 13:06:10 EDT 2006 daykov@physics.cornell.edu
darcs failed: resource vanished
It looks like we've reverted this bug again. :( It'd be nice to add a
check for the testsuite for this bug, since it seems to pop up pretty
regularly and get fixed pretty regularly. (submitting the bug report from
work...)
--
David Roundy
|
msg619 (view) |
Author: droundy |
Date: 2006-04-18.11:12:40 |
|
This one seems a bit tricky, as I can't reproduce it in a "small"
repository. :( I've got a test script which fails to reproduce it. :( The
bug shows up in all the large, real repositories I've tried, but I can't
see what the difference is, why the test repos (and one "real" repo with
only four patches in it) don't run into this trouble. :(
--
David Roundy
http://www.darcs.net
|
msg625 (view) |
Author: droundy |
Date: 2006-05-01.14:27:08 |
|
Okay, I tracked this one down. Here's the patch (whose comment explains
how this came to pass).
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
New patches:
[fix error is is_pipe test in error reporting. (fixes Issue160)
David Roundy <droundy@darcs.net>**20060501142114
The trouble was that Ian (quite naturally) assumed that my C function
stdout_is_a_pipe returned nonzero for true, whereas for some very, very
backwards reason it returned zero for true, and its result was properly
interpreted. So I caused this bug by my (unexplained) backwards
programming, but it was introduced when Ian refactored the C code. :(
]
<
> {
hunk ./SignalHandler.lhs 58
exitWith $ ExitFailure $ 1
die_with_string e | take 6 e == "STDOUT" =
do is_pipe <- stdout_is_a_pipe
- - when is_pipe $
+ when (not is_pipe) $
hPutStrLn stderr $ "\ndarcs failed: "++drop 6 e
exitWith $ ExitFailure $ 2
die_with_string e = do hPutStrLn stderr $ "\ndarcs failed: "++e
hunk ./changelog.in/entries/droundy 12
# If other developers prefer, they could also create a new file for each
# change. I'm still working out how best to handle this new changelog
# scheme.
+
+match: name "fix error is is_pipe test.*fixes Issue160"
+
+> Fixed bug leading to a spurious "darcs failed: resource vanished" error
+> message when darcs output is piped to a program such as head that then
+> exits. (Issue160, David Roundy)
match: name "speed up replace by first checking on each line"
}
Context:
[TAG 1.0.7pre1
Tommy Pettersson <ptp@lysator.liu.se>**20060427095905]
Patch bundle hash:
8baaa65ed94963be2361cc1fc4b86a4bb2040845
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.1 (GNU/Linux)
iD8DBQFEVhoLQ6uZI9PVvOwRAl+cAKDQpDwCDtF6GFK36DvMqcr4MrgNHQCffcox
75ICVJW/CKVcbgecZHmGfE0=
=vBWp
-----END PGP SIGNATURE-----
|
msg644 (view) |
Author: tommy |
Date: 2006-05-13.18:08:37 |
|
included in 1.0.7
|
|
Date |
User |
Action |
Args |
2006-04-17 19:11:30 | droundy | create | |
2006-04-18 11:12:41 | droundy | set | status: unread -> unknown messages:
+ msg619 |
2006-05-01 14:17:39 | droundy | set | nosy:
+ jch |
2006-05-01 14:27:09 | droundy | set | messages:
+ msg625 |
2006-05-01 15:26:48 | droundy | set | topic:
+ Patch |
2006-05-04 16:06:41 | jch | set | status: unknown -> resolved-in-unstable |
2006-05-13 18:08:38 | tommy | set | status: resolved-in-unstable -> resolved messages:
+ msg644 |
2009-08-06 17:43:49 | admin | set | nosy:
+ markstos, jast, Serware, dmitry.kurochkin, darcs-devel, zooko, dagit, mornfall, simon, kowey, beschmi, thorkilnaur, - droundy, jch |
2009-08-06 20:49:35 | admin | set | nosy:
- beschmi |
2009-08-10 21:51:32 | admin | set | nosy:
+ jch, - markstos, darcs-devel, zooko, jast, dagit, Serware, mornfall |
2009-08-25 17:56:57 | admin | set | nosy:
+ darcs-devel, - simon |
2009-08-27 13:47:12 | admin | set | nosy:
jch, tommy, kowey, darcs-devel, thorkilnaur, dmitry.kurochkin |
|