Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Jemplate CPAN distribution.

Report information
The Basics
Id: 23883
Status: resolved
Priority: 0/
Queue: Jemplate

People
Owner: rkrimen [...] cpan.org
Requestors: ishigaki [...] cpan.org
Cc:
AdminCc:

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



Subject: t/list.t fails because perl for Win32 returns 0 by -s 'directory'
Log and patch attached.
Subject: log
Download log
application/octet-stream 1017b

Message body not shown because it is not plain text.

Subject: patch
Download patch
application/octet-stream 536b

Message body not shown because it is not plain text.

Subject: Re: [rt.cpan.org #23883] t/list.t fails because perl for Win32 returns 0 by -s 'directory'
Date: Tue, 19 May 2009 15:39:35 +0200
To: bug-Jemplate [...] rt.cpan.org
From: kmx <kmx [...] volny.cz>
Hi, could you please apply the patch proposed by ISHIGAKI. I would appreciate Jemplate package to be functional on Win32 platform. In my opinion the more proper patch for Jemplate.pm would be: - unless (-e $arg) { next; } # file exists - unless (-s $arg) { next; } # file size > 0 + next unless (-e $arg); # file or directory exists + next if (-f $arg && -z $arg); # if regular file then size > 0 But ISHIGAKI's patch works as well. Thanks -- kmx
kmx, Ishigaki Fixed in commit 3c49d96e16c584cfe3ece83936986e6c6578949a Thanks for the submission