Hi Sean,
I think I saw another potential problem, lines 233 to 237 should be
changed from
if (tmpDir.exists()) {
recursiveDelete(tmpDir);
} else {
tmpDir.mkdirs();
}
to
if (tmpDir.exists()) {
recursiveDelete(tmpDir);
}
tmpDir.mkdirs();
to always make sure the temporary path exists.
Sebastian
Sean Owen schrieb:
> That must be it. I had removed the '/' earlier since on OS X the temp
> dir path ends with '/', and at the time I believed it was the cause of
> some other failures (which I'm guessing I was wrong about). I can
> easily make the logic account for both cases.
>
> Sean
>
> On Tue, Apr 6, 2010 at 11:24 AM, Sebastian Schelter
> <sebastian.schel...@zalando.de> wrote:
>
>> Hi Sean,
>>
>> I can only do I guess why the test fails:
>>
>> Line 225 in ItemSimilarityTest is missing a / when constructing the path
>> to the temporary directory:
>>
>> String tmpDirPath = System.getProperty("java.io.tmpdir") +
>> ItemSimilarityTest.class.getCanonicalName();
>>
>> which makes it
>>
>> /tmporg.apache.mahout.cf.taste.hadoop.similarity.item.ItemSimilarityTest
>>
>> on my system for example. Maybe the hudson user on the CI server is not
>> allowed to create this path.
>>
>> Sebastian
>>
>>
