Skip Menu |

This queue is for tickets about the Term-ReadLine-Perl CPAN distribution.

Report information
The Basics
Id: 20237
Status: open
Priority: 0/
Queue: Term-ReadLine-Perl

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

Bug Information
Severity: Normal
Broken in: (no value)
Fixed in: (no value)



Subject: test file prompt needs to include how to exit the prompt
"Enter a perl expression" doesn't clearly indicate that typing "exit" is what gets out of it and moves on. This could be a problem for less sophisicated users. Please update the prompt statement in the next release.
CC: undisclosed-recipients: ;
Subject: Re: [rt.cpan.org #20237] test file prompt needs to include how to exit the prompt
Date: Mon, 3 Jul 2006 03:27:23 -0700
To: via RT <bug-Term-ReadLine-Perl [...] rt.cpan.org>
From: Ilya Zakharevich <nospam-abuse [...] ilyaz.org>
On Sun, Jul 02, 2006 at 08:28:29AM -0400, via RT wrote: Show quoted text
> "Enter a perl expression" doesn't clearly indicate that typing "exit" is > what gets out of it and moves on. This could be a problem for less > sophisicated users. Please update the prompt statement in the next release.
Do you mean the case when T::R::P is disabled? Otherwise 'exit' is the "default" value, so just pressing Enter will do it... Thanks, Ilya
Subject: Re: [rt.cpan.org #20237] test file prompt needs to include how to exit the prompt
Date: Mon, 03 Jul 2006 07:33:49 -0400
To: bug-Term-ReadLine-Perl [...] rt.cpan.org
From: David Golden <dagolden [...] cpan.org>
Ilya Zakharevich via RT wrote: Show quoted text
> Do you mean the case when T::R::P is disabled? Otherwise 'exit' is > the "default" value, so just pressing Enter will do it...
Apologies for the short diagnostic message. At least during testing with T::R::P 1.0302 for Win32 Vanilla Perl 5.8.8, just pressing Enter didn't do it. (Could this be an issue with newlines?) Here's an example of the output (the third example is just hitting "Enter"): Show quoted text
> [C:\Temp\cpan\build\Term-ReadLine-Perl-1.0302]
>> dmake test
> C:\vanilla-perl\perl\bin\perl.exe "-Iblib\lib" "-Iblib\arch" test.pl > Features present: preput 1 getHistory 1 addHistory 1 attribs 1 ornaments 1 appname 1 minli > ne 1 autohistory 1 newTTY 1 tkRunning 1 setHistory 1 > Flipping rl_default_selected each line. > Enter arithmetic or Perl expression: 1 > 1 > Enter arithmetic or Perl expression: 1+1 > 2 > Enter arithmetic or Perl expression: > Use of uninitialized value in print at test.pl line 45, <IN> line 3. > at test.pl line 45 > > Enter arithmetic or Perl expression: exit > > > [C:\Temp\cpan\build\Term-ReadLine-Perl-1.0302]
Regards, David
CC: undisclosed-recipients: ;
Subject: Re: [rt.cpan.org #20237] test file prompt needs to include how to exit the prompt
Date: Mon, 3 Jul 2006 13:08:14 -0700
To: via RT <bug-Term-ReadLine-Perl [...] rt.cpan.org>
From: Ilya Zakharevich <nospam-abuse [...] ilyaz.org>
On Mon, Jul 03, 2006 at 07:34:45AM -0400, via RT wrote: Show quoted text
> Apologies for the short diagnostic message. At least during testing > with T::R::P 1.0302 for Win32 Vanilla Perl 5.8.8, just pressing Enter > didn't do it.
Looks like a bug. Show quoted text
> (Could this be an issue with newlines?)
Hard to imagine how this could happen... Do not know how to debug it remotely... Yours, Ilya
Subject: Re: [rt.cpan.org #20237] test file prompt needs to include how to exit the prompt
Date: Mon, 03 Jul 2006 17:09:23 -0400
To: bug-Term-ReadLine-Perl [...] rt.cpan.org
From: David Golden <dagolden [...] cpan.org>
Ilya Zakharevich via RT wrote: Show quoted text
> <URL: http://rt.cpan.org/Ticket/Display.html?id=20237 > > > On Mon, Jul 03, 2006 at 07:34:45AM -0400, via RT wrote:
>> Apologies for the short diagnostic message. At least during testing >> with T::R::P 1.0302 for Win32 Vanilla Perl 5.8.8, just pressing Enter >> didn't do it.
> > Looks like a bug.
For reference, I'm running under TERM=dumb. (Not sure why -- I think I needed it at some point in the past.) It looks like the default passed to readline is just ignored in the call to "readline_dumb". More generally, it looks like there are a lot of ways for readline to return without taking into account the default. I only see one small block of code in that section that references $_[1] and I'm not even sure how that generates a default value. Regards, David