Skip Menu |

This queue is for tickets about the CPAN CPAN distribution.

Report information
The Basics
Id: 17771
Status: resolved
Priority: 0/
Queue: CPAN

People
Owner: Nobody in particular
Requestors: adamk [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Unimportant
Broken in: 1.85
Fixed in: (no value)



Subject: Win32 30shell.t message makes scary beep, and has other issues
Ignoring the issue of whether 30shell.t on things other than linux actually works, the way it fails is strange and scary and could be mistaken for an error. The three problems are 1. It makes a weird beep (\a presumably), harsher and quite unlike the normal Win32 beeps. 2. It puts a dollar sign in front of the \$CPAN_RUN_SHELL_TEST environment variable. 3. It warns, which shows the error line information, making it look like a big error. The skip should look normal and less scary. Something like... 1..0 # Skip: Only tested on linux. Set CPAN_RUN_SHELL_TEST=1 to force Then ditch the warn and exit normally.
Subject: Re: [rt.cpan.org #17771] Win32 30shell.t message makes scary beep, and has other issues
Date: Tue, 21 Feb 2006 04:16:52 +0100
To: bug-CPAN [...] rt.cpan.org
From: andreas.koenig.gmwojprw [...] franz.ak.mind.de (Andreas J. Koenig)
Show quoted text
>>>>> On Mon, 20 Feb 2006 19:13:39 -0500 (EST), " via RT" <bug-CPAN@rt.cpan.org> said:
Show quoted text
> Mon Feb 20 19:13:38 2006: Request 17771 was acted upon. > Transaction: Ticket created by ADAMK > Queue: CPAN > Subject: Win32 30shell.t message makes scary beep, and has other issues > Owner: Nobody > Requestors: ADAMK@cpan.org > Status: new > Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=17771 >
Show quoted text
> Ignoring the issue of whether 30shell.t on things other than linux > actually works, the way it fails is strange and scary and could be > mistaken for an error.
Show quoted text
> The three problems are
Show quoted text
> 1. It makes a weird beep (\a presumably), harsher and quite unlike the > normal Win32 beeps.
Show quoted text
> 2. It puts a dollar sign in front of the \$CPAN_RUN_SHELL_TEST > environment variable.
Show quoted text
> 3. It warns, which shows the error line information, making it look like > a big error.
Show quoted text
> The skip should look normal and less scary. Something like...
Show quoted text
> 1..0 # Skip: Only tested on linux. Set CPAN_RUN_SHELL_TEST=1 to force
Show quoted text
> Then ditch the warn and exit normally.
Thanks, I have already removed this code earlier this morning, around rev. 643 or 644 -- andreas