On Fri, 2005-07-15 at 08:27 -0700, Andy Dougherty wrote: > After running 'make test', Parrot leaves the following files lying around > in /tmp. I think there's from t/perl/Parrot_Docs.t.
> There are two problems: > 1. Parrot should clean up after itself. > 2. Parrot should probably pick directory names less > likely to collide with other legitimate uses of /tmp. Agreed. This patch appears to fix #1. If there are no objections in a day or two, I'll apply it. I have an idea on how to fix #2 too. -- c
Index: t/perl/Parrot_Docs.t =================================================================== --- t/perl/Parrot_Docs.t (revision 8629) +++ t/perl/Parrot_Docs.t (working copy) @@ -121,9 +121,9 @@ sub teardown { - Parrot::Docs::File->new(tmp_file_path('file.pod'))->delete; - Parrot::Docs::Directory->new(tmp_dir_path('src')); - Parrot::Docs::Directory->new(tmp_dir_path('tgt')); + Parrot::Docs::File->new(tmp_file_path('file.pod'))->delete(); + Parrot::Docs::Directory->new(tmp_dir_path('src'))->delete(); + Parrot::Docs::Directory->new(tmp_dir_path('tgt'))->delete(); } # tmp_dir_path(@dirs)