darcs

Issue 645 external.pl test hangs under certain conditions

Title external.pl test hangs under certain conditions
Priority bug Status resolved
Milestone Resolved in
Superseder Nosy List darcs-devel, dmitry.kurochkin, eivuokko, jaredj, kowey, markstos, thorkilnaur, tommy, wglozer
Assigned To
Topics Mac

Created on 2008-02-05.10:12:03 by kowey, last changed 2009-08-27.13:59:25 by admin.

Messages
msg3118 (view) Author: kowey Date: 2008-02-05.10:11:59
The test uses a trivial shell script to stand in for ssh, via the DARCS_SSH
environment variable.

For some reason, darcs insists on using ssh instead of that trivial shell script.

And strangely, the test to make sure it's not (just) doing that passes (we check
to see if the script was launched, i.e. if it created the fake-ssh file, and
indeed it does).

I know it's trying to call the real ssh somewhere because if I replace foo.bar
with darcs.net, I get a passphrase prompt.

Puzzled.
msg3121 (view) Author: markstos Date: 2008-02-05.14:27:37
Thanks for the report Eric. What's your shell environment that this happens in?
(zsh again?). Do you have alternate shell to try running the script in?

You can emulate how the test suite runs the test and still run just one test
like this:

cd ./test
./bin/prove -v external.pl

(a shell script would be: perl shell_harness script.sh).
msg3124 (view) Author: droundy Date: 2008-02-05.15:40:31
I think Eric that this bug is fixed by the patch you sent in? I'm tentatively
setting this to resolved, please correct me if I'm wrong.
msg3138 (view) Author: kowey Date: 2008-02-05.22:22:30
On Tue, Feb 05, 2008 at 15:40:33 -0000, David Roundy wrote:
> I think Eric that this bug is fixed by the patch you sent in? I'm tentatively
> setting this to resolved, please correct me if I'm wrong.

To my dismay (and surprise), no, it is not fixed by my patch.
Huh...
msg3169 (view) Author: markstos Date: 2008-02-07.04:05:09
Eric,

Let's get a rundown of your environment where this bug is triggered. The test
passes for me under these conditions, with Darcs 1.0.9, or a recent build from
"unstable", built with GHC 6.6.1:

- Perl 5.8.8
- Bash 3.2.25
- Ubuntu Gutsy Linux 7.10. 

Is this on Windows by chance? I do see a line in script that is Windows-specific.
msg4737 (view) Author: kowey Date: 2008-05-16.23:57:15
Argh, this really should have fallen under the
if-it-takes-less-than-two-minutes-do-it-right-now rule

Hmm... now passes under MacOS X 10.5 :-/ Any Tiger users see this fail? It may
be worth getting a slave for the buildbot.

Bah.  I'm marking this resolved.  In the worst case, we can move the relevant
test to network/

Passing:
- MacOS X 10.5 (darwin 9.2.2)
- zsh 4.3.4
- Perl 5.8.8

Failing:
- MacOS X 10.4
- zsh ???
- Perl ???
History
Date User Action Args
2008-02-05 10:12:03koweycreate
2008-02-05 14:27:39markstossetstatus: unread -> unknown
nosy: + markstos
messages: + msg3121
2008-02-05 15:40:33droundysetstatus: unknown -> resolved
nosy: droundy, tommy, beschmi, kowey, markstos
messages: + msg3124
2008-02-05 22:22:32koweysetstatus: resolved -> unknown
nosy: droundy, tommy, beschmi, kowey, markstos
messages: + msg3138
2008-02-07 04:05:10markstossetstatus: unknown -> waiting-for
nosy: droundy, tommy, beschmi, kowey, markstos
messages: + msg3169
title: external.pl test hangs -> external.pl test hangs under certain conditions
2008-02-17 01:38:33markstossetstatus: waiting-for -> deferred
nosy: + wglozer, eivuokko, rgm, jaredj
topic: + Windows
assignedto: kowey
2008-05-16 23:57:18koweysetstatus: deferred -> resolved
nosy: + dagit
topic: + Mac, - Windows, X_DeprecatedTopic_Testing
messages: + msg4737
assignedto: kowey ->
2008-05-17 05:39:43rgmsetnosy: - rgm
2009-08-06 17:53:17adminsetnosy: + jast, Serware, dmitry.kurochkin, darcs-devel, zooko, mornfall, simon, thorkilnaur, - droundy, wglozer, eivuokko, jaredj
2009-08-06 20:56:34adminsetnosy: - beschmi
2009-08-10 22:13:22adminsetnosy: + wglozer, eivuokko, jaredj, - darcs-devel, zooko, jast, Serware, mornfall
2009-08-11 00:06:08adminsetnosy: - dagit
2009-08-25 18:04:43adminsetnosy: + darcs-devel, - simon
2009-08-27 13:59:25adminsetnosy: tommy, kowey, markstos, wglozer, darcs-devel, eivuokko, thorkilnaur, jaredj, dmitry.kurochkin