https://bz.apache.org/SpamAssassin/show_bug.cgi?id=7986

--- Comment #12 from Sidney Markowitz <sid...@sidney.com> ---
Is there any reason not to use File::Temp::tempdir(CLEANUP => 1) to create a
temporary directory to use for the socket path during the tests? I can see that
we want $workdir created in SATest.pm to be in the test's log directory so we
can conveniently look at test errors. But there is no reason for the tests to
use that for $sockpath, which is the only time the path length is a problem.

In other words, for the various tests that were patched in revision 1900586
change

my $sockpath = $workdir."/spamd.sock";

into something like

my $tempdir = File::Temp::tempdir(CLEANUP => 1);
my $sockpath = $tempdir."/spamd.sock";

Or perhaps define mk_safe_tmpdir() in SATest.pm again, but a smaller one that
just returns File::Temp::tempdir(CLEANUP => 1) and call that in the tests.

$workdir could keep its existing definition for all other uses.

-- 
You are receiving this mail because:
You are the assignee for the bug.

Reply via email to