From 92508f95d17cda6a6995cd4650e72c3ac1187c37 Mon Sep 17 00:00:00 2001 From: Nicholas Clark Date: Wed, 13 Jun 2012 16:45:40 +0200 Subject: *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, intending that to call File::stat::stat(). However the refactoring of the test script (all part of the same commit) from C to C (not in a BEGIN block) intentionally eliminated the export of &File::stat::stat. This means that plain C 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 | 1 + 1 file changed, 1 insertion(+) (limited to 'MANIFEST') diff --git a/MANIFEST b/MANIFEST index 86da43ea14..f9cb161108 100644 --- a/MANIFEST +++ b/MANIFEST @@ -4237,6 +4237,7 @@ lib/File/Find/t/find.t See if File::Find works lib/File/Find/t/taint.t See if File::Find works with taint lib/FileHandle.pm Backward-compatible front end to IO extension lib/FileHandle.t See if FileHandle works +lib/File/stat-7896.t A test for ID 20011110.104 lib/File/stat.pm By-name interface to Perl's builtin stat lib/File/stat.t See if File::stat works lib/filetest.pm For "use filetest" -- cgit v1.2.1