Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the CGI CPAN distribution.

Report information
The Basics
Id: 95634
Status: rejected
Priority: 0/
Queue: CGI
People
Owner: Nobody in particular
Requestors: cae [...] yandex.ru
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: 3.65
Fixed in: (no value)

History Show all quoted text
  Wed May 14 06:36:08 2014 http://cae.ya.ru/ - Ticket created
Subject: find_tempdir error finding write permission directory
Subroutine find_tempdir has a bug: ============ bug version ============== for (@TEMP) { do {$TMPDIRECTORY = $_; last} if -d $_ && -w _; } ======================================= instead normal: =========== no bug version ============ for (@TEMP) { do {$TMPDIRECTORY = $_; last} if -d $_ && -w $_; } =======================================
  Wed May 14 09:04:13 2014 mark [...] stosberg.com - Correspondence added
Subject: Re: [rt.cpan.org #95634] find_tempdir error finding write permission directory
Date: Wed, 14 May 2014 06:03:36 -0700
To: "http://cae.ya.ru/ via RT" <bug-CGI [...] rt.cpan.org>
From: Mark Stosberg <mark [...] stosberg.com>
Show quoted text
> Subroutine find_tempdir has a bug: > > ============ bug version ============== > for (@TEMP) { > do {$TMPDIRECTORY = $_; last} if -d $_ && -w _; > } > ======================================= > instead normal: > > =========== no bug version ============ > for (@TEMP) { > do {$TMPDIRECTORY = $_; last} if -d $_ && -w $_; > } > =======================================
Using "_" instead of $_ is correct, not a bug. See the bottom of: perldoc -f -X Do you a test built with Test::More that shows that current code fails a test? Mark
  Wed May 14 09:04:13 2014 The RT System itself - Status changed from 'new' to 'open'
  Wed May 14 09:40:14 2014 cae [...] yandex.ru - Correspondence added
Subject: Re: [rt.cpan.org #95634] find_tempdir error finding write permission directory
Date: Wed, 14 May 2014 17:39:48 +0400
To: bug-CGI [...] rt.cpan.org
From: "Sergey A.Eremenko" <cae [...] yandex.ru>
Hello, Mark, 14.05.2014 17:04 you wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=95634 > >
>> Subroutine find_tempdir has a bug: >> >> ============ bug version ============== >> for (@TEMP) { >> do {$TMPDIRECTORY = $_; last} if -d $_ && -w _; >> } >> ======================================= >> instead normal: >> >> =========== no bug version ============ >> for (@TEMP) { >> do {$TMPDIRECTORY = $_; last} if -d $_ && -w $_; >> } >> =======================================
> > Using "_" instead of $_ is correct, not a bug.
Sorry, I undestand it. In my code using &main::_ subroutine. CGI call it during this code. I will rename it for compatible with CGI.pm. -- Regards, Sergey A.Eremenko mailto:cae@yandex.ru
  Wed May 14 10:13:03 2014 mark [...] stosberg.com - Correspondence added
Subject: Re: [rt.cpan.org #95634] find_tempdir error finding write permission directory
Date: Wed, 14 May 2014 07:12:52 -0700
To: "cae [...] yandex.ru via RT" <bug-CGI [...] rt.cpan.org>
From: Mark Stosberg <mark [...] stosberg.com>
Show quoted text
> > Using "_" instead of $_ is correct, not a bug.
> > > Sorry, I undestand it. In my code using &main::_ subroutine. CGI call it > during this code. I will rename it for compatible with CGI.pm.
To be clear, this is not a CGI.pm compatibility issue. You have chosen a name which conflicts with a core perl variable, which any other module that tests files might use as well. Your solution of renaming it will solve it. Good luck with Yandex-- it's always good to have some competition among search engines. (I presume you are working on Yandex, but perhaps you just have a Yandex email address). Mark
  Thu May 22 08:21:55 2014 LEEJO [...] cpan.org - Correspondence added
This issue has been copied to: https://github.com/leejo/CGI.pm/issues/134 please take all future correspondence there. This ticket will remain open but please do not reply here. This ticket will be closed when the github issue is dealt with.
  Thu May 22 11:42:38 2014 LEEJO [...] cpan.org - Correspondence added
Closing as per reasons listed above.
  Thu May 22 11:42:38 2014 LEEJO [...] cpan.org - Status changed from 'open' to 'rejected'