This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
*Actually* test that bug 20011110.104 is fixed.
authorNicholas Clark <nick@ccl4.org>
Wed, 13 Jun 2012 14:45:40 +0000 (16:45 +0200)
committerNicholas Clark <nick@ccl4.org>
Thu, 21 Jun 2012 06:59:00 +0000 (08:59 +0200)
commit92508f95d17cda6a6995cd4650e72c3ac1187c37
tree6fd2857e2b31b99af9c3cc02eb7832ce17876477
parent17c017a51cc9557d7d50c5113b67db8442762c56
*Actually* test that bug 20011110.104 is fixed.

Bug ID 20011110.104 (RT #7896) was fixed by commit 2f173a711df6278f in Nov
2001, but the test that commit added never actually tested this.

The initial problem was that the new code, as written, used C<stat>,
intending that to call File::stat::stat(). However the refactoring of the
test script (all part of the same commit) from C<use File::stat;> to
C<use_ok( 'File::stat' );> (not in a BEGIN block) intentionally eliminated
the export of &File::stat::stat. This means that plain C<stat> is the
core builtin.

Fixing this as-is to File::stat::stat() won't help, as tests have
subsequently been added earlier in the script that trigger the autoloading
of Symbol by File::stat (commit 83716b1ec25b41f2 in Feb 2002). Moving the
tests earlier won't help now that the test uses File::Temp, as that uses
IO::Seekable which uses IO::Handle, which unconditionally loads Symbol.

The simplest solution seems to be to move the test to its own file.
MANIFEST
Porting/Maintainers.pl
lib/File/stat-7896.t [new file with mode: 0644]
lib/File/stat.t