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)



Subject: find_tempdir error finding write permission directory
Download (untitled) / with headers
text/plain 370b
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 $_; } =======================================
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>
Download (untitled) / with headers
text/plain 579b
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
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>
Download (untitled) / with headers
text/plain 761b
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
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>
Download (untitled) / with headers
text/plain 615b
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
Download (untitled) / with headers
text/plain 241b
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.
Closing as per reasons listed above.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.